Skip to content

Bug: fix play with menu vis bug - #448

Open
kylebenzo wants to merge 3 commits into
JMBeresford:mainfrom
kylebenzo:bug/fix-play-with-menu-vis-bug
Open

Bug: fix play with menu vis bug#448
kylebenzo wants to merge 3 commits into
JMBeresford:mainfrom
kylebenzo:bug/fix-play-with-menu-vis-bug

Conversation

@kylebenzo

Copy link
Copy Markdown

Fixes a visualization bug showing the "Play with" submenu on a game.

image

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

This is duplicated in #447 , please remove from this PR for clarity.

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

This is duplicated in #447 , please remove from this PR for clarity.

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

This is duplicated in #447 , please remove from this PR for clarity.

<div
className={cn(
"flex relative",
"[&_*[data-radix-popper-content-wrapper]]:contents sm:[&_*[data-radix-popper-content-wrapper]]:block",

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

These classes are necessary for the popover to correctly render on mobile. Can we look into a different solution here?

It would be helpful to have a description of the root cause for this visual bug.

@JMBeresford
JMBeresford force-pushed the main branch 8 times, most recently from b46014c to ee77021 Compare January 27, 2026 02:38
@JMBeresford
JMBeresford force-pushed the main branch 18 times, most recently from 3b579a9 to da8f60d Compare February 5, 2026 01:08
@JMBeresford
JMBeresford force-pushed the main branch 25 times, most recently from 2b76cc7 to 798c1b7 Compare February 8, 2026 00:58
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