Bug #11975
Colour picker min values for patch are wrongly displayed
100%
Description
Hi,
This issue is most easily demonstrated on blue sky, but does occur elsewhere. Using colour picker to select an are on the blue sky will report correct values for mean and (I think) max values, but for the minimum it always displays a green patch. If the patch is saved by using the add button then for min value a small green patch is displayed with appropriate RGB values. If the colour picker pipette is now deselected by clicking the small green patch in the live samples area changes to blue with corresponding changed RGB values. The large green patch above remains. 4 files attached show mean value after selecting piece of sky, min value, the saved min value and the change after the colour picker is switched off.
regards,
Jim
Associated revisions
Fix #11975: Min color picker was broken
This fixes a typo, introduced in
27e5861b7d1e7f110ecbb33131bc83f1e8c832ac 6.5 years ago ...
(cherry picked from commit 25050d71b482d595b84267d87c95ba97434b3073)
History
#1
Updated by Jim Robinson almost 2 years ago
On further investigation I see that the the B value of the RGB triplet for the min value of the selected patch is displayed as zero until the colour picker pipette is deselected.
#2
Updated by Jim Robinson almost 2 years ago
Sorry further investigation suggests that it is a little more complicated, but it seems that one of the values of the RGB triplet is initially recorded wrongly, but is corrected in the live samples area once the picker pipette is deselected.
#3
Updated by Tobias Ellinghaus almost 2 years ago
- System changed from Windows to all
- % Done changed from 0 to 20
- Status changed from New to Triaged
#4
Updated by Tobias Ellinghaus almost 2 years ago
- % Done changed from 20 to 100
- Status changed from Triaged to Fixed
Applied in changeset darktable|25050d71b482d595b84267d87c95ba97434b3073.
#5
Updated by Roman Lebedev almost 2 years ago
- Target version set to 2.6.0
Fix #11975: Min color picker was broken
This fixes a typo, introduced in
27e5861b7d1e7f110ecbb33131bc83f1e8c832ac 6.5 years ago ...