-
-
Notifications
You must be signed in to change notification settings - Fork 419
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
Do not blend color in QtColorBox with black using opacity #5827
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
There is a need to update tests also |
Fixed |
Codecov Report
@@ Coverage Diff @@
## main #5827 +/- ##
==========================================
+ Coverage 89.86% 89.91% +0.04%
==========================================
Files 615 615
Lines 52474 52473 -1
==========================================
+ Hits 47158 47182 +24
+ Misses 5316 5291 -25
|
Czaki
approved these changes
May 12, 2023
Czaki
pushed a commit
that referenced
this pull request
Jun 19, 2023
…ion; bugfix? ) (#5827) # Description For some reason that I cannot fully understand, the displayed color in QtColorBox is multiplied with the layer opacity, which is equal to blending the selected color with black. I find it very misleading as it just distorts displayed colors making them harder to associate with the true colors. It becomes especially confusing when you use low levels of opacity (20-30%), which makes all colors look grey and indistinguishable. How it now looks like in napari depending on different opacity: ![image](https://github.com/napari/napari/assets/5459767/01b55cfd-6ce4-494d-a0fa-a8116ee35d6b) This PR just removes it making it always show the actual color. P.S. It has been puzzling me for a while why colors are always darker than the real colors in the color box. I've only recently realized that their appearance is depending on the opacity. ## Type of change - [X] Bug-fix (non-breaking change which fixes an issue) ## Final checklist: - [X] My PR is the minimum possible work for the desired functionality
Czaki
pushed a commit
that referenced
this pull request
Jun 21, 2023
…ion; bugfix? ) (#5827) # Description For some reason that I cannot fully understand, the displayed color in QtColorBox is multiplied with the layer opacity, which is equal to blending the selected color with black. I find it very misleading as it just distorts displayed colors making them harder to associate with the true colors. It becomes especially confusing when you use low levels of opacity (20-30%), which makes all colors look grey and indistinguishable. How it now looks like in napari depending on different opacity: ![image](https://github.com/napari/napari/assets/5459767/01b55cfd-6ce4-494d-a0fa-a8116ee35d6b) This PR just removes it making it always show the actual color. P.S. It has been puzzling me for a while why colors are always darker than the real colors in the color box. I've only recently realized that their appearance is depending on the opacity. ## Type of change - [X] Bug-fix (non-breaking change which fixes an issue) ## Final checklist: - [X] My PR is the minimum possible work for the desired functionality
Czaki
pushed a commit
that referenced
this pull request
Jun 21, 2023
…ion; bugfix? ) (#5827) # Description For some reason that I cannot fully understand, the displayed color in QtColorBox is multiplied with the layer opacity, which is equal to blending the selected color with black. I find it very misleading as it just distorts displayed colors making them harder to associate with the true colors. It becomes especially confusing when you use low levels of opacity (20-30%), which makes all colors look grey and indistinguishable. How it now looks like in napari depending on different opacity: ![image](https://github.com/napari/napari/assets/5459767/01b55cfd-6ce4-494d-a0fa-a8116ee35d6b) This PR just removes it making it always show the actual color. P.S. It has been puzzling me for a while why colors are always darker than the real colors in the color box. I've only recently realized that their appearance is depending on the opacity. ## Type of change - [X] Bug-fix (non-breaking change which fixes an issue) ## Final checklist: - [X] My PR is the minimum possible work for the desired functionality
Czaki
pushed a commit
that referenced
this pull request
Jun 21, 2023
…ion; bugfix? ) (#5827) # Description For some reason that I cannot fully understand, the displayed color in QtColorBox is multiplied with the layer opacity, which is equal to blending the selected color with black. I find it very misleading as it just distorts displayed colors making them harder to associate with the true colors. It becomes especially confusing when you use low levels of opacity (20-30%), which makes all colors look grey and indistinguishable. How it now looks like in napari depending on different opacity: ![image](https://github.com/napari/napari/assets/5459767/01b55cfd-6ce4-494d-a0fa-a8116ee35d6b) This PR just removes it making it always show the actual color. P.S. It has been puzzling me for a while why colors are always darker than the real colors in the color box. I've only recently realized that their appearance is depending on the opacity. ## Type of change - [X] Bug-fix (non-breaking change which fixes an issue) ## Final checklist: - [X] My PR is the minimum possible work for the desired functionality
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
For some reason that I cannot fully understand, the displayed color in QtColorBox is multiplied with the layer opacity, which is equal to blending the selected color with black. I find it very misleading as it just distorts displayed colors making them harder to associate with the true colors. It becomes especially confusing when you use low levels of opacity (20-30%), which makes all colors look grey and indistinguishable.
How it now looks like in napari depending on different opacity:
![image](https://private-user-images.githubusercontent.com/5459767/237683952-01b55cfd-6ce4-494d-a0fa-a8116ee35d6b.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjIzNjA2MTQsIm5iZiI6MTcyMjM2MDMxNCwicGF0aCI6Ii81NDU5NzY3LzIzNzY4Mzk1Mi0wMWI1NWNmZC02Y2U0LTQ5NGQtYTBmYS1hODExNmVlMzVkNmIucG5nP1gtQW16LUFsZ29yaXRobT1BV1M0LUhNQUMtU0hBMjU2JlgtQW16LUNyZWRlbnRpYWw9QUtJQVZDT0RZTFNBNTNQUUs0WkElMkYyMDI0MDczMCUyRnVzLWVhc3QtMSUyRnMzJTJGYXdzNF9yZXF1ZXN0JlgtQW16LURhdGU9MjAyNDA3MzBUMTcyNTE0WiZYLUFtei1FeHBpcmVzPTMwMCZYLUFtei1TaWduYXR1cmU9ODU0MzQ0NWIzNzMwODVjMGRiNTQ5YTI2MGM5MzFlMzA5ZTk3MjYxZGYzNWRkNDY4NDRjMWExNjdiOTU1ZGNjNCZYLUFtei1TaWduZWRIZWFkZXJzPWhvc3QmYWN0b3JfaWQ9MCZrZXlfaWQ9MCZyZXBvX2lkPTAifQ.jB49IXez32sQXRYJnkR_hLrHo-TXcS6Hi_MlxurOGSY)
This PR just removes it making it always show the actual color.
P.S. It has been puzzling me for a while why colors are always darker than the real colors in the color box. I've only recently realized that their appearance is depending on the opacity.
Type of change
Final checklist: