design(projects): make the new-project composer clearly read as create - #207
Merged
Conversation
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.
Design-polish pass — driven by the LLM feedback panel (
npm run panel, wedding scenario). The panelist's top complaint (delight 3/10): "Had to type 'Wedding' multiple times without clear feedback on whether it was creating or searching… no visual distinction between search, create, and navigation actions."Looking at the flow, the cause is real: when the "New project" button expands into its composer, it drops the "+ New project" framing entirely — leaving a bare text input in a dashed box next to a generic "Add" button, which reads ambiguously like a search field.
Fix (CSS + a small markup tweak, tokens only)
Now the composer unmistakably communicates "create a project" on both viewports.
Also in this PR — a panel-tool fix
The same panel run surfaced a bug in the harness itself: the Amazon Nova Pro panelist errored because
amazon.nova-pro-v1:0isn't callable on-demand. Corrected its model id to the inference-profile formus.amazon.nova-pro-v1:0inscripts/llm-panel/scenarios.mjs, so all four panelists run.Before → after
Full quality gate green; presentational + a scripts-only tool fix. Verified against the sandbox (
AWS_PROFILE=personal).Note: the panel's other ask — a success confirmation toast after creating — is a behavior change (new UI/state), outside this loop's visual-polish mandate, so it's logged as a backlog item rather than included here.