search

Brookke / SEPR-Assessment2

1 stars 6 forks source link

Create main menu #59

Closed TheProgrammingDuck closed 7 years ago

joeShuff commented 7 years ago

Is this for the designing or the actual development?

Brookke commented 7 years ago

Actual development I believe, although I am not sure if it is needed in our brief?

joeShuff commented 7 years ago

Definitely think we should do it. Are we using those things you pinned on slack? Probably worth posting the link here aswell

TheProgrammingDuck commented 7 years ago

http://www.tempoparalelo.com/blog/?p=76 You mean this one? There's some decent youtube tutorials on how to do it aswell.

Brookke commented 7 years ago

Okay just commit small and often, also make sure its using scene2D stages and ui like this http://www.tempoparalelo.com/blog/?p=76 as the seems like one of the better ways of doing it

Brookke commented 7 years ago

@TheProgrammingDuck Are you free tonight I can come on join me and we can go through the main menu if you want?

Brookke commented 7 years ago

Any updates @TheProgrammingDuck

Brookke commented 7 years ago

So I imagine the main menu will be a simple screen with just the 3 buttons for now:

jasonmash commented 7 years ago

For the implementation of the main menu, can we structure the code as follows:

You're probably wondering why I've suggested separating the MainMenuScreen and the MainMenu element classes - the reason for this is for reusability. We've been discussing having a pause menu, and it is very likely that this will have much of the same functionality as the Main Menu, and would make sense to reuse the code. Splitting up the UI elements from the screen logic will help with that. Also, in general, splitting up code is great for maintainability!

Is that okay with you @TheProgrammingDuck? Also @Brookke @joeShuff and the rest of the team, if you've got ideas for how the Main Menu should be implemented, I'd suggest posting them here. It'd be good to get this right first time and not have the room situation all over again 😃

Brookke commented 7 years ago

Yeah it makes sense to me :)

joeShuff commented 7 years ago

Sounds fine to me.

TheProgrammingDuck commented 7 years ago

Fully agree on this. To make it more maintainable, I'll be making skins for the buttons. Will be making commits soon.

TheProgrammingDuck commented 7 years ago

Main menu has been made and fully works. Could you guys have a look on my branch and help make some design decisions? Was thinking of putting a title at the top then seeing if we can get the animated duck that Brooke previously made for the website onto the menu as well.

Brookke commented 7 years ago

So I personally think that it should be in a splash screen that once complete changes the screen to the main menu, but the rest of the team needs to discuss this too.

Joe started work on the duck thing however it ended up in the menu element which means it'd be there if we used the menu element for pausing as well. I have created a branch called Joe-Splash-screen-reference that way we still have the code. I have reverted vishals branch so he can carry on as normal for now then we can look at adding it somewhere at a later date.

One thing I will point out that I know @jasonmash will too 😜 is that this isn't needed for now, however it would be nice yes as its a nice intro to the game.

jasonmash commented 7 years ago

@TheProgrammingDuck How are you getting on with this? It'd be good to see a new pull request with those requested changes from #92 soon

Brookke commented 7 years ago

Done in #96