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] When Tooltip is in TableHead, popper incorrectly appears in the upper left corner of screen #8467

Closed
1 task done
mattadjohnson opened this issue Sep 29, 2017 · 5 comments
Assignees
Labels
bug 🐛 Something doesn't work external dependency Blocked by external dependency, we can’t do anything about it

Comments

@mattadjohnson
Copy link

mattadjohnson commented Sep 29, 2017

  • I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior

The <Tooltip> popper should abide by the placement prop (default: bottom) when opening from within a cell in a <TableHead>

Current Behavior

The <Tooltip> popper always appears in the top left corner of the screen when opening from within a cell in a <TableHead>

Steps to Reproduce (for bugs)

  1. Create a <Table> with any number of rows/columns
  2. Place a <Tooltip> inside a cell in the <TableHead>
  3. Hover over the <Tooltip> to open the popper
  4. Notice that the popper is always in the upper left corner of the screen
  5. Notice that any <Tooltip> in a cell in the <TableBody> seems to work fine

You can see all of this here: https://codesandbox.io/s/nkpmr083lm

Context

I'm trying to give my users a help icon on certain column values in the table header that they can hover over to see a tooltip describing what it means.

Your Environment

Tech Version
Material-UI v1.0.0-beta.12
Material-UI Icons v1.0.0-beta.10
React v15.6.2 and v16.0.0 (tried both)
Browser Chrome Version 61.0.3163.100 (Official Build) (64-bit)
@rosskevin
Copy link
Member

Bug definitely, not sure if this upstream in popper.js or our initialization/use of it.

@FezVrasta any thoughts?

@rosskevin rosskevin added bug 🐛 Something doesn't work v1 labels Sep 29, 2017
@oliviertassinari
Copy link
Member

The issue seems to be on our side. Having a deeper look now.

@oliviertassinari
Copy link
Member

Actually no, it's on popper.js side.

@oliviertassinari oliviertassinari added the external dependency Blocked by external dependency, we can’t do anything about it label Sep 30, 2017
@oliviertassinari
Copy link
Member

Still, I have found a workaround.

@FezVrasta
Copy link

Care to open an issue on the Popper.js repository to track the problem?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 Something doesn't work external dependency Blocked by external dependency, we can’t do anything about it
Projects
None yet
Development

No branches or pull requests

4 participants