-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Add an option to allow players add full Chinese glyphs ranges. #79322
Add an option to allow players add full Chinese glyphs ranges. #79322
Conversation
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Fix words.
Spell checker encountered unrecognized words in the in-game text added in this pull request. See below for details. Click to expand
This alert is automatically generated. You can simply disregard if this is inaccurate, or (optionally) you can also add the new words to Hints for adding a new word to the dictionary
|
Could be hidden behind language value, instead of an option. |
Could you explained this more clearly? I'm sorry that I don't so understand this. Do you mean select whether load glyphs in the language option or a similar option? |
Can you explain in brief why these have been rejected? My understanding (please correct me if I'm wrong) is that even with this PR the game would be broken when played in CN/TW out of the box (i.e. 躯 would not be displayed correctly), and the player would need to explicitly go and enable this option for the game to be playable. Is it perhaps (also?) a translation issue? I.e would it be more appropriate to use something like 胴 instead of 躯 for the "torso"? (Sorry, I don't speak Chinese, just going off of what google translate suggests) |
Truly correct and reasonable. The reason I don't do this is I don't sure if loading a full range have a bad influence in those poor performance device and why former developer load only common glyphs. It will be helpful if someone could tell me the reason why we load glyphs by language used. This seems block the use of foreign mods without i18n.
Actually '躯' is most common translation for 'torso'. And this is a question wider than this character. About 2% ( or less? or more? ) characters used in game are missing in common range. Common range only cover a small part of full Chinese range, and I guess this common mean 'common in daily conversations'. CDDA have a lot of words not so 'common'. |
If this is being left as an option it either needs some way to handle being changed ingame or a note adding to the option's description that it won't change until restart |
This requires restart.
Summary
None
Purpose of change
Two reasons:
Describe the solution
Add an option, to allow players to load full Chinese glyph ranges if they want.
Describe alternatives you've considered
Testing
Use the option and check the glyphs: all Chinese glyphs loaded.
Additional context
I don't sure if similar option could be useful for other language. but this is VERY helpful for Chinese lang to kill those not-in-range glyphs at large.