-
Notifications
You must be signed in to change notification settings - Fork 889
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 #1774 - Dropdown doesn't match theme in dark mode #2199
Conversation
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.
- Its almost invisible in the black theme
- in the dark theme the items in the list dont get highlighted anymore
on another note: dont just ignore the template. u can just remove headers that are unnecessary to you and fill in the other parts. See example
Pull request was converted to draft
I fixed the highlight in dark theme, but I don't know how should i fix the invisible problem in black theme. The way I proposed is to change the |
Sure, we could definitely give that a try |
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.
Looks Good To Me, Thank you for your contribution!
Edit: We need 3 reviewers in order to get this merged but dont let that (slow) process discourage u in contributing more if u want.
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.
Tested dark & black & light mode
Video for dark mode on fixed menu:
Screen.Recording.2022-05-13.at.09.20.24.mov
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
Fix playlist dropdown menu theme not match
Pull Request Type
Related issue
Closes #1774
Description
Screenshots (if appropriate)
Desktop