Skip to content

Feature: Align admin tooltips - #3626

Merged
BornToBeRoot merged 2 commits into
mainfrom
feature/admin-tooltips
Oct 19, 2026
Merged

BornToBeRoot merged 2 commits into
mainfrom
feature/admin-tooltips

Conversation

@BornToBeRoot

Copy link
Copy Markdown
Owner

Changes proposed in this pull request

  • Align admin tooltips (add hint/message)

To-Do

Contributing

By submitting this pull request, I confirm the following:

Copilot AI balanced review requested due to automatic review settings October 19, 2026 14:54
@github-actions github-actions Bot added this to the next-release milestone Oct 19, 2026
@mergify

mergify Bot commented Oct 19, 2026 •

Copy link
Copy Markdown
Contributor

This pull request does not currently match the merge queue conditions, so it cannot be queued from here. The box comes back if it matches again.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

The focused tooltip changes preserve permission enforcement, and no blocking issues were identified.

No open issues 🎉

What changed in this PR

Standardizes permission tooltips across NETworkManager so disabled administrative actions explain why elevation is required.

Changes:

  • Adds a reusable permission-tooltip helper and applies it across five views.
  • Reuses wrapping styles for permission hints and error tooltips.
  • Documents the improvements in the changelog.
File Description
Website/​docs/​changelog/​next-release.md Documents permission hints and error-text wrapping.
Source/​NETworkManager/​Views/​NetworkInterfaceView.xaml Adds configuration permission hints.
Source/​NETworkManager/​Views/​NeighborTableView.xaml Adds entry-management permission hints.
Source/​NETworkManager/​Views/​HostsFileEditorView.xaml Adds editing permission hints.
Source/​NETworkManager/​Views/​FirewallView.xaml Adds rule-management permission hints.
Source/​NETworkManager/​Views/​DiscoveryProtocolView.xaml Adds a capture permission hint.
Source/​NETworkManager/​Resources/​Styles/​ToolTipStyles.xaml Introduces a reusable wrapping style.
Source/​NETworkManager/​Resources/​Styles/​RectangleStyles.xaml Enables wrapping for error-icon tooltips.
Source/​NETworkManager.Utilities.WPF/​PermissionToolTipHelper.cs Implements permission-dependent tooltips.

Review effort: Balanced

@BornToBeRoot
BornToBeRoot merged commit 0bf2a59 into main Oct 19, 2026
3 of 5 checks passed
@BornToBeRoot
BornToBeRoot deleted the feature/admin-tooltips branch October 19, 2026 15:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants