Skip to content

Fix defaults behavior for disabled browsers#233

Merged
mortenn merged 3 commits intomainfrom
fix/issue-228-disabled-defaults
Mar 18, 2026
Merged

Fix defaults behavior for disabled browsers#233
mortenn merged 3 commits intomainfrom
fix/issue-228-disabled-defaults

Conversation

@mortenn
Copy link
Owner

@mortenn mortenn commented Mar 18, 2026

Summary

  • Keep disabled-browser defaults visible in the test view while skipping disabled browsers during actual launch.
  • Polish the defaults grid so disabled selections are easier to spot and in-cell editing stays visually stable.
  • Make local debugging easier by loading development logging settings from the existing launch profiles.

Test plan

  • dotnet clean "src/BrowserPicker.App/BrowserPicker.App.csproj" -p:Version=1.0.0
  • dotnet build "src/BrowserPicker.App/BrowserPicker.App.csproj" -p:Version=1.0.0
  • Manually verified defaults behavior when disabling and re-enabling a browser used by a matching rule.

mortenn added 3 commits March 18, 2026 01:04
Keep matched defaults visible in diagnostics while skipping disabled browsers during launch. Refresh the test defaults view when browser state changes and add targeted logs so browser selection mismatches are easier to diagnose.

Made-with: Cursor
Improve the defaults grid so edited cells stay visually stable, selected rows remain readable, and disabled browser selections are easier to distinguish. Also give the match-type picker enough room to show its value cleanly.

Made-with: Cursor
Run the existing debug profiles in the Development environment so appsettings.Development.json is loaded during local debugging. Also nest the development settings file under appsettings.json so the logging configuration is easier to find.

Made-with: Cursor
@mortenn mortenn merged commit c49d48b into main Mar 18, 2026
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