Skip to content

fix(layout): prioritise chart height across all laptop resolutions - #5

Merged
SyntaxSphinx merged 1 commit into
mainfrom
feat/tradeflow-console-replay-and-engine-hardening
Aug 7, 2026
Merged

SyntaxSphinx merged 1 commit into
mainfrom
feat/tradeflow-console-replay-and-engine-hardening

Conversation

@SyntaxSphinx

Copy link
Copy Markdown
Collaborator

Invert center-column flex so chart grows to fill space (flex:1 1 0) while the Execution Feed becomes a compact fixed strip (flex:0 0 var(--feed-h)). OrderTicket now also flexes to fill its column, matching chart height exactly, and OrderJourney uses the same feed-h variable - satisfying the same-height requirement for Book/Chart/Ticket across all breakpoints.

  • Replace --chart-h and --ticket-h with a single --feed-h variable
  • Add CSS vars for pipeline/NBBO chrome density (pipeline-pad-y, pipeline-inner-gap, pipeline-node-size, nbbo-pad-y)
  • Tag pipeline subtext .pipeline-node-sub; hide at <=1440px to reclaim ~10px
  • workspace overflow set to hidden; no workspace-level scroll bar
  • Height breakpoints (800px/720px) tighten dock and feed further

Tested: 1920x1080 / 1600x900 / 1536x864 / 1440x900 / 1366x768 / 1280x720

Invert center-column flex so chart grows to fill space (flex:1 1 0) while
the Execution Feed becomes a compact fixed strip (flex:0 0 var(--feed-h)).
OrderTicket now also flexes to fill its column, matching chart height exactly,
and OrderJourney uses the same feed-h variable - satisfying the same-height
requirement for Book/Chart/Ticket across all breakpoints.

- Replace --chart-h and --ticket-h with a single --feed-h variable
- Add CSS vars for pipeline/NBBO chrome density (pipeline-pad-y,
  pipeline-inner-gap, pipeline-node-size, nbbo-pad-y)
- Tag pipeline subtext .pipeline-node-sub; hide at <=1440px to reclaim ~10px
- workspace overflow set to hidden; no workspace-level scroll bar
- Height breakpoints (800px/720px) tighten dock and feed further

Tested: 1920x1080 / 1600x900 / 1536x864 / 1440x900 / 1366x768 / 1280x720
@SyntaxSphinx
SyntaxSphinx merged commit 6b014f7 into main Aug 7, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant