feat: UI/UX improvement#9
Conversation
There was a problem hiding this comment.
Code Review
This pull request introduces a unified toast notification system using a new useAppToast hook, adds tab count badges, and refactors the sidebar to display and filter vaults directly. It also improves accessibility with keyboard navigation on tables and cards, refines UI styling (such as a dedicated 'Danger zone' for purging secrets), and caps bulk deletion concurrency to prevent API throttling. The reviewer pointed out a React anti-pattern in Sidebar.tsx where the VaultRow component is defined inside the Sidebar component, which causes unnecessary unmounting/remounting and loss of keyboard focus during re-renders.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
Defining VaultRow inside Sidebar created a new component type each render, remounting the subtree and dropping keyboard focus during filtering. Convert to renderVaultRow helper. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
No description provided.