Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tooltip can overlap with Menu when close to window border because of flip #7268

Open
2 tasks done
jvllmr opened this issue Dec 16, 2024 · 1 comment · May be fixed by #7281
Open
2 tasks done

Tooltip can overlap with Menu when close to window border because of flip #7268

jvllmr opened this issue Dec 16, 2024 · 1 comment · May be fixed by #7281

Comments

@jvllmr
Copy link
Contributor

jvllmr commented Dec 16, 2024

Dependencies check up

  • I have verified that I use latest version of all @mantine/* packages

What version of @mantine/* packages do you have in package.json?

7.15.1

What package has an issue?

@mantine/core

What framework do you use?

Next.js

In which browsers you can reproduce the issue?

All

Describe the bug

In my project I have a component which has both a menu and a tooltip and the tooltip is only visible when clicking the component / opening the menu. When the component gets close to the top window border because of scroll, the tooltip covers the menu. In this case I don't care if the tooltip would be cut off by the window border, but it is important that the menu is visible.

If possible, include a link to a codesandbox with a minimal reproduction

https://codesandbox.io/p/sandbox/mantine-react-template-forked-vvtdj6

Possible fix

Add a prop to tooltip which allows to disable/remove the flip floating-ui middleware. Maybe similar to Menu's middlewares prop.

Self-service

  • I would be willing to implement a fix for this issue
@rtivital
Copy link
Member

You are welcome to submit a pr that adds an option to configure floating ui middlewares

@jvllmr jvllmr linked a pull request Dec 18, 2024 that will close this issue
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 a pull request may close this issue.

2 participants