Skip to content

fix(form): an order link lands on the form, framed in the accent line; release 1.15.0 - #24

Merged
anaidenko merged 2 commits into
mainfrom
fix/order-scroll-to-form
Oct 11, 2026
Merged

anaidenko merged 2 commits into
mainfrom
fix/order-scroll-to-form

Conversation

@anaidenko

@anaidenko anaidenko commented Oct 11, 2026 •

Copy link
Copy Markdown
Owner
  • The form and the email render only in the browser, after the browser has scrolled to
    #contact on load, so a link from another page (/audit's order links) stopped short of the form
    by the block's height: 590 px at 1440×900. Once the block is in, the section scrolls into view
    again, as the page's CSS says: an instant scroll during the browser's own smooth one landed
    12–45 px off on CI
  • An order link's form is framed like a marked project, the accent line on its left and a tint
    behind it; a plain visit's form is unchanged

…; release 1.15.0

- The form and the email render only in the browser, after the browser has scrolled to
  #contact on load, so a link from another page (/audit's order links) stopped short of the form
  by the block's height: 590 px at 1440×900. Once the block is in, the section scrolls into view
  again
- An order link's form is framed like a marked project, the accent line on its left and a tint
  behind it; a plain visit's form is unchanged
On CI the form mounted while the browser's own smooth scroll to #contact still ran, and an
instant scroll mixed with it, landing 12–45 px off. A scroll that follows the page's
scroll-behavior replaces the running one and lands on the margin. The test now throttles the CPU
to reproduce that order and waits for the scroll to settle.
@anaidenko
anaidenko merged commit 90a57d7 into main Oct 11, 2026
1 check passed
@anaidenko
anaidenko deleted the fix/order-scroll-to-form branch October 11, 2026 07:53
anaidenko added a commit that referenced this pull request Oct 11, 2026
#24 merged as 1.15.0 after #23 had released that number; its changes ship as 1.15.1.
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