Skip to content

fix(docs): correct mobile copy page menu positioning#976

Merged
Yeom-JinHo merged 1 commit into
magicuidesign:mainfrom
d1rshan:fix/docs-copy-page-button
May 29, 2026
Merged

fix(docs): correct mobile copy page menu positioning#976
Yeom-JinHo merged 1 commit into
magicuidesign:mainfrom
d1rshan:fix/docs-copy-page-button

Conversation

@d1rshan

@d1rshan d1rshan commented May 29, 2026

Copy link
Copy Markdown
Contributor

Description

Fix the mobile Copy Page menu positioning on docs pages.

This removes the custom empty PopoverAnchor from DocsCopyPage and keeps the mobile menu
on the standard Radix trigger/content path so the popover is positioned relative to the
actual trigger button.

Changes

  • removed the unused custom PopoverAnchor from DocsCopyPage
  • kept the mobile menu on the default PopoverTrigger + PopoverContent pattern
  • adjusted the mobile popover alignment to align="end"
  • left desktop DropdownMenu behavior unchanged

Motivation

On mobile, the Copy Page action menu could render in the wrong position because the
component defined a custom popover anchor without anchoring it to a real element.

Removing that anchor fixes the positioning issue at the source while keeping the
implementation aligned with how other Radix menus are structured in the codebase.

Breaking Changes

None.

Screenshots

Device / Browser / Viewport: iPhone/Chrome or Safari/mobile viewport under sm

Before After
image image

@vercel

vercel Bot commented May 29, 2026

Copy link
Copy Markdown

@d1rshan is attempting to deploy a commit to the product-studio Team on Vercel.

A member of the Team first needs to authorize it.

@Yeom-JinHo Yeom-JinHo self-requested a review May 29, 2026 14:35

@Yeom-JinHo Yeom-JinHo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@d1rshan
Tysm for your contribution.

@Yeom-JinHo Yeom-JinHo merged commit 6f662f3 into magicuidesign:main May 29, 2026
5 of 6 checks passed
@d1rshan d1rshan deleted the fix/docs-copy-page-button branch May 29, 2026 15:01
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