Skip to content

Settings: split URL probe controls into Security tab#254

Merged
mortenn merged 3 commits intomainfrom
issue/251-granular-network-settings
Mar 26, 2026
Merged

Settings: split URL probe controls into Security tab#254
mortenn merged 3 commits intomainfrom
issue/251-granular-network-settings

Conversation

@mortenn
Copy link
Owner

@mortenn mortenn commented Mar 26, 2026

Summary

  • split the legacy network toggle into granular redirect and favicon probe settings while keeping backward compatibility with existing settings
  • add a dedicated Security tab with probe presets, keep lookup timeout in Behaviour, and hide the URL bar favicon slot when probing is skipped
  • fix designer/runtime favicon rendering so the URL bar icon preview works consistently

Test plan

  • Build the solution with dotnet build BrowserPicker.sln -p:Version=1.0.0
  • Verify the MainWindow designer shows the URL bar favicon preview
  • Manually verify security profile presets and migrated settings behavior in the app

Fixes #251

Add granular redirect and favicon probe settings with a dedicated Security tab, preserve compatibility with legacy settings, and reuse cached favicons so the picker updates cleanly without a restart.

Made-with: Cursor
@mortenn mortenn force-pushed the issue/251-granular-network-settings branch from 0287861 to 7bdfd36 Compare March 26, 2026 20:59
mortenn added 2 commits March 26, 2026 22:10
Use enum-backed profile properties for the Security presets so the UI labels can change independently from the underlying preset logic.

Made-with: Cursor
Move the security profile UI onto shared value objects so the predefined presets stay in one place and the menu is driven from the shared profile list.

Made-with: Cursor
@mortenn mortenn merged commit 0dee0cf into main Mar 26, 2026
@mortenn mortenn deleted the issue/251-granular-network-settings branch March 26, 2026 21:50
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.

Settings: granular network permissions (split DisableNetworkAccess)

1 participant