Skip to content

Commit 920df9d

Browse files
april-roszkowskiEvergreen
authored andcommitted
[Port][6000.3][UUM-133930] Fix Shader Graph icon badges overflowing their container
1 parent 9fe6a77 commit 920df9d

5 files changed

Lines changed: 31 additions & 6 deletions

File tree

Packages/com.unity.shadergraph/Editor/Data/Nodes/Utility/RedirectNodeView.cs

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -141,8 +141,7 @@ public bool FindPort(SlotReference slot, out ShaderPort port)
141141
public void AttachMessage(string errString, ShaderCompilerMessageSeverity severity)
142142
{
143143
ClearMessage();
144-
IconBadge badge;
145-
badge = IconBadge.CreateComment(errString);
144+
IconBadge badge = IconBadgeWrapper.CreateComment(errString);
146145

147146
Add(badge);
148147
badge.AttachTo(outputContainer, SpriteAlignment.RightCenter);

Packages/com.unity.shadergraph/Editor/Drawing/Views/MaterialNodeView.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -195,11 +195,11 @@ public void AttachMessage(string errString, ShaderCompilerMessageSeverity severi
195195
IconBadge badge;
196196
if (severity == ShaderCompilerMessageSeverity.Error)
197197
{
198-
badge = IconBadge.CreateError(errString);
198+
badge = IconBadgeWrapper.CreateError(errString);
199199
}
200200
else
201201
{
202-
badge = IconBadge.CreateComment(errString);
202+
badge = IconBadgeWrapper.CreateComment(errString);
203203
}
204204

205205
Add(badge);

Packages/com.unity.shadergraph/Editor/Drawing/Views/PropertyNodeView.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -326,11 +326,11 @@ public void AttachMessage(string errString, ShaderCompilerMessageSeverity severi
326326
IconBadge badge;
327327
if (severity == ShaderCompilerMessageSeverity.Error)
328328
{
329-
badge = IconBadge.CreateError(errString);
329+
badge = IconBadgeWrapper.CreateError(errString);
330330
}
331331
else
332332
{
333-
badge = IconBadge.CreateComment(errString);
333+
badge = IconBadgeWrapper.CreateComment(errString);
334334
}
335335

336336
Add(badge);
Lines changed: 24 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,24 @@
1+
using GraphViewIconBadge = UnityEditor.Experimental.GraphView.IconBadge;
2+
3+
namespace UnityEditor.ShaderGraph
4+
{
5+
// Wrapper class for internal access to GraphView's IconBadge.
6+
// Allows for creation of badges which don't manually layout their label's dimensions. Works around
7+
// a GraphView bug.
8+
internal static class IconBadgeWrapper
9+
{
10+
public static GraphViewIconBadge CreateError(string message)
11+
{
12+
var result = GraphViewIconBadge.CreateError(message);
13+
result.m_ComputeTextBoundingBox = false;
14+
return result;
15+
}
16+
17+
public static GraphViewIconBadge CreateComment(string message)
18+
{
19+
var result = GraphViewIconBadge.CreateComment(message);
20+
result.m_ComputeTextBoundingBox = false;
21+
return result;
22+
}
23+
}
24+
}

Packages/com.unity.shadergraph/Editor/Util/IconBadgeWrapper.cs.meta

Lines changed: 2 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)