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

DRAGONS: Add localized hardcoded copyright screen and main menu #4589

Merged
merged 9 commits into from
Mar 5, 2023

Conversation

tag2015
Copy link
Contributor

@tag2015 tag2015 commented Jan 3, 2023

This PR applies the following changes to the DRAGONS engine:

  • Shows the localized hardcoded (EN/FR/DE) copyright/splash screen and main menu,
  • Hides the "Previews" menu entry for all versions except US.

Fixes bug 11696

The "spurious pixels" and text background issue 11644 will be addressed
separately


int lastMenuItem = 1;

uint16 copyright[4][11][40] = {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You don't need to convert these to uint16, English, German and French should work fine, and the non-English characters can be escaped.

Check how other engines are handling hardcoded strings, e.g. itedata.cpp in SAGA:

https://github.com/scummvm/scummvm/blob/master/engines/saga/itedata.cpp

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the suggestion, I already tried escaping the special characters but couldn't get it to work.
I'll look into it

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You basically write it like this:

"Blazing Dragons ist gesch\x81""tztes"

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should be fine now :)

@sev-
Copy link
Member

sev- commented Feb 7, 2023

@tag2015 Perhaps, you need a helping hand with converting those strings to a more readable format?

@tag2015
Copy link
Contributor Author

tag2015 commented Feb 7, 2023

Sorry, I put this PR aside as it wasn't really urgent (also I wanted to have a look at the palette issues in the splash screen) and then I forgot about it. I'll see if I can do the changes requested

@sev-
Copy link
Member

sev- commented Feb 25, 2023

@tag2015 I'd love to see it merged :D There were people asking about German version on Facebook.

@tag2015
Copy link
Contributor Author

tag2015 commented Feb 26, 2023

@tag2015 I'd love to see it merged :D There were people asking about German version on Facebook.

Should be ready by tomorrow!

Also update AddAsciiText method to support special characters
@sev-
Copy link
Member

sev- commented Mar 5, 2023

Awesome! Merging.

@sev- sev- merged commit c6e2cfb into scummvm:master Mar 5, 2023
@tag2015 tag2015 deleted the blazing_options branch March 5, 2023 09:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants