Add an option to show or hide the menu #14
Conversation
|
I've tested this and can confirm it works, however, the F5 Key is already taken in the code, and the menu show/hide key wasn't added to the keyboard switch statement in options.cpp, after fixing these issues (set show ui to F9 in my case), it works. |
Thanks for the feedback! i've fixed both of these issues 🙂 didn't saw the F5 was already taken, sorry. |
SpaceCadetPinball/winmain.cpp
Outdated
| @@ -573,6 +584,9 @@ int winmain::event_handler(const SDL_Event* event) | |||
| pause(); | |||
| options::keyboard(); | |||
| break; | |||
| case SDLK_F0: | |||
There was a problem hiding this comment.
This should be SDLK_F9, F0 is not defined.
|
Other than F9 issue, everything else looks good to me, I will accept this. |
prototux
force-pushed
the
option_nogui
branch
from
27703fd
to
89f4b6d
Compare
And that's why coding at 3AM is a bad idea 😅 it should be fixed if i didn't messed up the history with the amend. |
|
Thank you, merged. |
This add an checkbox (in the menu itself, with key F5 as well) "Show Menu", enabled/checked by default.
This option is useful in case the pinball is the only item displayed on the screen, in my case, as a fun addition on my mini arcade machine (https://i.imgur.com/nz3u48A.jpg) 🙂
Didn't tested it yet, i'll update the PR after it has been tested