refactor: polish Trade UI, Schedule, and nav pills#90
Merged
Conversation
HeaderBar NavPill — active state now solid accent (pink) with subtle glow shadow, hovered state lights up with soft accent bg. Far stronger visual feedback than the previous accent-bg-strong ghost variant. DcaSchedulesPanel — minimalist redo. Header drops the rotation glyph, becomes a thin uppercase mono label + tabular count. Rows lose the raised surface and become flat with a 6px status dot (live = pink + glow, paused = muted). Title becomes mono with arrow notation; meta line uses subtle bullet dividers. Action buttons go fully ghost (transparent, muted) — they only carry weight on hover. Footnote becomes a single-line dashed-rule foot, no italics. ExecutionPanel FormCard — step number becomes a thin 22px outline circle (mono digit) instead of an accent-filled pill — title leads the eye. Subtitle moves to right with uppercase mono treatment. Card surface gets a faint glass tint + soft inset highlight so it reads as a layered surface instead of a stark outlined box. Header gets a dashed underline for tight visual separation. Sub-labels become uppercase mono caps for label/value clarity. All values driven by design-system tokens; no new colours introduced.
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.
Summary
Three small but meaningful visual upgrades:
HeaderBar nav pills — active state goes from a soft accent ghost to a solid pink with subtle glow. Hovered pills light up with the soft-accent bg. Active page is unmistakable; cursor feedback is now alive.
DcaSchedulesPanel — full minimalist redo. Header is a flat uppercase mono label + tabular count, no glyph. Rows go flat with a 6px status dot (live = pink + glow, paused = muted gray) leading the title. Title is mono with arrow notation; meta line uses bullet dividers. Action buttons (Pause / Cancel) are fully ghost — they only earn ink on hover. Footnote becomes a single dashed-rule line.
ExecutionPanel FormCard — step number becomes a thin 22px outline circle so the title leads the eye instead of a fat colored pill. Subtitle moves right as uppercase mono caps. Card surface picks up a faint glass tint + soft inset highlight (reads as layered, not boxy). Header gets a dashed underline for tight separation. Sub-labels become uppercase mono caps for label/value clarity.
All polish driven by design-system tokens — no new colors introduced.
Test plan