sezero / quakespasm

QuakeSpasm -- A modern, cross-platform Quake game engine based on FitzQuake.
https://sourceforge.net/projects/quakespasm/
GNU General Public License v2.0
244 stars 97 forks source link

Add new game message hint #82

Closed Diordany closed 1 year ago

Diordany commented 1 year ago

The new game message didn't give a hint as to which button should be pressed to confirm or cancel. This message has been present since the original release by id software, and I thought it helped to simply add "y/n" at the end of the message for anyone who isn't familiar with how the modal message works.

Diordany commented 1 year ago

Yes sorry about that @alexey-lysiuk, my IDE's editor automatically changed the other lines. I'll take care of it.

Also, do you want me to change y/n to (y/n) instead?

alexey-lysiuk commented 1 year ago

Also, do you want me to change y/n to (y/n) instead?

Let's wait for another opinion. For me, it should match with the rest of SCR_ModalMessage() calls.

Diordany commented 1 year ago

Let's wait for another opinion. For me, it should match with the rest of SCR_ModalMessage() calls.

Alright, got it.

sezero commented 1 year ago

Also, do you want me to change y/n to (y/n) instead?

That would be good, yes.

sezero commented 1 year ago

And: squash the commits into a single one, please.

Diordany commented 1 year ago

I've made the changes @sezero.

sezero commented 1 year ago

OK, this is in. Thanks.