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

[#40] Localization Revisions, Preparation, and Maintenance #41

Merged
merged 21 commits into from
Nov 15, 2023

Conversation

kgar
Copy link
Owner

@kgar kgar commented Nov 11, 2023

#40

Major Localization / Settings Overhaul

Warning

Breaking changes: Numerous config settings have changed meaning and name. Any setting whose name has changed will reset to its default value. If your sheet setup has changed when upgrading, visit Sheet Settings and/or Theme Settings to re-apply your desired settings.

  • All localization files have been synchronized to the en.json lang file.
  • Numerous settings have been revised to be more positive-oriented (e.g., changing "Disable health bar." to "Use HP Bar")
  • All settings have been given "Title Case" to keep with Foundry / dnd5e setting name conventions
  • Numerous setting descriptions have been shortened, revised for clarity, and revised to remove advice about other modules or how to use the settings. They have instead been fashioned as simply a cafeteria plan of features.
  • The Settings Info tab has been filled out information and links.

Coming Soon: Weblate for Tidy 5e Sheets

As part of this upgrade, @kgar is setting up the Tidy 5e Sheets rewrite in Foundry Hub Weblate so that localization is easier and doesn't require translators to fuss with Github or pull requests. Weblate also provides a number of compelling quality-of-life features.

Delineated Sheet Edit from Sheet Lock.
@kgar kgar self-assigned this Nov 11, 2023
kgar and others added 8 commits November 12, 2023 11:51
Revamped the context menu actions and other localization variables to say less.

Compacted some composite variables that were structurally imposing unintentional language conventions on potential translations.

Applied some light revisions to wording and capitalization.

Made search-related localization say less.
Refactored freeze key to be fix key in the ItemInfoCard component.

Reversed wording on inspiration tracker animation so that it is positive-oriented rather than a disabling feature.

Standardized NPC and vehicle setting headers. Removed unused variables.
Finished initial pass reviewing and revising grammar and word choice.

Renamed numerous settings.

Reversed the logic on numerous settings so that they are positive-oriented to simplify their use and stick to the convention.

Separated standard encumbrance into character and vehicle encumbrance settings to make configuration simpler and less about other modules and more about whether to show these sheet parts. This conceptually removes them from being specifically for the purpose of rule variance or alternative encumbrance modules.
Updated module.json with lang files.
Swapped out top-level localization namespace for the temp alpha testing one.
Synchronized all lang files with en.json.

Reversed settings:
- characterJournalTabDisabled -> useJournalTabForCharacter
- journalTabNPCDisabled -> useJournalTabForNpc

Renamed settings:
- enableClassicControlsForCharacter -> useClassicControlsForCharacter
- enableClassicControlsForNpc -> useClassicControlsForNpc
- enableClassicControlsForVehicle -> useClassicControlsForVehicle

Added context settings where appropriate. Sorted type declaration props accordingly.
- traitLabelsEnabled -> showTraitLabels

Reversed
- classListDisabled -> showClassList

Cleared all localization hints and labels for showClassList since it has fundamentally changed in meaning.
@kgar kgar added cleanup Code cleanup, refactoring, restructuring, etc. localization labels Nov 14, 2023
@kgar kgar marked this pull request as ready for review November 15, 2023 16:22
@kgar kgar merged commit 6fbf08a into main Nov 15, 2023
1 check passed
@kgar kgar deleted the alpha-40-localization branch November 15, 2023 16:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cleanup Code cleanup, refactoring, restructuring, etc. localization
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant