feat(ui): lead with the meat mark and drop the tree for single-file reviews - #19
Merged
Conversation
…eviews Replace the robot avatar in the header with the meat logo, rendered with object-fit: contain so the die-cut outline survives — the old circular cover crop would have sliced it off. ai.png had no remaining reference afterwards, so its embed and plumbing go with it. Remove the meatcheck logo and wordmark from the foot of the sidebar, along with the .brand and .logo rules that only it used. Hide the sidebar entirely when the review covers one file: there is nothing to navigate, so the aside is skipped and the workspace collapses to a single column. The narrow-viewport rule needs its own override because the stacked layout reserves an auto-sized row for the tree. With no sidebar, the markdown preview's 980px measure is centred rather than left-aligned. The padding sits on the container, not the blocks, so list items — which live inside a <ul>/<ol> wrapper — stay in step with the headings around them. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_0134abbE1ZayWc3qeFMeXEZN
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.
What changed
Header now leads with the meat mark.
ai.png(the robot sticker) is replaced bylogo.png. The old.header-avatarwas a 56px circle withobject-fit: cover, which would have sliced off the die-cut white outline that makes the mark read as a sticker — it's nowobject-fit: containin a 72px box, with a-10pxblock margin absorbing the artwork's transparent top and bottom bands so the header ends up 6px shorter than before rather than taller.With that swap
ai.pnghad no remaining reference, so its embed directive,avatarBytes, and theAvatartemplate field are gone and the file is deleted.Sidebar footer removed. The
.sidebar-brandblock is out of the template, along with its CSS and the.brand/.logorules that only it used.Single-file reviews hide the left column. New
(*ReviewModel).SingleFile()countsDiffFilesin diff mode andFilesotherwise. At 1 the<aside>isn't rendered at all — no collapse chevron, no resize handle, no--sidebar-widthstyle — and.workspacegetsno-sidebar. The filename stays in the column header, so nothing is lost.Markdown measure centres when the sidebar is gone.
.md-block's 980pxmax-widthis a good reading measure, but left-aligned in a full-width panel it left a growing right gutter. The padding goes on.markdown-file-previewrather than asmargin-inline: autoon the blocks — list items sit inside a<ul>/<ol>wrapper, not directly in the container, so auto margins would have centred each item within its list and drifted every bullet out of step with the headings.Two things worth a reviewer's eye
max-width: 900pxblock needs its own.workspace.no-sidebaroverride. That media query setsgrid-template-rows: auto 1frfor the stacked tree-above-file layout; without the override the file would be squashed into the auto row with no tree to justify it.Testing
Three tests added in
http_render_test.go: single-file, multi-file, single-file-diff.gofmt,go vet, and the full suite pass.🤖 Generated with Claude Code
https://claude.ai/code/session_0134abbE1ZayWc3qeFMeXEZN