-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Added automatic dark note on light background for midi clips. #6539
Conversation
Thanks! Can you share a screenshot of it working for various colors both light and dark? |
I feel the muted clips contrast on grey isn't as good as it could be. Not sure it's a problem, but what would happen if we tried to make those notes get slightly lighter? It would have an effect on the rest, but curious how much. |
This is great. Personally I don't see an issue with the contrast, but if we change it, it should be done inside IMHO (not necessarily related to this PR) muted clips should always turn grey, no matter what color they have. Also when selecting clips they should always turn blue. Currently custom colored clips are just darkened when selected or muted... see |
I stressed through the review yesterday and had a second look. The demo track Greippi - Krem Kaakkuja has colored tracks so it lends itself to comparing before/after this fix. Top row ------- default theme - unmuted/muted, classic theme - unmuted/muted |
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.
See comment #6539 (comment)
In b224f4d the colors of the unmuted tracks are now darker/less visible.
… prefer lighter notes in order to maintain the original look and feel.
Looks good to go! Caveat, I haven't tried it in any other themes than the ones we ship but this looks like a nice change to me. |
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.
The code looks fine, and works with the default themes. A more thorough implementation could use the luminance of the colour rather than the HSL lightness, and compare it to the luminance of the note colour rather than a fixed constant.
Closes #6526