Repository navigation
fix(ui): resolve UX audit issues across workflows - #324
Merged
Merged
Conversation
- Zones: bulk add groups read the modal result as an array and did nothing - Resource imports: the view link was always disabled by disabled="false" - Edges: editing an edge hid the one-time API key of the last created edge - Drivers: the Modules tab only listed the first page of modules Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- Edit modals ask before Escape, backdrop or X close them with unsaved changes, block closing while saving, and show the saving text - Saving one settings level or metadata block keeps edits on the others - Zone tree CSV import confirms before it creates zones - Bulk driver update starts with no drivers selected Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- Delete confirmations name the item and use a red Delete button - Start and stop system confirmations name the system - Error toasts stay open until dismissed - Failed list loads show an error instead of an empty list - The sidebar no longer says a list is empty while it loads Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- Below 1024px the list sidebar gives way to the item picker, so the details pane keeps its width on tablets - Placeholders are readable in dark mode - Popover menus close on Escape and on navigation - Logout moves to the bottom of the user menu Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- Icon-only buttons get an accessible name from their tooltip or an explicit label, and icon names are no longer read aloud - Hotkeys no longer fire while a select or menu has focus - Validation toasts list readable field names - Fix typos and wrong booking limit messages, name the broker on delete - Zones get their own nav icon Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
This branch was successfully deployed
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
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.
Problem
A UX audit of the app found broken actions, ways to lose unsaved work, unclear confirmations, failed loads that looked like empty lists, a cramped tablet layout, and icon buttons without accessible names.
Changes
Each commit is one block of fixes.
I tested the layout and menu changes in mock mode (
?mock=true) at 660px, 820px and 1280px, in light and dark mode.Not in this PR:
Reviewed with sol-loop: 10 rounds with GPT-6.1-Sol.
Changes made by Claude Opus 5.5 (
claude-opus-5-5) in Claude Code, running in T3 Code.🤖 Generated with Claude Code