Skip to content

refactor(vibenet): size the Validity price chart to its container - #129

Open
BrianBland wants to merge 1 commit into
brianbland/base-422-vibenet-validity-central-actorsfrom
brianbland/validity-ui-1-chart-sizing
Open

refactor(vibenet): size the Validity price chart to its container#129
BrianBland wants to merge 1 commit into
brianbland/base-422-vibenet-validity-central-actorsfrom
brianbland/validity-ui-1-chart-sizing

Conversation

@BrianBland

@BrianBland BrianBland commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Part 1/9 of the Validity demo UI rework (stacked on #128).

Replaces the fixed 960×440 viewBox with a ResizeObserver-measured canvas so the chart draws in CSS pixels at any container shape. This prepares the chart to sit beside the order ticket (part 3) without letterboxing — at today's full-width layout this is visually a no-op.

  • Axis text and candle geometry stay crisp because scales derive from measured pixels, not a scaled viewBox.
  • The header price/change readout now derives from candles directly, so it renders before the first measurement.
  • toCandles and friends are untouched; existing PriceCandles.test.ts still passes.

Generated with Claude Code

Full stack view below
image

@vercel

vercel Bot commented Sep 2, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
omni-ui Ready Ready Preview Sep 3, 2026 12:21am UTC

Request Review

@cb-heimdall

Copy link
Copy Markdown
Collaborator

🟡 Heimdall Review Status

Requirement Status More Info
Reviews 🟡 0/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1

Replace the fixed 960x440 viewBox with a ResizeObserver-measured
canvas so the chart draws in CSS pixels at any container shape.
Prepares the chart to sit beside the order ticket without
letterboxing; visually a no-op at today's full-width layout.

Generated with Claude Code

Co-Authored-By: Claude <noreply@anthropic.com>
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.

2 participants