Skip to content

H-6729: Fix slideover positioning - #9104

Open
CiaranMn wants to merge 1 commit into
mainfrom
cm/fix-slideover-positioning
Open

H-6729: Fix slideover positioning#9104
CiaranMn wants to merge 1 commit into
mainfrom
cm/fix-slideover-positioning

Conversation

@CiaranMn

Copy link
Copy Markdown
Member

🌟 What is the purpose of this PR?

Changes in #9071 affected the supply chain slideover positioning. This PR fixes it.

@CiaranMn CiaranMn self-assigned this Jul 27, 2026
@cursor

cursor Bot commented Jul 27, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
Localized UI animation/positioning change in one shared slide-over component with no auth, data, or API impact.

Overview
Restores correct supply chain slide-over positioning after a regression from #9071.

The drawer still slides in/out by animating right (not transform, so nested position: fixed modals stay viewport-relative). The change moves that offset off Panda css() classes: the positioner no longer toggles positionerVisible / a base right: -960px in stylesheets. ArkDialog.Positioner now sets style={{ right: visible ? 0 : "-960px" }} while keeping the right transition on positionerBase.

Reviewed by Cursor Bugbot for commit cb9527a. Bugbot is set up for automated code reviews on this repo. Configure here.

@github-actions github-actions Bot added area/apps > hash* Affects HASH (a `hash-*` app) type/eng > frontend Owned by the @frontend team area/apps labels Jul 27, 2026
@vercel

vercel Bot commented Jul 27, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
hash Ready Ready Preview, Comment Jul 27, 2026 11:25am
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
hashdotdesign-tokens Ignored Ignored Jul 27, 2026 11:25am
petrinaut Skipped Skipped Jul 27, 2026 11:25am

@CiaranMn
CiaranMn added this pull request to the merge queue Jul 27, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jul 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/apps > hash* Affects HASH (a `hash-*` app) area/apps type/eng > frontend Owned by the @frontend team

Development

Successfully merging this pull request may close these issues.

3 participants