fheroes2
fheroes2 copied to clipboard
Make building names lowercased when possible
Preliminary checks
- [X] I've checked that there aren't other open issues on the same topic.
Describe the problem requiring a solution
Now that lowercasing of strings were implemented in https://github.com/ihhub/fheroes2/pull/4976 more strings can be made lowercase to stay consistent with the implementation in the aformentioned PR.
- [ ] This could be done for buildings. Build %{BuildingName}. These are not lowercased in the Original Game (OG):
Completed tasks
-
[x] Select {Monster} - This is lowercased in the OG
-
[x] Recruit {Monster} . There are at least two of them:
This second one is not in lowercase in the OG but the one above is.
-
[x] Many strings in battle log as well could use this. These are lowercased in the OG.
Describe the possible solution
You basically just need to use Translation::StringLower()
like done in https://github.com/ihhub/fheroes2/pull/4976/files#diff-a4e630f2f0de5508d00515b54754d4a2e5db85014707a61735b2bc5ef4024769R287
There might need to be made extra contexts for the various strings to avoid a sentence starting with a lowercased string.
If we want to keep some of these strings in uppercase to keep in line with the OG, then maybe the best solution is to make it possible to specify it for different languages. Code that does something related to this for German is already present.
Additional info
No response
@zenseii.
For my part, I prefer to keep the names of creatures, buildings, etc. with the first letter capitalized (at least for the English version). I find it more pleasing to the eye, especially in the "titles". And it also makes it easier to find them in a text. For other languages, it could be adjusted to fit the text conventions.
There had already been discussions or remarks on this subject if I remember correctly (no time to search). :)