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

Adjusting the width of the NodeGraph affects the zoom #10

Closed
bentoogood opened this issue Mar 27, 2013 · 4 comments · Fixed by #729
Closed

Adjusting the width of the NodeGraph affects the zoom #10

bentoogood opened this issue Mar 27, 2013 · 4 comments · Fixed by #729
Assignees
Labels
bug Issues representing bugs

Comments

@bentoogood
Copy link
Contributor

Should be more like cropping.

@ghost ghost assigned johnhaddon Mar 27, 2013
@johnhaddon
Copy link
Member

This relates to #46 as they both require adjustments to the IECore::CameraController.

@ghost ghost assigned johnhaddon May 10, 2013
@johnhaddon
Copy link
Member

Does this ticket also apply to the image and scene views in the viewer? They also scale rather than crop...

@thiasbxl
Copy link

thiasbxl commented Feb 3, 2014

As of Gaffer 0.89.0, adjusting the size of the node graph also moves the center of the focus. This result in the graph suddenly focusing in a slightly different area, as if it was snapping to some sort of grid. Note sure why that is but I suspect there is something not quite working with this ticket. I think the previous behavior was less disrupting than this new one, would be great if this could be fixed.
To repro :
pan AND adjust the zoom in the graph, only then resize the graph and you'll see the graph focus center snappingto a different location.

Thx
-M

@johnhaddon
Copy link
Member

Sorry about that Mathias - I made a noddy error in the crop calculation, and failed to spot it in my testing. This Cortex pull request should fix the issue :

ImageEngine/cortex#218

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Issues representing bugs
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants