Skip to content

Profile picker display modes and nullable App.Settings#238

Merged
mortenn merged 2 commits intomainfrom
fix/app-settings-nullability-picker
Mar 21, 2026
Merged

Profile picker display modes and nullable App.Settings#238
mortenn merged 2 commits intomainfrom
fix/app-settings-nullability-picker

Conversation

@mortenn
Copy link
Owner

@mortenn mortenn commented Mar 21, 2026

Summary

  • Profile picker: Rebuild \PickerChoices\ for grouped vs flat profile display and sort modes (including usage-based ordering in flat mode). Browser list UI, behaviors, and converters support the new layout.
  • **Nullable \App.Settings**: \IBrowserPickerConfiguration?\ with guarded startup (background tasks, ViewModel creation), theme subscription in \OnStartup, and \GetContentThemeBrushes\ fallback when settings are absent. \Program\ keeps a local configuration reference for the initial content theme dictionary. \MainWindow\ pattern-matches settings for size persistence; digit shortcuts follow \PickerChoices\ order.

Commits

  1. \ eat: profile picker display modes and supporting UI\
  2. \ ix: nullable App.Settings and null-safe main window\

Verification

  • \dotnet build src/BrowserPicker.App/BrowserPicker.App.csproj -p:Version=1.0.0 -p:Platform=x64\ (0 warnings)

No linked issue — use \Fixes #N\ in a follow-up edit if this should close one.

Made with Cursor

mortenn added 2 commits March 21, 2026 01:26
- Rebuild PickerChoices for grouped vs flat profiles and sort order (incl. usage-based flat list)

- Browser list UI, behaviors, and converters for profile rows and flags

- Configuration/feedback VM updates; settings and profile discovery tweaks

- Shared model and IBrowserPickerConfiguration adjustments; gitignore/DotSettings

Made-with: Cursor
- Model Settings as IBrowserPickerConfiguration?; guard background tasks, ViewModel init, OnStartup theme hookup

- GetContentThemeBrushes uses Settings?.ThemeMode with BrowserPicker.ThemeMode fallback (avoids WPF ThemeMode ambiguity)

- Program keeps a local configuration reference for content theme before Run()

- MainWindow: pattern-match settings everywhere; digit shortcuts follow PickerChoices order

Made-with: Cursor
@mortenn mortenn merged commit 5edcb4c into main Mar 21, 2026
@mortenn mortenn deleted the fix/app-settings-nullability-picker branch March 23, 2026 21:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant