Commit 85a4339
GraphNodeManager Update (#13069)
* Nested Empty List Not Showing
- added a recursion to the IsNodeEmptyList method to factor in nested empty lists
* Filter Label Consistent Capitalisation
- fixed inconsistency in capitalization of Filter titles
* Export Tooltip
- 'Export to Excel' changed to 'Export'
* Tooltip Export and Copy Flip
- used a different style of Dynamo Tooltip supporting a positioning for the arrow to adjust for these 2 edge cases
* DismissedAlerts Not Updated in Automatic Mode
- added a dedicated property inside the NodeModel to raise PropertyChanged when updating the DismissedAlerts
- as the external view extension has no access to the NodeViewModel, the code previously acquired the DismissedAlerts count from the .Count value of the DismissedAlert ObservableCollection. However, this wasn't triggering an update and was not being detected inside the external code of the view extension
* Icons to Filters
- added Icons to the Filter elements in the UI
* Aligned Filters to Figma
- reshuffled order of filters
- changed 'Information' to 'Info' title
* Export Default Name
- added a default file name based on the name of the current graph
- fixed a bug when using the CSV file extension dropdown
* Export Unfiltered Nodes Only
- only export nodes that haven't been filtered in the UI
* Renamed Node Blue Dot
- added an identifier to the renamed nodes
- IMPORTANT - exposed a method from NodeModelExtensions to allow to check if a node has been renamed!
* Comments: nameof
- using nameof(<PropertyName>) instead of string value
* Comments: GetOriginalName Internal, used InternalsVisibleTo instead
- reverted a change making GetOriginalName public
- included 'GraphNodeManagerViewExtension' to see internals from DynamoCore using InternalsVisibleTo
- comment fix
* Summary Comments Added to FilterViewModel
- added comments to public properties
* Fixed Summary Comments After Change
- added the additional parameter that was recently introduced to the comment
* Comments: IEnumerable to Array Simplified
- changed the way we get an array from the ItemsSource
* Comments: Braces
- minor change
* SearchTextBox Update
- added a localization resource for 'Search' watermark text
- 'Search' watermark will now disappears on mouse entering the textbox
* DataGrid Visual Changes
- removed cell margin to create 'continuous' vertical lines
- aligned icons and headings
* Export Dual Tooltip
- added an alternative tooltip that shows when nodes are being filtered
* Select Node Behavior Change
- now would always select (zoom in around) a Node, so if the user navigates away in the Canvas, they can zoom back around the Node through GNM
* Multiple Visual Updates
- set negative left/right margin to create 'borderless' look
- updated hidden icon (resolution)
- multiple visual tweaks to the datagrid lement
* Warning/Error Text Contrast
- updated font families to default to ArtifaktElementBold for main text and ArtifaktElementRegular for detail information text
- updated detailed information text to the default Text color
* Hover State Colors, Consistent Selection Color
- changed alternating row hover colors to be different to the selected row color
- replaced hard-coded color values with resource colors
* Dismissed Warning Italic Font
- now styles dismissed warnings message in Italic (missing Artefakt Italic resource, placehoder Ariel at the moment)
* Any Empty Lists and Null
- changed the methods to detect if a node contains Empty List or Null to any instead of all (will now show the respective symbol if any occurrences are found in the node outputs)
* GNM EmptyList and Null Test
- added unit test checking if the number of Nodes containing Null or Empty List matches what is shown on the UI
- added a new dynamo graph to test against
* Remove Dismissed Icon
- removed icon when a warning is dismissed
* Naming Convention
- removed underscore to field name for dismissedAlertsCount in NodeModel
* Images as Resource
- Filter Items now use images as resources converting them to ImageSource from bitmap
* Removed/Replaced Converters
- replaced converters with Dynamo converters
- removed old and unused converters
* Color Resources Replaced
- replaced most of the color resources with the identical counterparts already present in the main dynamo resource dictionaries
* Converter Namespace Rename
- renamed the namespace containing all extension specific converters to a more generic name
* Merge Fix
- fixed small issue after merge on upstream master
* Reverted Changes, Fix Tests Fail
- reverted changes made to the DynamoModern dictionary - the changes were breaking 2 tests
Co-authored-by: reddyashish <43763136+reddyashish@users.noreply.github.com>1 parent 753d4ec commit 85a4339
File tree
22 files changed
+1371
-255
lines changed- src
- DynamoCoreWpf
- UI/Themes/Modern
- ViewModels/Core
- DynamoCore
- Graph/Nodes
- Properties
- GraphNodeManagerViewExtension
- Controls
- Converters
- Images
- Properties
- Utilities
- ViewModels
- test
- DynamoCoreWpfTests/ViewExtensions
- pkgs/Dynamo Samples/extra
22 files changed
+1371
-255
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1081 | 1081 | | |
1082 | 1082 | | |
1083 | 1083 | | |
1084 | | - | |
| 1084 | + | |
1085 | 1085 | | |
1086 | 1086 | | |
1087 | 1087 | | |
1088 | 1088 | | |
| 1089 | + | |
| 1090 | + | |
| 1091 | + | |
| 1092 | + | |
| 1093 | + | |
| 1094 | + | |
| 1095 | + | |
| 1096 | + | |
| 1097 | + | |
| 1098 | + | |
| 1099 | + | |
| 1100 | + | |
| 1101 | + | |
| 1102 | + | |
| 1103 | + | |
| 1104 | + | |
| 1105 | + | |
| 1106 | + | |
| 1107 | + | |
| 1108 | + | |
| 1109 | + | |
| 1110 | + | |
| 1111 | + | |
1089 | 1112 | | |
1090 | 1113 | | |
1091 | 1114 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
48 | 48 | | |
49 | 49 | | |
50 | 50 | | |
| 51 | + | |
51 | 52 | | |
52 | 53 | | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
5166 | 5166 | | |
5167 | 5167 | | |
5168 | 5168 | | |
5169 | | - | |
| 5169 | + | |
5170 | 5170 | | |
5171 | 5171 | | |
5172 | 5172 | | |
| |||
5253 | 5253 | | |
5254 | 5254 | | |
5255 | 5255 | | |
5256 | | - | |
| 5256 | + | |
5257 | 5257 | | |
5258 | 5258 | | |
5259 | 5259 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
950 | 950 | | |
951 | 951 | | |
952 | 952 | | |
| 953 | + | |
| 954 | + | |
953 | 955 | | |
954 | | - | |
| 956 | + | |
| 957 | + | |
| 958 | + | |
| 959 | + | |
| 960 | + | |
| 961 | + | |
| 962 | + | |
| 963 | + | |
| 964 | + | |
| 965 | + | |
| 966 | + | |
| 967 | + | |
955 | 968 | | |
956 | 969 | | |
957 | 970 | | |
| |||
Lines changed: 40 additions & 36 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | | - | |
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
2 | 8 | | |
3 | 9 | | |
4 | | - | |
5 | | - | |
6 | | - | |
7 | | - | |
8 | | - | |
9 | | - | |
10 | | - | |
11 | | - | |
| 10 | + | |
| 11 | + | |
12 | 12 | | |
13 | 13 | | |
14 | | - | |
15 | | - | |
16 | | - | |
17 | | - | |
18 | | - | |
19 | | - | |
20 | | - | |
21 | | - | |
| 14 | + | |
| 15 | + | |
22 | 16 | | |
23 | 17 | | |
24 | 18 | | |
| |||
27 | 21 | | |
28 | 22 | | |
29 | 23 | | |
30 | | - | |
| 24 | + | |
31 | 25 | | |
32 | 26 | | |
33 | | - | |
| 27 | + | |
34 | 28 | | |
| 29 | + | |
35 | 30 | | |
36 | 31 | | |
37 | 32 | | |
38 | 33 | | |
39 | | - | |
40 | | - | |
41 | | - | |
| 34 | + | |
| 35 | + | |
| 36 | + | |
| 37 | + | |
| 38 | + | |
42 | 39 | | |
43 | | - | |
44 | | - | |
45 | | - | |
46 | | - | |
| 40 | + | |
| 41 | + | |
| 42 | + | |
| 43 | + | |
47 | 44 | | |
48 | 45 | | |
49 | 46 | | |
50 | 47 | | |
51 | | - | |
| 48 | + | |
52 | 49 | | |
53 | 50 | | |
54 | 51 | | |
55 | | - | |
| 52 | + | |
| 53 | + | |
| 54 | + | |
56 | 55 | | |
57 | | - | |
58 | | - | |
| 56 | + | |
| 57 | + | |
59 | 58 | | |
60 | 59 | | |
61 | | - | |
| 60 | + | |
| 61 | + | |
| 62 | + | |
| 63 | + | |
62 | 64 | | |
63 | 65 | | |
64 | 66 | | |
65 | 67 | | |
66 | 68 | | |
67 | 69 | | |
68 | 70 | | |
69 | | - | |
| 71 | + | |
70 | 72 | | |
| 73 | + | |
| 74 | + | |
71 | 75 | | |
72 | 76 | | |
73 | 77 | | |
74 | 78 | | |
75 | | - | |
| 79 | + | |
76 | 80 | | |
77 | 81 | | |
78 | 82 | | |
79 | 83 | | |
80 | 84 | | |
81 | 85 | | |
82 | | - | |
| 86 | + | |
83 | 87 | | |
84 | 88 | | |
85 | 89 | | |
| |||
111 | 115 | | |
112 | 116 | | |
113 | 117 | | |
114 | | - | |
| 118 | + | |
115 | 119 | | |
116 | 120 | | |
117 | 121 | | |
| |||
126 | 130 | | |
127 | 131 | | |
128 | 132 | | |
129 | | - | |
| 133 | + | |
130 | 134 | | |
131 | 135 | | |
132 | 136 | | |
| |||
Lines changed: 14 additions & 4 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
12 | 12 | | |
13 | 13 | | |
14 | 14 | | |
| 15 | + | |
15 | 16 | | |
16 | | - | |
| 17 | + | |
17 | 18 | | |
18 | 19 | | |
19 | 20 | | |
20 | | - | |
| 21 | + | |
21 | 22 | | |
22 | 23 | | |
23 | 24 | | |
| |||
36 | 37 | | |
37 | 38 | | |
38 | 39 | | |
39 | | - | |
| 40 | + | |
| 41 | + | |
| 42 | + | |
| 43 | + | |
| 44 | + | |
| 45 | + | |
| 46 | + | |
| 47 | + | |
| 48 | + | |
| 49 | + | |
40 | 50 | | |
41 | 51 | | |
42 | 52 | | |
| |||
68 | 78 | | |
69 | 79 | | |
70 | 80 | | |
71 | | - | |
| 81 | + | |
72 | 82 | | |
73 | 83 | | |
74 | 84 | | |
| |||
Lines changed: 11 additions & 11 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
5 | 5 | | |
6 | 6 | | |
7 | 7 | | |
| 8 | + | |
8 | 9 | | |
9 | 10 | | |
10 | 11 | | |
11 | 12 | | |
12 | 13 | | |
| 14 | + | |
| 15 | + | |
13 | 16 | | |
14 | 17 | | |
15 | 18 | | |
| |||
18 | 21 | | |
19 | 22 | | |
20 | 23 | | |
21 | | - | |
| 24 | + | |
22 | 25 | | |
23 | 26 | | |
24 | 27 | | |
25 | 28 | | |
26 | 29 | | |
27 | | - | |
| 30 | + | |
28 | 31 | | |
29 | 32 | | |
30 | 33 | | |
31 | 34 | | |
32 | 35 | | |
33 | 36 | | |
34 | 37 | | |
35 | | - | |
| 38 | + | |
36 | 39 | | |
37 | 40 | | |
38 | | - | |
| 41 | + | |
39 | 42 | | |
40 | 43 | | |
41 | 44 | | |
| |||
94 | 97 | | |
95 | 98 | | |
96 | 99 | | |
97 | | - | |
98 | | - | |
99 | | - | |
100 | | - | |
101 | | - | |
102 | | - | |
103 | | - | |
| 100 | + | |
| 101 | + | |
| 102 | + | |
104 | 103 | | |
105 | 104 | | |
106 | 105 | | |
| |||
112 | 111 | | |
113 | 112 | | |
114 | 113 | | |
| 114 | + | |
115 | 115 | | |
116 | 116 | | |
117 | 117 | | |
| |||
Lines changed: 15 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
37 | 37 | | |
38 | 38 | | |
39 | 39 | | |
| 40 | + | |
| 41 | + | |
| 42 | + | |
| 43 | + | |
| 44 | + | |
| 45 | + | |
| 46 | + | |
| 47 | + | |
| 48 | + | |
| 49 | + | |
| 50 | + | |
| 51 | + | |
| 52 | + | |
| 53 | + | |
| 54 | + | |
40 | 55 | | |
41 | 56 | | |
42 | 57 | | |
| |||
0 commit comments