Skip to content

feat: polish schema selector UI - #5

Merged
vitorhugo-dotnet merged 19 commits into
mainfrom
feat/schema-selector-ui-polish
Aug 11, 2026
Merged

feat: polish schema selector UI#5
vitorhugo-dotnet merged 19 commits into
mainfrom
feat/schema-selector-ui-polish

Conversation

@vitorhugo-dotnet

@vitorhugo-dotnet vitorhugo-dotnet commented Aug 10, 2026

Copy link
Copy Markdown
Owner

Implements the approved schema selector UI polish spec and implementation plan.

Scope:

  • ArrowUp/ArrowDown schema navigation with clamped active option
  • Enter confirmation and Escape close behavior
  • ARIA active-descendant plus scroll-into-view for keyboard navigation
  • vertical-only dark styled schema scrolling and long-name truncation
  • compact GitHub footer link through Wails BrowserOpenURL
  • Windows ICO regenerated with transparent corners in 16/32/48/64/128/256 px frames

Verification:

  • TDD RED runs confirmed the new keyboard, scrolling, and footer tests failed before implementation
  • GitHub Actions run 31432218654: Test job passed (Go tests, frontend tests, TypeScript)
  • GitHub Actions run 31432218654: Windows Wails build passed, executable verification passed, artifact upload passed
  • built go-script-sql-runner.exe was downloaded and inspected; all six PNG icon frames from the transparent ICO are embedded byte-for-byte in the executable and each has corner alpha 0

No schema persistence/execution semantics were changed.

@vitorhugo-dotnet
vitorhugo-dotnet force-pushed the feat/schema-selector-ui-polish branch from 8a8c491 to 0563e61 Compare August 10, 2026 21:05
@vitorhugo-dotnet
vitorhugo-dotnet marked this pull request as ready for review August 10, 2026 21:10
@vitorhugo-dotnet
vitorhugo-dotnet merged commit 794cce5 into main Aug 11, 2026
3 checks passed
@vitorhugo-dotnet
vitorhugo-dotnet deleted the feat/schema-selector-ui-polish branch August 11, 2026 10:13
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