) * fix(ux): pre-beta cleanup pass — vault prompt, eye icon, monaco worker, radio Bundled UX cleanup ahead of the first beta build for v1.0.19 dogfooding. Added (#110): reusable PasswordInput with an Eye/EyeOff toggle. Applied to the connection dialog's DB password, SSH password, SSH key passphrase, and the vault credential password in Settings. AI API key field deliberately skipped — has a left-icon layout that doesn't compose cleanly with the right-side toggle. Per-instance visibility state, hidden by default, toggle in normal tab order. Fixed (#109): the vault credential picker was prompting on manual connections too. The branch in DatabaseExplorer.tsx only checked "does any vault credential exist anywhere", not the connection's own isVault flag. Tightened to require conn.isVault === true && !conn.vaultCredentialId — manual + legacy-untagged connections now go straight to direct connect. Fixed (#102): wired self.MonacoEnvironment.getWorker in monacoSetup.ts using Vite's ?worker bundling. Diff / search / link computation now runs off the main thread instead of logging "Could not create web worker(s)" on every editor mount in release builds. CSP already permitted this via worker-src 'self' blob: — no policy change. Fixed (ux): radio dot in Settings → Updates was offset down-right. The inner dot's m-[3px] overflowed the outer's 10px inner space (after the 2px border). Replaced with flex centering — works at any size. #76 ("show 'Connecting…' on clicked node, not 'Select a database' in footer") was already fixed in v1.0.16 (per its CHANGELOG entry matching the current code). The issue just never got closed — closing it separately, no code change here. All five touchpoints sized to be testable on the first v1.0.19 beta build: each one has a visible, reproducible interaction the user can verify in a single session. * a11y(password-input): add focus-visible ring to eye toggle CodeRabbit nitpick on #111. focus:outline-none drops the native focus indicator and the existing color-only focus state (text-primary) isn't strong enough for keyboard users. Adds focus-visible:ring-2 in the brand accent color so tabbed focus is clearly visible. focus-visible (not focus) so mouse clicks don't flash a ring. --------- Co-authored-by: Nicolaas Grobler <nicolaasgrob@gmail.com>

bug: vault-vs-manual prompt fires on manual connections when any vault credential exists #109

Description

Metadata

Metadata

Assignees

Labels

Projects

Milestone

Relationships

Development

Issue actions