-
-
Notifications
You must be signed in to change notification settings - Fork 184
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
Fix incorrect sidebar and sidebar item color #484
Conversation
…ThemeData` itself to avoid breaking changes
Co-authored-by: Reuben Turner <[email protected]>
(`ignoringSemantics` is deprecated)
I just noticed that version 2.0.2 has been released which caused some merge conflicts. I’ll resolve those soon. |
@Adrian-Samoticha has an issue been filed for this bug? I'd like to see what it looks like prior to your changes - additionally, is this bug only present in light mode, or dark mode as well? |
The sidebar appears darker than it should both in light and dark mode, though it is obviously more noticeable in light mode. The cause is that the Here’s a screenshot from #471. This may not be the best image to showcase the effect, but I think the issue is visible: The sidebar appears visibly darker than it would in a native macOS app. I haven’t seen any issue related to that bug. In fact, I originally only intended to fix the color of the sidebar buttons in this PR, but the incorrect sidebar color was getting in the way of doing that, so I fixed it. |
Gotcha, thanks for the clarification. I'll try to review ASAP. |
Alright, merge conflicts resolved. :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
This PR fixes a bug that caused the sidebar to appear darker than intended and attempts to make the sidebar items mimic the color of macOS’ native sidebar items more closely.
This is what the example app now looks like in light mode with the accent color set to green:
Unfortunately, the colors appear a little off, particularly when the window is placed on top of a dark background. This is because the sidebar is still rendered beneath the sidebar items, which is not the case for native macOS apps. I could not find a solution to this issue.
Also, the color of the sidebar icons has remained the same and will be addressed on a future PR.
Pre-launch Checklist
CHANGELOG.md
with my changes