-
Notifications
You must be signed in to change notification settings - Fork 636
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Pm mypackages - cherrypick animation screen #14270
Merged
Merged
Conversation
This file contains 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
- added MyPackages controll - added warning navigation screen when no local packages are installed
- now populates separately and is not linked to SearchResults (not influenced by search changes)
- added the loading animation screen (still WIP) - connected controls and visibility binding - since this particular animation cannot be set in style, created custom control for the loading stripe
reddyashish
reviewed
Aug 16, 2023
src/DynamoCoreWpf/ViewModels/PackageManager/PackageManagerSearchViewModel.cs
Outdated
Show resolved
Hide resolved
reddyashish
reviewed
Aug 16, 2023
src/DynamoCoreWpf/ViewModels/PackageManager/PackageManagerSearchViewModel.cs
Outdated
Show resolved
Hide resolved
reddyashish
reviewed
Aug 16, 2023
reddyashish
reviewed
Aug 16, 2023
src/DynamoCoreWpf/Views/PackageManager/Controls/PackageManagerPackagesControl.xaml.cs
Show resolved
Hide resolved
reddyashish
reviewed
Aug 16, 2023
src/DynamoCoreWpf/Views/PackageManager/Controls/PackageManagerPackagesControl.xaml.cs
Outdated
Show resolved
Hide resolved
LGTM with some comments. |
reddyashish
reviewed
Aug 16, 2023
reddyashish
reviewed
Aug 16, 2023
- changed sorting order
reddyashish
approved these changes
Aug 16, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Cherry-pick from #14198 to break it down into smaller chunks.
Purpose
This is a follow-up PR on the package integration PR. It builds on the continuous design guides as per this Figma link.
Even though a lot of areas have been improved, this is still work in progress. With the main skeleton of the solution in place, the major directions of work are:
Screenshots
01 loading screen
02 discard loading timeout
03 package details extension interaction
04 package manager contents
05 package manager to preferences interaction
Declarations
Check these if you believe they are true
*.resx
filesRelease Notes
Progression work on the Package Manager
Reviewers
@reddyashish
@QilongTang
FYIs
@Amoursol