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

UI text #4648

Closed
wants to merge 16 commits into from
Closed

UI text #4648

wants to merge 16 commits into from

Conversation

bubblobill
Copy link
Collaborator

@bubblobill bubblobill commented Jan 17, 2024

Identify the Bug or Feature request

progresses #4600

Description of the Change

Typo, spelling, grammar fixes, etc.

Possible Drawbacks

None

Documentation Notes

N/A

Release Notes

N/A


This change is Reviewable

…oup.

Refactored CampaignPropertiesDialog string keys to have consistent case.
Renamed import predefined key to be more explicit.
Reworked light/sight help text to include translations alongside literal strings.
Added some documentation for one of the obscure string operations.
---
Hyphenated stat-sheet
Campaign Properties Dialog reworked:
Centred Token Types label in token props panel over the types list.
Removed trailing colons from all labels.
Right-aligned all labels to associated input fields.
Renamed key for "import predefined" button for more specificity.
Passing all tests.
- Right-align labels
- Remove trailing colons
- Align token property type label
Hyphenation
…UI_Text

� Conflicts:
�	src/main/java/net/rptools/maptool/client/ui/campaignproperties/CampaignPropertiesDialogView.form
�	src/main/java/net/rptools/maptool/client/ui/campaignproperties/CampaignPropertiesDialogView.java
�	src/main/resources/net/rptools/maptool/language/i18n.properties
�	src/main/resources/net/rptools/maptool/language/i18n_en_au.properties
�	src/main/resources/net/rptools/maptool/language/i18n_en_gb.properties
@bubblobill bubblobill closed this Jan 23, 2024
@bubblobill bubblobill deleted the UI_Text branch January 23, 2024 05:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant