Skip to content
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

color calibration module - improve white balance picker handling #18299

Open
rgr59 opened this issue Jan 26, 2025 · 1 comment
Open

color calibration module - improve white balance picker handling #18299

rgr59 opened this issue Jan 26, 2025 · 1 comment
Labels
feature: redesign current features to rewrite scope: UI user interface and interactions

Comments

@rgr59
Copy link

rgr59 commented Jan 26, 2025

The white balance picker of the color calibration module can lead to strange results if the user is not aware that he has set a target color different from neutral. This is well documented, but every so often people (me included) stumble over this. There is an "active" appended to the label of the "area color mapping" collapsible section if the values differ from neutral since some time now, but it can easily be overlooked.

This could be improved by a change in the color calibration GUI:

  • the current picker always and only sets the white balance using neutral values
  • in the "area color mapping" collapsible section, a second picker is added that, depending on "area mode", picks a color from a source image or applies the color to target images.

Of course this would be a change of the GUI and I don't know if this is wanted. But it could help avoid mistakes: a user wanting to transfer colors between images must expand the section to get hold of the picker, so he is automatically aware of this functionality and also sees the currently configured target values.

@ralfbrown ralfbrown added the scope: UI user interface and interactions label Jan 26, 2025
@elstoc
Copy link
Contributor

elstoc commented Feb 1, 2025

I suggested this same UI when the area color mapping feature was introduced and the original author of the functionality strongly objected (though he's left the project now). I very much support this feature request and IMO something similar should be done in the exposure module (in which case the main picker should set the exposure to 50%).

@elstoc elstoc added the feature: redesign current features to rewrite label Feb 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature: redesign current features to rewrite scope: UI user interface and interactions
Projects
None yet
Development

No branches or pull requests

3 participants