Skip to content

Commit 70bc339

Browse files
PaulDemeulenaerejulienf-unity
authored andcommitted
Fix ordering for "+" button in blackboard (#197)
1 parent 769b8a0 commit 70bc339

File tree

2 files changed

+8
-2
lines changed

2 files changed

+8
-2
lines changed

com.unity.visualeffectgraph/CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
1111
- Prevent infinite compilation loop [Case 1298466](https://issuetracker.unity3d.com/product/unity/issues/guid/1298466/)
1212
- Remove some useless compilation triggers (modifying not connected or disabled nodes for instance)
1313
- Modified Sign operator node output for float when input is 0.0f [Case 1299922](https://fogbugz.unity3d.com/f/cases/1299922/)
14+
- Use alphabetical order in type list in blackboard "+" button [Case 1304109](https://issuetracker.unity3d.com/product/unity/issues/guid/1304109/)
1415

1516
## [10.3.1] - 2020-01-26
1617

com.unity.visualeffectgraph/Editor/GraphView/Blackboard/VFXBlackboard.cs

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -402,6 +402,11 @@ void OnAddOutputParameter(object parameter)
402402
newParam.isOutput = true;
403403
}
404404

405+
private static IEnumerable<VFXModelDescriptor> GetSortedParameters()
406+
{
407+
return VFXLibrary.GetParameters().OrderBy(o => o.name);
408+
}
409+
405410
void OnAddItem(Blackboard bb)
406411
{
407412
GenericMenu menu = new GenericMenu();
@@ -412,7 +417,7 @@ void OnAddItem(Blackboard bb)
412417
menu.AddSeparator(string.Empty);
413418
}
414419

415-
foreach (var parameter in VFXLibrary.GetParameters())
420+
foreach (var parameter in GetSortedParameters())
416421
{
417422
VFXParameter model = parameter.model as VFXParameter;
418423

@@ -518,7 +523,7 @@ void OnAddOutputParameterMenu()
518523
{
519524
GenericMenu menu = new GenericMenu();
520525

521-
foreach (var parameter in VFXLibrary.GetParameters())
526+
foreach (var parameter in GetSortedParameters())
522527
{
523528
VFXParameter model = parameter.model as VFXParameter;
524529

0 commit comments

Comments
 (0)