Skip to content

fix: improve UI/UX with cursor styles, SSH file picker, and sidebar loading state#147

Merged
dory-finn merged 1 commit intomainfrom
fix/ui-ux-issues-145
Apr 3, 2026
Merged

fix: improve UI/UX with cursor styles, SSH file picker, and sidebar loading state#147
dory-finn merged 1 commit intomainfrom
fix/ui-ux-issues-145

Conversation

@solarhell
Copy link
Copy Markdown
Collaborator

@solarhell solarhell commented Apr 3, 2026

Summary

Fixes #145

  • Cursor pointer: Added cursor-pointer to interactive elements that were missing it — CatalogHeader toggle button, FieldHelp tooltip button, and SSH authentication radio buttons/labels
  • SSH private key file picker: Added a "Select file" button next to the SSH private key textarea, allowing users to select .pem, .key, .pub, .ppk files from their local system instead of only pasting
  • Sidebar loading state: Added skeleton loading indicators to the explorer sidebar while databases are being fetched, preventing the sidebar from appearing empty during data loading

Test plan

  • Verify cursor changes to pointer when hovering over catalog expand/collapse button, field help icons, and SSH radio buttons
  • Test SSH private key file selection: select a .pem or .key file and verify it populates the textarea
  • Verify explorer sidebar shows skeleton placeholders while databases are loading, then transitions to the database tree
  • Verify all 4 locale files (en, zh, ja, es) render "Select file" correctly

…oading state (#145)

- Add cursor-pointer to interactive elements missing it (CatalogHeader, FieldHelp, SSH radio buttons)
- Add file picker button for SSH private key selection (.pem, .key, .pub, .ppk)
- Add skeleton loading state to explorer sidebar while databases are loading
- Add i18n translations for "Select File" in all locales
@vercel
Copy link
Copy Markdown

vercel bot commented Apr 3, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
dory Ready Ready Preview, Comment Apr 3, 2026 3:55am
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
dory-admin Ignored Ignored Apr 3, 2026 3:55am
dory-beta Ignored Ignored Apr 3, 2026 3:55am

Request Review

@solarhell
Copy link
Copy Markdown
Collaborator Author

Build Verification

  • Vercel deployment: Passed (deployed successfully)
  • Local yarn build: Passed (no errors)
  • TypeScript type check: Passed (tsc --noEmit)
  • Playwright: Failed due to Vercel preview auth restriction ("Invalid origin"), not related to code changes

Preview URL authentication blocks guest login, so UI screenshots need to be verified locally or after merging.

Copy link
Copy Markdown
Contributor

@dory-finn dory-finn left a comment

Choose a reason for hiding this comment

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

LGTM

@dory-finn dory-finn merged commit 374e392 into main Apr 3, 2026
5 of 6 checks passed
@solarhell solarhell deleted the fix/ui-ux-issues-145 branch April 3, 2026 05:29
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.

🐛 UI/UX & Functionality Issues

2 participants