Round pixelsPerPoint up rather than down when calculating icon scale

tspiller · tspiller · commit 00249568fe7f · 2024-11-20T13:41:36.000Z
diff --git a/Packages/com.unity.inputsystem/InputSystem/Editor/Internal/GUIHelpers.cs b/Packages/com.unity.inputsystem/InputSystem/Editor/Internal/GUIHelpers.cs
@@ -43,7 +43,7 @@ public static void DrawLineSeparator(string label = null)
         public static Texture2D LoadIcon(string name)
         {
             var skinPrefix = EditorGUIUtility.isProSkin ? "d_" : "";
-            var scale = Mathf.Clamp((int)EditorGUIUtility.pixelsPerPoint, 0, 4);
+            var scale = Mathf.Clamp(Mathf.CeilToInt(EditorGUIUtility.pixelsPerPoint), 0, 4);
             var scalePostFix = scale > 1 ? $"@{scale}x" : "";
             if (name.IndexOfAny(Path.GetInvalidFileNameChars()) > -1)
                 name = string.Join("_", name.Split(Path.GetInvalidFileNameChars()));

Original file line number	Diff line number	Diff line change
`@@ -43,7 +43,7 @@ public static void DrawLineSeparator(string label = null)`
`43`	`43`	`public static Texture2D LoadIcon(string name)`
`44`	`44`	`{`
`45`	`45`	`var skinPrefix = EditorGUIUtility.isProSkin ? "d_" : "";`
`46`		`- var scale = Mathf.Clamp((int)EditorGUIUtility.pixelsPerPoint, 0, 4);`
	`46`	`+ var scale = Mathf.Clamp(Mathf.CeilToInt(EditorGUIUtility.pixelsPerPoint), 0, 4);`
`47`	`47`	`var scalePostFix = scale > 1 ? $"@{scale}x" : "";`
`48`	`48`	`if (name.IndexOfAny(Path.GetInvalidFileNameChars()) > -1)`
`49`	`49`	`name = string.Join("_", name.Split(Path.GetInvalidFileNameChars()));`