Merge pull request #5468 from johnhaddon/focusIconSizing

StandardNodeGadget : Remove raster-space sizing for focus border
GafferHQ · Sep 25, 2023 · 1e465b0 · 1e465b0
2 parents 1f61561 + 16c1a3a
commit 1e465b0
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 6 deletions.
diff --git a/Changes.md b/Changes.md
@@ -1,6 +1,10 @@
 1.2.10.x (relative to 1.2.10.3)
 ========
 
+Fixes
+-----
+
+- GraphEditor : Removed dynamic raster-space sizing of focus icon, as it caused excessive overlap with other nodes at certain zoom levels and on certain high resolution displays (#5435).
 
 1.2.10.3 (relative to 1.2.10.2)
 ========

diff --git a/src/GafferUI/StandardNodeGadget.cpp b/src/GafferUI/StandardNodeGadget.cpp
@@ -778,12 +778,7 @@ void StandardNodeGadget::renderLayer( Layer layer, const Style *style, RenderRea
 
 float StandardNodeGadget::focusBorderWidth() const
 {
-	// Compute a fixed size in raster space, clamped to a maximum
-	const ViewportGadget *viewport = ancestor<ViewportGadget>();
-	const V3f p0 = viewport->rasterToGadgetSpace( V2f( 0 ), this ).p0;
-	const V3f p1 = viewport->rasterToGadgetSpace( V2f( 0, 1.0f ), this ).p0;
-	float pixelSize = ( p0 - p1 ).length();
-	return min( g_maxFocusWidth, max( 0.75f, 8.0f * pixelSize ) );
+	return 0.8;
 }
 
 void StandardNodeGadget::setHighlighted( bool highlighted )