Dnenov restyle visual restyle#13075
Merged
reddyashish merged 23 commits intoDynamoDS:masterfrom Jul 19, 2022
Merged
Conversation
- part of the overall visual restyle in Q3 2022 - style matched to the existing DynamoMessageBox - interim commit
- swapped window icons
- restyle Edit text pop-up window
- restyle to the Dynamo About window
- changed Edit Node to dark theme as per Figma
- small tweaks to UI of Edit, About and TermsOfUse Windows
- visual update to the Usage Reporting Agreement Window - changed the fontsize for all RichTextFiles (this is a limitation)
- restyle to Dynamo crash window
- the Generic Task Dialog is the base class that is used to display the 'Graph Contains Unresolved Nodes and Cannot be Saved dialog.' Restyled to match the new Theme.
-contextually rename Node directly when under the ZoomThreshold
- small visual fix to the About Dialog - added documentation for the new methods incorporated in the NodeView
Contributor
|
@Jingyi-Wen as per the Slack conversation, could you please update the two nodal graphics here? One sad face and the other band-aid face ones! 😁 |
- Graph contains unresolved nodes dialog: - blue button, image change - Dynamo has crashed - image change - About Dynamo - Dynamo icon text needs to be changed to darker color. Same for version text
Collaborator
|
@dnenov |
reddyashish
reviewed
Jul 12, 2022
- removed png files that are no longer needed
- multiple dialog texts have been updated - crash dialog - unresolved nodes - minor alignment to control buttons of dialogs
- updated 'About' dialog following team meeting - green color of (Up-to-date) changed - (Up-to-date) changed to (Up to date) - the dialog is now resizable with grip (slow performance noted, not sure how to optimize or if it is possible) - minimize, maximize buttons added
- implemented a version of the high-density design style - generally less margins all around
- added the ability to close down a dialog window by hitting the Escape key
- fixed the style of the text box displaying the details of a crash, if expanded - a new style created, old style is still in use so remains unchanged
Collaborator
Author
|
The unification of all the dialog icons (minimize, maximize, close, normalize) will be done in a separate PR. |
- fixed the double 'Dynamo' text of the Crash Dialog Title inside the US Recourses
reddyashish
approved these changes
Jul 18, 2022
Collaborator
|
Thanks Deyan for fixing the regressions. Running the job again to make sure everything is passing. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Purpose
Visual restyle for dialog-type UI elements.
Declarations
Check these if you believe they are true
*.resxfilesRelease Notes
Reviewers
@reddyashish
FYIs
@Amoursol