Uh oh!
There was an error while loading. Please reload this page.
fix(amicode): pad entity-view modal + remove card-family gold tint - #16
Merged
Merged
Conversation
- Neutralize the gold-tinted hairline + accent wash on amicode-card (chip) and the shared ask-card / entity-rail / run-window surfaces (incl. the run-window hover) to plain border-base + bg-layer surfaces (Kate: drop the yellow side border). - Entity-view dialog body padding 4px 4px 0 -> 4px 20px 20px so content aligns with the header inset and History isn't jammed to the edge. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
- entity-view: content wrapper width min(680px,90vw) -> 100% so it fits the 640px dialog instead of overflowing and clipping on the right - problem-switcher: symmetric 20px padding (was 24/10 asymmetric), add a height cap (max-height 68vh) + internal scroll, and humanize the recorded timestamps (raw ISO -> relative via luxon) - remove the leftover gold left-rail on the question-body card Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Uh oh!
There was an error while loading. Please reload this page.
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 freeto 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.
What
Two UI-polish fixes to the Amicode entity surfaces (all in
packages/ui/src/amicode/amicode.css):Entity-view modal padding. The dialog body (Hamiltonian box, DETAILS/HISTORY, fields, footer) was inset only
4pxwhile the dialog header sits at20px, so content read as cramped and hugged the edges. Bumped[data-component="amicode-entity-view"]padding4px 4px 0→4px 20px 20px, so content aligns with the header inset and History no longer jams the bottom edge.Remove the gold tint from the card family. The
no-accent-railredesign kept a gold-tinted hairline + accent wash as the shared "AMICO identity" on the chip (amicode-card) and theask-card/entity-rail/run-windowsurfaces (including the run-window hover). Per product direction, neutralized these to a plainborder-basehairline +bg-layersurface — no yellow on resting or hover states.Intentionally left (functional / semantic, not the "yellow side border")
data-picked) — the selection indicator.Verification
dialog.css+amicode.css+ the exact DOM) across 380–900px — content padded, no clipping at any width.4px 20px 20px) and no longer contain the old gold borders (icon-accent) 20%/22%).bun typecheckclean (enforced by the pre-push hook).🤖 Generated with Claude Code