Skip to content

Server URL dialog improvements#10475

Merged
tool4ever merged 1 commit intoCard-Forge:masterfrom
MostCromulent:NetworkPlay/server-url-button-dialog
Apr 21, 2026
Merged

Server URL dialog improvements#10475
tool4ever merged 1 commit intoCard-Forge:masterfrom
MostCromulent:NetworkPlay/server-url-button-dialog

Conversation

@MostCromulent
Copy link
Copy Markdown
Contributor

@MostCromulent MostCromulent commented Apr 20, 2026

#10445 added a new server URL dialog on desktop - good addition!

A few improvements here:

-The "Server URL" button in the lobby window now correctly links to the new dialog rather than the old one
-The dialog width now automatically scales according to the width of the URLs and interface names so its not unnecessarily wide (was previously appearing for me with about 400px of blank space between items)
-Dialog saves a click by closing once copy button is pressed, matching behavior of previous dialog.
-Dialog now remembers the last URL you copied and marks that with a star - when opened in future this is automatically copied to your clipboard (as long as the URL is found in the current available list) and noted at the bottom of the dialog. This means e.g. if you only ever use Hamachi then that preference is remembered and you don't need to select it each time.

(@RafaelHGOliveira FYI, since previous commit was your work).

Screenshot 2026-04-21 080533

🤖 Generated with Claude Code

Point the lobby's "Server URL" button at the same per-interface address
dialog added to host startup in PR <!-- -->Card-Forge#10445. Instruction moved inside the
panel so dialog width fits the addresses; columns left-aligned with 30px
gaps. Copy buttons dismiss the dialog.

Remember the last-copied URL in a new NET_LAST_COPIED_URL pref. On open,
auto-copy and star the remembered row (fallback: first row); an italic
notice at the bottom confirms what was copied. Mobile continues to use
NetConnectUtil.copyHostedServerUrl — the new dialog is desktop-only Swing.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@tool4ever tool4ever merged commit 7844f25 into Card-Forge:master Apr 21, 2026
2 checks passed
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.

2 participants