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

[Enhancement]: Ventura-style Settings overhaul #141

Merged
merged 13 commits into from
May 10, 2023
Merged

Conversation

cryptoAlgorithm
Copy link
Member

@cryptoAlgorithm cryptoAlgorithm commented Apr 22, 2023

Overhauls Swiftcord's settings to closely match macOS Ventura's new settings style. Adopting

Screenshots

Some eye-candy 👁️🍬

Account

Screenshot 2023-05-10 at 2 23 03 PM

Appearance

Screenshot 2023-05-10 at 2 16 44 PM

About

about

Credits

Screenshot 2023-05-10 at 2 18 29 PM

Debug

Screenshot 2023-05-10 at 2 19 04 PM

The behaviour of the Settings window and navigation is complete, however the detail views themselves still need some work.

Resolves #142

force-unwrapping an optional to find the underlying Settings NSWindow caused a crash on macOS 12

fixes #144
As far as possible, the Form + Section pattern has been used for all settings views
patch(settings): update sidebar icon for “voice and video”
* fix some UI issues in the Accessibility page
* move sidebar toggle hide code to View extension
@cryptoAlgorithm cryptoAlgorithm marked this pull request as ready for review May 10, 2023 06:21
@cryptoAlgorithm
Copy link
Member Author

Looking back at where we were, it certainly looks way more polished now XD


How Settings looked just a few commits ago

@cryptoAlgorithm
Copy link
Member Author

I'd like to specially thank the amazing folks at CodeEdit, the corresponding PR has hinted me in the right direction especially in the last few commits!

PS: This PR actually started as a test bed for a contribution to CodeEdit, unfortunately I never got the time to merge my code into the CE codebase so I just passed them what I had worked on thus far it's been kindly merged in and built on by some really great devs.

@cryptoAlgorithm cryptoAlgorithm merged commit 81cedba into main May 10, 2023
@cryptoAlgorithm cryptoAlgorithm deleted the settings branch May 10, 2023 14:39
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.

[Enhancement]: Ventura-styled Settings
1 participant