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

Curve pipette misbehaving when using new zoom levels #3861

Closed
Beep6581 opened this issue May 4, 2017 · 5 comments
Closed

Curve pipette misbehaving when using new zoom levels #3861

Beep6581 opened this issue May 4, 2017 · 5 comments
Assignees
Labels
type: bug Something is not doing what it's supposed to be doing

Comments

@Beep6581
Copy link
Owner

Beep6581 commented May 4, 2017

  1. Download http://rawtherapee.com/shared/test_images/colortarget/NIKON%20D600%20daylight.nef
  2. Open, apply neutral.
  3. Custom curve, enable pipette
  4. Zoom to fit
  5. Hover over the bottom gray row of patches. You should see the curve position indicator (vertical line) stay static as you move your cursor over a single patch, and the position of the indicator should coincide with the value of the patch. Instead what I see is the indicator move about wildly as I move the pipette over a single patch. e.g. hovering the pipette over the brightest patch (bottom-left) and moving the pipette from the center of the patch to the lower-right edge I see the indicator move from the dark tones of the histogram all the way to the light tones.

It works correctly when I zoom using the standard zoom factors - 16%, 20%, 25%, 33%, 50%, 100%, 200%, etc.
It does not work when using the new zoom levels, e.g. 18%.
screenshot_20170504_143538
screenshot_20170504_143558
screenshot_20170504_143813

@Beep6581 Beep6581 added the type: bug Something is not doing what it's supposed to be doing label May 4, 2017
@heckflosse
Copy link
Collaborator

Confirmed

@agriggio
Copy link
Contributor

agriggio commented May 4, 2017

investigating...

@agriggio
Copy link
Contributor

agriggio commented May 4, 2017

I just pushed a fix, please confirm (reopening in the meantime)

@agriggio agriggio reopened this May 4, 2017
@heckflosse
Copy link
Collaborator

Fix confirmed. Thank you 👍

@Beep6581
Copy link
Owner Author

Beep6581 commented May 4, 2017

Fix confirmed, thank you.

@agriggio agriggio closed this as completed May 5, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug Something is not doing what it's supposed to be doing
Projects
None yet
Development

No branches or pull requests

3 participants