Skip to content

fix(vibenet): keep the Validity chart zoomed to the tape - #139

Merged
BrianBland merged 2 commits into
brianbland/validity-ui-6-offset-sliderfrom
brianbland/validity-ui-10-chart-zoom
Sep 3, 2026
Merged

fix(vibenet): keep the Validity chart zoomed to the tape#139
BrianBland merged 2 commits into
brianbland/validity-ui-6-offset-sliderfrom
brianbland/validity-ui-10-chart-zoom

Conversation

@BrianBland

Copy link
Copy Markdown
Contributor

Part 10 of the Validity demo UI rework.

Placing an order with a far-away target (say $1 against a $0.07 mid) permanently zoomed the chart out: the y-domain included every resting level and every fill's target price unconditionally — even fills that had scrolled out of the 3-minute window. The domain math moves into an exported domainBounds helper with the rule: the tape always fits, in-window fill prices and levels within 15% of the last price still show, and anything farther expands the view only while its order row is hovered. Unit tests pin the hover/window/near-band behavior.

Generated with Claude Code

@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 4:36pm UTC

Request Review

@cb-heimdall

cb-heimdall commented Sep 3, 2026

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

@BrianBland
BrianBland added this pull request to the merge queue Sep 3, 2026
@BrianBland
BrianBland removed this pull request from the merge queue due to a manual request Sep 3, 2026
BrianBland and others added 2 commits September 3, 2026 09:35
A not-before block predicate (block_number >=) joins the expiry
bound: pick Off/5s/15s in a Delay chip row beside Expiry, with
delay forced shorter than expiry and both under the 8130 envelope
in concurrent mode. Annotations explain the lower bound, the
review modal shows the start time, and delayed orders carry
delaySeconds/minBlock through storage and the Submitted list.

Generated with Claude Code

Co-Authored-By: Claude <noreply@anthropic.com>
One order with a far-away target used to stretch the y-domain for
its whole life and beyond: the domain included every fill's target
price forever, even after the fill scrolled out of the window.
Extract the domain math into domainBounds: the tape always fits,
in-window fill prices and near-mid levels still show, but anything
beyond 15% of the last price expands the view only while its order
row is hovered.

Generated with Claude Code

Co-Authored-By: Claude <noreply@anthropic.com>
@BrianBland
BrianBland force-pushed the brianbland/validity-ui-10-chart-zoom branch from fb95c7c to 80bc639 Compare September 3, 2026 16:36
@BrianBland
BrianBland force-pushed the brianbland/validity-ui-9-delay branch from 6572f86 to 99fdc6f Compare September 3, 2026 16:36
Base automatically changed from brianbland/validity-ui-9-delay to brianbland/validity-ui-6-offset-slider September 3, 2026 16:37
@BrianBland
BrianBland merged commit 69c2fb4 into brianbland/validity-ui-6-offset-slider Sep 3, 2026
9 of 10 checks passed
@BrianBland
BrianBland deleted the brianbland/validity-ui-10-chart-zoom branch September 3, 2026 16:37
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.

3 participants