Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactor MenuSeeder #115

Open
janekx21 opened this issue Aug 31, 2020 · 1 comment
Open

Refactor MenuSeeder #115

janekx21 opened this issue Aug 31, 2020 · 1 comment

Comments

@janekx21
Copy link
Owner

Die Datei MenuSeeder.kt ist voller redundanter stellen. Zudem sind die Komponenten nicht auf einzelne Gameobjekte aufgeteilt.

Das ziel des Tasks ist es ein Refactoring wie beim #114 umzusetzen um die Datei auf zu brechen.

@schaumbu
Copy link
Collaborator

schaumbu commented Sep 1, 2020

@janekx21 Die Umsetzung und das Wechseln zwischen Menüs ist einfacher, wenn ein alle GUI Elemente zu einem GameObjekt zusammengefasst werden. Die Redundanz kann man mit Factorys rausmachen - da geb ich dir recht.
Wenn man jedoch alle GUI Elemente als einzelne GameObjekts speichert und das Menü wechseln möchte, muss man einzeln hinterlegen, welche GameObjects removed werden sollen und welche noch bleiben müssen.
Mit dieser Variante muss einfach nur Scene.active.remove(this) in einen Menüwechsel-callback geschrieben werden.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants