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

StandardNodeGadget : Remove raster-space sizing for focus border #5468

Merged
merged 1 commit into from
Sep 25, 2023

Commits on Sep 20, 2023

  1. StandardNodeGadget : Remove raster-space sizing for focus border

    We were scaling the size up to make the focus icon more prominent when the graph editor was zoomed out. But this had the unwanted side effect of obscuring more parts of the underlying UI when zoomed out. So while managing focus was easier, some other operations - such as making a connection - became _impossible_ at some zoom levels. This appears to have been further exacerbated by a bug whereby the default scaling was not consistent between 2K and 4K displays, as reported in GafferHQ#5435, making some operations impossible at common zoom levels.
    
    Removing this dynamic sizing does make focus management a little harder, but the fixed sizing maintains the relative importance of UI elements across all zoom levels. Let's see how folks get on with it.
    
    Fixes GafferHQ#5435.
    johnhaddon committed Sep 20, 2023
    Configuration menu
    Copy the full SHA
    16c1a3a View commit details
    Browse the repository at this point in the history