-
Notifications
You must be signed in to change notification settings - Fork 43
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
Workspace integration #898
Workspace integration #898
Conversation
|
Opened AvaloniaUI/Avalonia#14437 |
e264312
to
9ab87ac
Compare
This PR conflicts with |
- removed "Newsfeed" - removed "Browse Games" - code cleanup
fd4cb1d
to
256b886
Compare
This PR doesn't conflict with |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See comments above
src/NexusMods.App.UI/WorkspaceSystem/AddPanelButton/AddPanelDropDownViewModel.cs
Outdated
Show resolved
Hide resolved
src/NexusMods.App.UI/WorkspaceSystem/AddPanelButton/AddPanelDropDownViewModel.cs
Outdated
Show resolved
Hide resolved
Codecov ReportAttention:
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## main #898 +/- ##
=========================================
+ Coverage 0 56.93% +56.93%
=========================================
Files 0 614 +614
Lines 0 19631 +19631
Branches 0 1503 +1503
=========================================
+ Hits 0 11177 +11177
- Misses 0 8118 +8118
- Partials 0 336 +336
Flags with carried forward coverage won't be shown. Click here to find out more.
|
src/NexusMods.App.UI/WorkspaceSystem/Page/PageFactoryController.cs
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I still think we might need an overview doc to outline the general structure, but otherwise the PR is Ok.
I'm going through the other one as well
Part of #706.