-
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
FOMOD UI #600
Merged
Merged
FOMOD UI #600
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
fdb5273
to
f11cf7c
Compare
This PR conflicts with |
…-styling # Conflicts: # src/Games/NexusMods.Games.FOMOD.UI/Step/GuidedInstallerStepView.axaml
Codecov Report
@@ Coverage Diff @@
## main #600 +/- ##
==========================================
- Coverage 68.05% 64.76% -3.29%
==========================================
Files 597 620 +23
Lines 16420 17527 +1107
Branches 1091 1134 +43
==========================================
+ Hits 11175 11352 +177
- Misses 4969 5893 +924
- Partials 276 282 +6
Flags with carried forward coverage won't be shown. Click here to find out more. |
halgari
approved these changes
Sep 18, 2023
Al12rs
approved these changes
Sep 18, 2023
7 tasks
Merged
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.
Part 3 of #316. Follow-up on #532 and #541.
TODO:
CheckBox
orRadioButton
depending on the group typeView should scroll to first invalid groupMake the window modalPanels are not going to be modal eitherSelect option on user interacting with checkbox/radio buttonAssetUrl
into a discriminated union for different types of assets #614):IGuidedInstaller
to get the step count and current step number fromUiDelegate
.TaskCompletionSource
toUserChoice.GoToNextStep
with all selected options.TaskCompletionSource
toUserChoice.GoToPreviousStep
InstructionsToModFiles
inFomodXMLInstaller
is very slow: 170 secs for 790 instructions while debugging, I reworked the methods, so it now only takes 74ms.the disabled button doesn't have a border for some reasonfixed)No highlighted option caseWe always highlight first optionOption has no details caseScrapped dedicated style for this for simplicity.Selection highlight styleSacrificed figma style to have something workingDifferences from figma design:
Agreed changes:
This was easier to implement and allowed updating the view as soon as user changed options.
This eventuality was not covered in the design, so this bypasses that.
Compromises and additions:
This shouldn't matter too much since it will be ported over to Panels once those are ready.
Didn't manage to find a good way to style only these ones and investigating more might have taken too much time.
This was cut mostly due to the hassle required to have a special styling case for that.
:pointerover
We attempted the lighter color, but we only managed to change the background, not the foreground elements to use a darker shade, making the Text and icons illegible when highlighted.
Having proper white based highlight could be possible but it needs time and code to support it, so this version was implemented for now to avoid wasting too much time on it.
Additional changes: