Skip to content

fix(ui): give drawer --inset a unit so popup slides#807

Open
boldurean wants to merge 1 commit into
cosscom:mainfrom
boldurean:fix/drawer-slide-inset-unitless-zero
Open

fix(ui): give drawer --inset a unit so popup slides#807
boldurean wants to merge 1 commit into
cosscom:mainfrom
boldurean:fix/drawer-slide-inset-unitless-zero

Conversation

@boldurean

@boldurean boldurean commented Jun 25, 2026

Copy link
Copy Markdown

--spacing(0) compiles to unitless 0; popup transforms add it inside calc() (translateY(calc(100% + … + var(--inset)))). Adding a number to a length is invalid calc → transform dropped → popup never starts off-screen, so it fades instead of sliding. All positions affected. 0px keeps the calc a valid length.

Closes #803


Open in Devin Review

`--spacing(0)` compiles to unitless `0`; popup transforms add it
inside calc() (`translateY(calc(100% + … + var(--inset)))`). Adding
a number to a length is invalid calc → transform dropped → popup
never starts off-screen, so it fades instead of sliding. All
positions affected. `0px` keeps the calc a valid length.

Closes cosscom#803
@vercel

vercel Bot commented Jun 25, 2026

Copy link
Copy Markdown

Someone is attempting to deploy a commit to the cal Team on Vercel.

A member of the Team first needs to authorize it.

@vercel vercel Bot temporarily deployed to Preview – coss-com-origin June 25, 2026 10:39 Inactive
@vercel

vercel Bot commented Jun 25, 2026

Copy link
Copy Markdown

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

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
coss-com-origin Skipped Skipped Jun 25, 2026 10:39am

Request Review

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Open in Devin Review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Drawer animations are not working correctly

1 participant