Skip to content

Improve Add Server dialog UX#827

Merged
erikdarlingdata merged 2 commits intodevfrom
improve-add-server-dialog-ux
Apr 10, 2026
Merged

Improve Add Server dialog UX#827
erikdarlingdata merged 2 commits intodevfrom
improve-add-server-dialog-ux

Conversation

@erikdarlingdata
Copy link
Copy Markdown
Owner

Summary

  • Replace the connection-success MessageBox popup with inline status in the DatabaseStatusPanel — connection result, database detection, and install guidance now flow naturally in one place
  • Add descriptive text explaining what "Install Now" does (creates database, collection jobs, stored procedures) so first-time users aren't left guessing
  • Size the dialog to the screen work area height and top-align it so post-test content is never hidden below the fold

Test plan

  • Open Add Server dialog — should fill screen height, top-aligned
  • Test connection to a server without PerformanceMonitor installed — no popup, inline "Connected to..." header with version + install guidance
  • Test connection to a server with current version — inline "up to date" message
  • Test connection to a server needing upgrade — inline upgrade prompt
  • Test connection failure — error MessageBox still works as before

🤖 Generated with Claude Code

erikdarlingdata and others added 2 commits April 10, 2026 10:16
…s to Logical CPUs

The timer tick handler set _isRefreshing = true before calling RefreshAllDataAsync,
which also checked _isRefreshing and returned immediately — every auto-refresh was a no-op.
Removed the redundant guard from the tick handler so RefreshAllDataAsync owns the flag.

Renamed "CPUs" to "Logical CPUs" in the FinOps server inventory tab (both Dashboard and Lite)
to clarify the relationship with the Cores/Socket column, which shows physical cores.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…indow

Replace the connection-success MessageBox popup with inline status in the
DatabaseStatusPanel so the entire flow (connection result, database detection,
install prompt) reads naturally in one place. Add guidance text explaining
what Install Now does for first-time users.

Size the dialog to the screen work area height and top-align it so the
post-test content is never hidden below the fold.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@erikdarlingdata erikdarlingdata merged commit 6fa4d05 into dev Apr 10, 2026
@erikdarlingdata erikdarlingdata deleted the improve-add-server-dialog-ux branch April 10, 2026 19:06
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