Skip to content

Comments

Add navigation from tray directly to the join server page#8451

Merged
myleshorton merged 1 commit intomainfrom
myles/systray-join-server
Feb 6, 2026
Merged

Add navigation from tray directly to the join server page#8451
myleshorton merged 1 commit intomainfrom
myles/systray-join-server

Conversation

@myleshorton
Copy link
Contributor

This pull request enhances the system tray functionality by adding a new menu item for joining a private server and improving the tray icon's user experience. The most important changes are:

System Tray Menu Enhancements:

  • Added a new join_server menu item to the system tray, which opens the main window and navigates the user to the join private server page when clicked.

User Experience Improvements:

  • Set the tray icon tooltip to display the application's localized name, improving accessibility and usability.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a new system tray entry point to open the app directly on the “Join Private Server” screen, and improves tray usability by setting a localized tooltip.

Changes:

  • Added a join_server tray menu item that opens the main window and navigates to JoinPrivateServer.
  • Set the tray icon tooltip to the localized application name ('app_name'.i18n).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +68 to +75
MenuItem(
key: 'join_server',
label: 'join_server'.i18n,
onClick: (_) {
// Open Lantern and navigate to the join server page
ref.read(windowProvider.notifier).open(focus: true);
appRouter.push(JoinPrivateServer());
},
Copy link

Copilot AI Feb 6, 2026

Choose a reason for hiding this comment

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

Tray menu label uses the i18n key join_server ("Join Server"), but other entry points to this flow use "Join Private Server" / "Join a Private Server" (join_private_server / join_a_private_server). Consider switching the tray label to the same i18n key for consistent wording (the route being opened is JoinPrivateServer).

Copilot uses AI. Check for mistakes.
@myleshorton myleshorton merged commit 585501d into main Feb 6, 2026
15 checks passed
@myleshorton myleshorton deleted the myles/systray-join-server branch February 6, 2026 21:32
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