[client] Add "Deselect All" Menu Item to Exit Node Menu

[hakansa]

Describe your changes

• Add "Deselect All" Menu Item to Exit Node Menu
• Only rebuild the Exit Nodes menu if the state is changed.

Issue ticket number and link

Stack

Checklist

• Is it a bug fix
• Is a typo/documentation fix
• Is a feature enhancement
• It is a refactor
• Created tests that fail without the change (if possible)
• Extended the README / documentation, if necessary

By submitting this pull request, you confirm that you have read and agree to the terms of the Contributor License Agreement.

[Copilot]

Pull Request Overview

Adds a “Deselect All” item to the system tray Exit Node menu so users can clear all selected exit nodes in one action.

• Introduces an exitNodeState struct and a cached exitNodeStates slice to skip redundant menu rebuilds.
• Updates recreateExitNodeMenu to append a “Deselect All” sub‐menu when any node is selected and wires up its click handler.
• Implements handleExitNodeMenuDeselectAll to call the existing deselect API and refresh the menu.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File	Description
client/ui/network.go	Added exitNodeState, updated menu rebuild logic, and new handler.
client/ui/client_ui.go	Introduced fields for caching exit node states and the deselect item.

Comments suppressed due to low confidence (1)

client/ui/network.go:480

• [nitpick] This new handler controls exit-node deselection and menu refresh logic but lacks any dedicated tests. Adding unit tests for both success and failure paths would help prevent regressions.

func (s *serviceClient) handleExitNodeMenuDeselectAll() ([]*proto.Network, error) {

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
1 Accepted issue

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarQube Cloud