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

feat: added customization options for Window Titles + ... #128

Merged
merged 6 commits into from
Jul 10, 2024

Conversation

chrisharper22
Copy link
Contributor

@chrisharper22 chrisharper22 commented Jul 10, 2024

This PR adds customization options for Window Titles: where they are shown and if shown at all.

Second, this PR renames the views for Settings in order to create consistency.

Additionally, this PR addresses some verbiage changes that I believe add clarification to what they do. (Note: on existing string keys, I only updated the English locale to prevent breaking other locales; this should either be updated in all locales or have the direct key(s) changed as well.)

Edit: also, apologizes — this PR contains multiple scopes: feature additions & minor code refactoring.

chrisharper22 and others added 6 commits July 9, 2024 19:24
removed AppearanceView.swift reference from project root
Renamed view structures to be consistent
add several more options for Window Title settings:
- Enable/disable
- Show in Window Switcher, Dock Previews, or Always
Updates verbiage used for various appearance settings for clarification
@chrisharper22 chrisharper22 changed the title Feature: added customization options for Window Titles + ... feat: added customization options for Window Titles + ... Jul 10, 2024
@chrisharper22
Copy link
Contributor Author

Also, should proper string keys be considered?

@ejbills ejbills mentioned this pull request Jul 10, 2024
@ejbills ejbills merged commit 4ed3390 into ejbills:main Jul 10, 2024
3 of 4 checks passed
@chrisharper22 chrisharper22 deleted the appearance-changes branch July 10, 2024 01:16
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.

2 participants