JeLLyNinjas / TerminalFighter

5 stars 2 forks source link

Tai/main menu options #106

Closed ctlai95 closed 6 years ago

bseto commented 6 years ago

I like it. However, I feel like you can make this more generic. We will add more options in the future. I know at the moment, our config files are used for user configurations, but I think we will also one day have our own developer configs where it's like "encrypted". (So maybe have all the values of title, start, and quit in a config).

Maybe for now, we can keep it like this as @enochtsang may argue that this is ok for the main menu.

But I feel in the future, we should have a pretty generic menu object where you can give it an array of menu options, and it'll just dynamically build the the menu.

The options could include (as an example):

ctlai95 commented 6 years ago

I really like those ideas, let's discuss it more in the future. Merging this to the main branch for now.