Skip to content
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

Reorder grid states #1037

Merged
merged 4 commits into from
Mar 7, 2024
Merged

Conversation

erri120
Copy link
Member

@erri120 erri120 commented Mar 6, 2024

Part of #906.

Fixes the order in which the possible grid states are generated. Now, we prioritize and put the primary action first, rather than last, depending on the orientation.

@erri120 erri120 added the Epic: Workspace System This is related to the Panels/Workspace system label Mar 6, 2024
@erri120 erri120 requested a review from a team March 6, 2024 15:22
@erri120 erri120 self-assigned this Mar 6, 2024
@erri120
Copy link
Member Author

erri120 commented Mar 7, 2024

Horizontal

2024-03-07_11-05-50
2024-03-07_11-05-56
2024-03-07_11-06-03

@erri120
Copy link
Member Author

erri120 commented Mar 7, 2024

Vertical

2024-03-07_11-06-56
2024-03-07_11-07-01
2024-03-07_11-07-05

@Al12rs
Copy link
Contributor

Al12rs commented Mar 7, 2024

That looks good to me, Great work!

@codecov-commenter
Copy link

codecov-commenter commented Mar 7, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 56.55%. Comparing base (75c3c2e) to head (81be06e).

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1037      +/-   ##
==========================================
- Coverage   56.55%   56.55%   -0.01%     
==========================================
  Files         648      648              
  Lines       21354    21360       +6     
  Branches     1664     1664              
==========================================
+ Hits        12077    12080       +3     
- Misses       8879     8882       +3     
  Partials      398      398              
Flag Coverage Δ
Linux 55.88% <100.00%> (+0.02%) ⬆️
Windows 55.77% <100.00%> (+0.01%) ⬆️
clean_environment_tests 56.54% <100.00%> (-0.01%) ⬇️
macOS 55.23% <100.00%> (-0.04%) ⬇️
network_tests ?

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
src/NexusMods.App.UI/WorkspaceSystem/GridUtils.cs 91.09% <100.00%> (+0.12%) ⬆️

... and 1 file with indirect coverage changes

@erri120 erri120 merged commit ccf4f22 into Nexus-Mods:main Mar 7, 2024
6 checks passed
@erri120 erri120 deleted the fix/906-state-ordering branch March 7, 2024 10:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Epic: Workspace System This is related to the Panels/Workspace system
3 participants