Vault browser replaces git review in the session side panel (stacked on #67) - #68
Merged
Merged
Conversation
Multi-agent audit (48 findings, adversarially triaged) against the amicode design system and ui-ux-pro-max critical checklists. Fixes: Tokens: update pill off the grey-on-light icon-accent color-mix fills onto --accent-fill-soft/--accent-edge; connections project rows off banned v1 vars (--border-weak-base/--surface-base); ask-card selected edge onto --accent-edge; new --accent-fill-hover token replaces two hand-rolled accent mixes; titlebar tabs onto the radius scale; channel badge raw hex dropped; run-gallery scrim into the 40-60% band. Type scale: off-scale weights (450/530/550/640/750/800) and sizes (9/11.5/12.5/15px) across solver-toggle, defaults-capsule, amicode.css, home-cards onto the sanctioned scales; off-grid paddings/gaps onto the 4px grid. Interaction: bounds-moving press/hover transforms (translateY) replaced with elevation/fill feedback; hover+focus-visible states added to yellow CTA, action cards, rail inspect/retry, starter chips, suggestion rows; busy save button gets wait cursor + reduced emphasis; sub-150ms transitions raised; MeetAmico shine + lift gated on reduced motion. A11y: onboarding-wizard inputs no longer strip the focus outline; action cards and MeetAmico are keyboard-operable (role, tabIndex, Enter/Space); profile inputs get real labels; edit-profile toggle gets aria-label/expanded; vault-panel text-glyph arrows replaced with the icon family; context-tree legend unhidden from AT; vault mount chips join the accent selected pattern. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…av, disclosures, one input recipe - Institution autocomplete is a real combobox: role/aria-expanded/ aria-activedescendant, arrow-key navigation, Enter picks, Escape closes; options are listbox options with an active highlight. - Context tree is keyboard-operable: the canvas takes focus, arrows walk the tree's flat order, Home/End jump, Enter/Space opens the focused node; the engine gained focus(id) and an aria-live region announces each node. role=img corrected to role=application. - 'Amico remembers' details unfold inline as disclosures instead of hiding in a hover-only title tooltip. - ONE text-input recipe: .amc-input (+ --compact) in amicode.css replaces the three divergent inline recipes across the onboarding wizard, profile editor, and connections forms — focus ring included so no caller ever strips an outline again. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
The side panel's Review tab, Git-changes list, and Unified/Split controls are removed (researchers don't review diffs there — Kate 2026-07-27); the Vault browser takes that spot as a first-class tab. One VaultBrowser body now serves two hosts: the session side panel (vault tab, synced to the global vaultPanel store so the titlebar button, palette command, and context-tree deep-links all land there) and the standalone drawer, which now serves only non-session routes (Landing/home). The file-tree column drops its changes/all pills and always shows the full tree. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
kateebonner
force-pushed
the
kate/vault-side-panel
branch
from
July 27, 2026 21:49
d548f35 to
faa56f7
Compare
kateebonner
marked this pull request as ready for review
July 27, 2026 22:20
kateebonner
added a commit
that referenced
this pull request
Jul 27, 2026
Context tree: organic Obsidian-like graph layout (stacked on #68)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Stacked on #67 — diff unique here is the last commit.
Per Kate's direction: the session side panel's git review goes away entirely (Review tab, Git-changes list, Unified/Split, the changes file-tree pill), and the Vault browser moves in as a first-class tab.
VaultBrowserbody, two hosts: the session side panel's Vault tab, and the standalone drawer which now serves only non-session routes (Landing/home). Both ride the samevaultPanelstore, so the titlebar button, palette command, and context-tree vault-node clicks land in whichever host the current view owns — deep-links still open straight onto the file.Verification: typecheck clean, 442 app tests green, lint 0 errors.
🤖 Generated with Claude Code