Skip to content

Graphics option name fix - #2700

Open
dmgreeny wants to merge 3 commits into
TheSuperHackers:mainfrom
dmgreeny:Display-options-fix
Open

dmgreeny wants to merge 3 commits into
TheSuperHackers:mainfrom
dmgreeny:Display-options-fix

Conversation

@dmgreeny

Copy link
Copy Markdown
Contributor

No description provided.

@dmgreeny dmgreeny added Minor Severity: Minor < Major < Critical < Blocker Text Is text, string, localization related labels Mar 20, 2025
@dmgreeny

Copy link
Copy Markdown
Contributor Author

Ready to merge

BP: "OPÇÕES DE VÍDEO AVANÇADAS"
PL: "Zaawansowane opcje wyświetlania"
US: "ADVANCED GRAPHICS SETTINGS"
DE: "ERWEITERTE GRAFIKEINSTELLUNGEN"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

You made some of these string upper case. Is this consistent with the other titles of the respective languages? Aka does DE have upper case titles for other screens? If not, then this would introduce an inconsistency.

Concerns DE and PL.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Can you please check DE with this branch. It doesnt load for me. All others work fine

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Fixed

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

All yaml files are lower case.

---
date: 2025-03-27

title: Changes Graphics option string

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

"Improves texts in graphics options screen"



changes:
- fix: Improved text in Graphics option string

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

  • tweak: Improves the title of the graphics options screen for all languages.

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

Labels

Minor Severity: Minor < Major < Critical < Blocker Text Is text, string, localization related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants