-
Notifications
You must be signed in to change notification settings - Fork 60
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
Reconsider the swatch design #19
Comments
Thoughts:
|
Regarding icon contrast too low (#43), it looks like it is not solved in Big Sur 11.3 (tried latest beta). So this redesign of the swatch seems like a better route 👍 |
Massive thanks for @bikrrr for these (edited down) ideas:
Big fan of the idea of moving the format into the toolbar, and I've made a branch to start testing that out. Would love to try that in a beta. This simplification also makes sense to me as part of the swatch redesign considerations. The other thing this helps us resolve is that the more menu causes a bunch of issues (primarily, that component in SwiftUI doesn't have a huge number of API options, which makes it difficult to customise when placed against different colours) so reducing the need for it at all, a ton of pain points will go away. And visually things will be both simplified and more findable. you go from this more menu and preferences, to a switcher for format and a copy button. Win win. 🙌 |
Some pretty significant changes to the UI in |
Yeah, this is something I've been thinking about as well. Was initially intended to aid readability, but had to make some changes to match Apple's defaults for the Touch Bar, so it doesn't make as much sense as before. Having a look as part of the wider swatch considerations. Thanks @ovanbiervliet. |
Great, you can try out an update to this in the |
Perfect! |
Great, this is merged in and will be released with |
From #3 (comment)
The text was updated successfully, but these errors were encountered: