Chat/terminal studio redesign + 7-feature discoverability & token-drawer fix - #33
Open
utkarshh37 wants to merge 2 commits into
Open
Chat/terminal studio redesign + 7-feature discoverability & token-drawer fix#33utkarshh37 wants to merge 2 commits into
utkarshh37 wants to merge 2 commits into
Conversation
- Introduced a new Message component for displaying chat messages with user, assistant, and wizard roles. - Removed Analyze, History, Score, and Wizard components to streamline functionality. - Updated AppState type to remove references to removed views and added a 'NONE' view type. - Enhanced user interaction with chips for assistant analysis and wizard options. - Improved styling and layout for better user experience in chat interface.
…er clip Composer: render all four inspector chips at rest (Score disabled with — until there's a draft, instead of hidden), rename FIT→MODELS, and replace the cryptic glyph buttons with labeled Wizard/Optimize/Compress pills. Token drawer: Tokens.tsx grids were repeat(4,1fr)/1fr-1fr-1fr from the old full-width view and overflowed the 460px DetailDrawer, clipping the last cards. Switch both to auto-fit minmax(140px,1fr) so they wrap; add flexShrink:0 to the drawer so it holds width on narrow viewports. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Contributor
Author
Verdict: PASS — approved, safe to merge. Build verified on branch (npm run build clean). |
Contributor
Author
Code Review Summary (Orion) — Verdict: PASSBuild check DONE: Discoverability fix reviewed line-by-line:
Non-blocking nit: actionBtn 10.5px uppercase + letterspacing may get tight in the drawer-open layout — worth an eyeball, not blocking. (Could not submit a formal approve since this PR is authored by the same account reviewing it — verdict stands as a comment.) |
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
2 participants
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.
Summary
chat/ChatSurface,Composer,Message,HistoryRail,DetailDrawer), replacing the SideNav + per-view layout.Discoverability fix (
chat/Composer.tsx)—disabled until there's a draft (was hidden entirely → read as "missing").FITchip renamed toMODELS.✧ Wizard/↑ Optimize/≡ Compress.Token drawer fix (
views/Tokens.tsx,chat/DetailDrawer.tsx)repeat(4,1fr)/1fr 1fr 1fr(old full-width view) and overflowed the 460px drawer, clipping the last cards. Switched torepeat(auto-fit, minmax(140px,1fr))so they wrap (2×2 stats, 2+1 cost).flexShrink: 0on the drawer so it holds 460px on narrow viewports.Test plan
/studiorenders; all 7 features visible at empty composer (Analyze=send, Score/Tokens/Context/Models chips, Wizard, History rail).npm run buildclean (please confirm in CI).🤖 Generated with Claude Code