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

Swap Positions of Secret and Level Title Messages #2039

Open
OpenRift412 opened this issue Nov 20, 2024 · 3 comments · May be fixed by #2064
Open

Swap Positions of Secret and Level Title Messages #2039

OpenRift412 opened this issue Nov 20, 2024 · 3 comments · May be fixed by #2064

Comments

@OpenRift412
Copy link

I always considered it to make more sense to have the map title announcement be front and center instead of being a normal message, a la EDuke32. Having the secret announcements be shown as a normal message is more fitting, given it matches the more modern standard set by the Unity and Kex ports.

I would also suggest tying both to the "colorize messages" option, having them otherwise use the native text color when the option is disabled.

@fabiangreffrath
Copy link
Owner

The "secret revealed" message comes from Hexen and has been in Doom ports like this since literally decades.

@OpenRift412
Copy link
Author

The "secret revealed" message comes from Hexen and has been in Doom ports like this since literally decades.

Fair point. Then I would just suggest making the map title use the same space and the colorize thing, then.

@fabiangreffrath
Copy link
Owner

Fair point. Then I would just suggest making the map title use the same space and the colorize thing, then.

The map announcement can get really long if the "author" field is filled in the UMAPINFO lump. I wouldn't want to occlude the player's view with such a wall of text in the midle of the screen right at the map start.

However, it has been reported that the map announcement gets cut short by the autosave message. 🙄

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

Successfully merging a pull request may close this issue.

2 participants