feat: redesign WPF UI with Fluent system#8
Merged
Conversation
Contributor
Author
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
remove "Duck" prefix from all resource keys in FluentDesign.xaml and update all references.
fix: add Csharpier ignore for xaml files fix: remove unused translation keys
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
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.
Description
Implemented a comprehensive Fluent-inspired WPF UI refresh across the app in one unified PR to avoid a mixed old/new experience.
Key changes:
FluentDesign.xamlresource dictionary with design tokens for spacing, radius, page layout, cards, toolbars, list rows, dialogs, empty states, search boxes, combo boxes, toggles, and subtle short fade transitions.OptimizationCategoryViewModelto avoid duplicate concurrent source fetches.Type of Change
fix:- non-breaking change which fixes an issue)feat:- adds a new optimization or feature)feat:- adds UI, services, or core app functionality)refactor:- code restructure that neither fixes a bug nor adds a feature)docs:)i18n:orchore:)Screenshots or Video (if applicable)
Not captured locally because this WPF app requires a Windows desktop/runtime; validation was performed via build on Linux with Windows targeting enabled and CI will run on
windows-latest.Additional Context
Validation performed locally:
dotnet restore optimizerDuck.slnx -p:EnableWindowsTargeting=truedotnet build optimizerDuck.slnx --configuration Release --no-restore -p:EnableWindowsTargeting=truedotnet format optimizerDuck.slnx --verify-no-changes --include optimizerDuck/UI/ViewModels/Optimizer/OptimizationCategoryViewModel.cs --no-restoregit diff --checkdotnet test optimizerDuck.Test/optimizerDuck.Test.csproj --configuration Release --no-build -p:EnableWindowsTargeting=truecannot execute on this Linux VM becauseMicrosoft.WindowsDesktop.Appis Windows-only. CI should run tests onwindows-latestper the workflow.Link to Devin session: https://app.devin.ai/sessions/bf7d4a16910547ed9edd63891a20eeaa
Requested by: @itsfatduck