Python Node Editor Visual Update - part #2#13763
Merged
reddyashish merged 22 commits intoDynamoDS:masterfrom Mar 13, 2023
Merged
Python Node Editor Visual Update - part #2#13763reddyashish merged 22 commits intoDynamoDS:masterfrom
reddyashish merged 22 commits intoDynamoDS:masterfrom
Conversation
- initial commit for python editor visual restyle - for general idea of the desired scope, visit https://www.figma.com/file/1q7EWQGYO7pPDhyLf8nuwW/Python-node-editor-restyling?node-id=0%3A1&t=QGCSTKllX8Q7OnhK-0
…editor-visualupdate
- updated python scrip editor icons
- minor update for one of the icons
- due to readability issues, changed the hyperlink text color
- first implementation working
- added custom Python indentation strategy based around line ending with a column
- WIP code, still playing with the avalon editor visual capabilities - Implemented the 'save on exit' (still missing prompt) - Folding strategy works more or less correctly, except we are not tracking folded states (which we can) - more work needed, but should we?
…nvizupdate-folding
- added buttons for the undo/redo and zoom-in/out functionalities
- updated keywords as per latest Figma color scheme
- added a warning message with controls when user tries to exit the Script interface by pressing the Escape button, but has unsaved changes - made Avalon edit support classes Internal
Contributor
|
@dnenov can you please also put an image or GIF inside of this PR? We often refer back to these when creating presentations etc 😊 |
- now will correctly continue after ":" not being the end of the text line (no tabbing)
- testing screen resolution against 24x24 icons
Collaborator
Author
Of course! Adding them in as we are closing these chapters. |
- updated icons back to 48x48px
- will disable any user interaction while in 'warning' mode - forces the user to `keep editing` -> `save` -> `close`
8 tasks
8 tasks
sm6srw
approved these changes
Mar 1, 2023
reddyashish
reviewed
Mar 13, 2023
reddyashish
reviewed
Mar 13, 2023
reddyashish
reviewed
Mar 13, 2023
Collaborator
|
@dnenov There is one conflict on this PR. PTAL. |
- re[placed with dynamo library brushes where possible
- unsaved changes prompt title and text localized
- removed old or unnecessary comments - kept comments that help to clarify the logic
reddyashish
approved these changes
Mar 13, 2023
Collaborator
reddyashish
left a comment
reddyashish
left a comment
There was a problem hiding this comment.
LGTM once the conflict is resolved
Collaborator
|
@dnenov Thank you for addressing everything. Will wait for the self-service to finish. |
Collaborator
|
Passed, merging this. |
sm6srw
pushed a commit
to sm6srw/Dynamo
that referenced
this pull request
Mar 29, 2023
* Python editor visual restyle - initial commit - initial commit for python editor visual restyle - for general idea of the desired scope, visit https://www.figma.com/file/1q7EWQGYO7pPDhyLf8nuwW/Python-node-editor-restyling?node-id=0%3A1&t=QGCSTKllX8Q7OnhK-0 * Icon Update - updated python scrip editor icons * Hover icon update - minor update for one of the icons * Change hyperlink color - due to readability issues, changed the hyperlink text color * Text Folding - first implementation working * Custom indentation strategy added - added custom Python indentation strategy based around line ending with a column * TabFoldingStrategy changes, saves on Esc exist, WIP - WIP code, still playing with the avalon editor visual capabilities - Implemented the 'save on exit' (still missing prompt) - Folding strategy works more or less correctly, except we are not tracking folded states (which we can) - more work needed, but should we? * Undo/Redo, Zoom-in/out buttons added - added buttons for the undo/redo and zoom-in/out functionalities * Keywords color update - updated keywords as per latest Figma color scheme * Warning bar on unsaved changes exit added - added a warning message with controls when user tries to exit the Script interface by pressing the Escape button, but has unsaved changes - made Avalon edit support classes Internal * Fix tab folding - now will correctly continue after ":" not being the end of the text line (no tabbing) * Small icons test - testing screen resolution against 24x24 icons * Back to 48x48 icons - updated icons back to 48x48px * Disable UI when prompt to save changes - will disable any user interaction while in 'warning' mode - forces the user to `keep editing` -> `save` -> `close` * Color brushes replaced with Dynamo brushes where possible - re[placed with dynamo library brushes where possible * Localized unsaved changes prompt texts - unsaved changes prompt title and text localized * Cleaned up tabfoldingstrategy comments - removed old or unnecessary comments - kept comments that help to clarify the logic * Update ScriptEditorWindow.xaml
sm6srw
pushed a commit
that referenced
this pull request
Apr 5, 2023
* Python editor visual restyle - initial commit - initial commit for python editor visual restyle - for general idea of the desired scope, visit https://www.figma.com/file/1q7EWQGYO7pPDhyLf8nuwW/Python-node-editor-restyling?node-id=0%3A1&t=QGCSTKllX8Q7OnhK-0 * Icon Update - updated python scrip editor icons * Hover icon update - minor update for one of the icons * Change hyperlink color - due to readability issues, changed the hyperlink text color * Text Folding - first implementation working * Custom indentation strategy added - added custom Python indentation strategy based around line ending with a column * TabFoldingStrategy changes, saves on Esc exist, WIP - WIP code, still playing with the avalon editor visual capabilities - Implemented the 'save on exit' (still missing prompt) - Folding strategy works more or less correctly, except we are not tracking folded states (which we can) - more work needed, but should we? * Undo/Redo, Zoom-in/out buttons added - added buttons for the undo/redo and zoom-in/out functionalities * Keywords color update - updated keywords as per latest Figma color scheme * Warning bar on unsaved changes exit added - added a warning message with controls when user tries to exit the Script interface by pressing the Escape button, but has unsaved changes - made Avalon edit support classes Internal * Fix tab folding - now will correctly continue after ":" not being the end of the text line (no tabbing) * Small icons test - testing screen resolution against 24x24 icons * Back to 48x48 icons - updated icons back to 48x48px * Disable UI when prompt to save changes - will disable any user interaction while in 'warning' mode - forces the user to `keep editing` -> `save` -> `close` * Color brushes replaced with Dynamo brushes where possible - re[placed with dynamo library brushes where possible * Localized unsaved changes prompt texts - unsaved changes prompt title and text localized * Cleaned up tabfoldingstrategy comments - removed old or unnecessary comments - kept comments that help to clarify the logic * Update ScriptEditorWindow.xaml
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
5 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
This PR is a follow-up and is based on #13732. It goes further in customizing the Python Script editor in a number of ways.
Exit saved/unsaved script
Folding tests
Declarations
Check these if you believe they are true
*.resxfilesRelease Notes
Reviewers
@sm6srw
@Amoursol
FYIs
@Jingyi-Wen
@hwahlstrom