Redesign annotate page with improved layout and mobile support#406
Closed
Redesign annotate page with improved layout and mobile support#406
Conversation
Full visual overhaul: two-panel grid (sticky preview + wizard), numbered circle stepper replacing tab bar, unified notice styling, collapsible reference panel, on-palette colors throughout, and removal of <br> hacks. Co-Authored-By: Claude <noreply@anthropic.com>
…button Co-Authored-By: Claude <noreply@anthropic.com>
Co-Authored-By: Claude <noreply@anthropic.com>
Co-Authored-By: Claude <noreply@anthropic.com>
- Add collapsible preview card on mobile with smooth animation - Compact screenshot (4:3 ratio), reduce content min-height on mobile - RecordType grid single-column on narrow screens - Switch stepper colors from wine to amber for dark-background readability Co-Authored-By: Claude <noreply@anthropic.com>
Co-Authored-By: Claude <noreply@anthropic.com>
Co-Authored-By: Claude <noreply@anthropic.com>
josh-chamberlain
requested changes
Feb 17, 2026
Contributor
josh-chamberlain
left a comment
There was a problem hiding this comment.
This is a good start! I like much of what it did. Other stuff is either a lateral move or actively worse, so please allow me to smack down some of its ideas. Once these are resolved, I think it's ready for dev testing and likely quick approval.
- For it to address #405, it needs to address all the requirements. In particular the rounded corners irk me.
- In general, we should not be using scoped styles and creating classes—almost ever. This will be very important to reinforce with Claude. We're using Tailwind, and the Tailwind philosophy (good reading!). Styles are separate from structure. It makes it much easier to see what's going on, rather than scrolling up and down to see which styles are applied to what. And if we're styling semantically, we don't need to create our own styles almost ever—tailwind has done that for us. Joy. I would love to see all those classes moved inline. It even supports states.
- In general, for colors of buttons and their hierarchy, reference the map's sidebar. This was fine-tuned recently with an eye for compactness and cascading hierarchy, as well as looking nice in both dark and light mode.
- Never amber or orange, please! Primary action buttons are
gold. We have a range of golds and goldneutrals that are intended to show buttons of lower hierarchy or rendering information that is not clickable in wines and wineneutrals, all while being readable on any background imaginable.

- On this topic, things should only be gold if they're actively clickable—we dim them if they're inactive, and they are only gold if they are the
primarybutton button for the workflow. The design system already has preset components for button-primary and -secondary and such, and they can be modified from there, but it's a great starting point. We want to give people a cue for what the intended action is—as it stands, the multiple gold (or amber) things are competing for attention; what's most important? - The dark wine text is not a good look on the black background. We have a whole set of lovely tints and shades of wine, gold, wineneutral, and goldneutral. It may be best to reference the
map sidebarfor hints as to how to use with those.

- Tinted backgrounds reduce color contrast, which reduces readability and strains the eyes. If a background is tinted, it should be done with a very light touch—i.e. an
-800tint on a-900background. - The inline SVGs are quite the sight—but we should just use
fontawesomeicons for those. Claude is usually pretty good at picking out appropriate ones and adding them properly. - The container under the header, in which live such components as
select a location, does not need its own tinted background and padding. Those items are united by their placement, and don't need to be boxed off from the header or prev/next buttons.
…n system Primary action buttons and interactive states use brand-gold-*, non-clickable info banners and decorative elements use goldneutral-*. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…r selections Gold now appears only on the Next/Skip (TabControls) and Submit (Confirm) buttons. Selected states use brand-wine, stepper indicators use wineneutral, links use goldneutral-700/800 for adaptive contrast across light and dark mode backgrounds. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 task
Contributor
|
Probably can close this one out now right ? |
Contributor
Author
You bet we can! |
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.
#405
Summary
<br>spacer hacks, unified notice patterns, converted components to Composition APITest plan
🤖 Generated with Claude Code