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

[IconButton] Tooltip hidden behind TableRowColumn's row #4671

Closed
lucasbento opened this issue Jul 10, 2016 · 19 comments · Fixed by #5014
Closed

[IconButton] Tooltip hidden behind TableRowColumn's row #4671

lucasbento opened this issue Jul 10, 2016 · 19 comments · Fixed by #5014
Labels
bug 🐛 Something doesn't work

Comments

@lucasbento
Copy link

Problem description

When used inside a <TableRowColumn/>, <IconButton/>'s tooltip appears to be behind the rows, it appears during the transition and disappears on row's limit.

IconButton tooltip error

Steps to reproduce

Simply include a <IconButton/> component using tooltip prop inside a <TableRowColumn/>.

Versions

  • Material-UI: 0.15.2
  • React: 15.2.1
  • Browser: Chrome 51.0.2704.106 (64-bit)
@bteepell
Copy link

Have you found a resolution to this?

@lucasbento
Copy link
Author

@bteepell nope, just did not use it since I had to deliver the project fast.

But still waiting for a fix or workaround.

@bteepell
Copy link

I changed the height of the tablerow to 65px from default 48 and size of the IconButton style to 24px,24px. Worked for me

@bteepell
Copy link

This is not only a Chrome issue.

@mpontikes mpontikes mentioned this issue Aug 5, 2016
13 tasks
@seminull
Copy link

I'm having this issue also. It would be nice workaround if the tooltipPosition had a straight "left" or "right" option.

@lucasbento
Copy link
Author

This issue was previously fixed by #5014, which introduced the regression #5377.

We should definitely follow what @oliviertassinari said:

The <Tooltip /> should most likely be render outside of the current DOM hierarchy.

@lucasbento lucasbento reopened this Oct 12, 2016
@lucasbento lucasbento added the bug 🐛 Something doesn't work label Oct 12, 2016
@lmammino
Copy link

lmammino commented Nov 3, 2016

Very interested on this as well!

@bteepell
Copy link

style set to overflow: 'visible' works for me on all browsers but in IE 11 and IE only the very first row the Tooltip does not overflow.

@peterjcaulfield
Copy link

Thanks @bteepell! this is the only thing that worked for me in stopping the tooltip being hidden by it's parent container 👍

@JustinBeckwith
Copy link

I was also able to fix this by using position: absolute on the container in the row that has my buttons.

@alexhayes
Copy link
Contributor

@bteepell's suggestion works however be aware that you need to change the tooltipPosition of the last row, for example;

<TableRowColumn style={{ overflow: 'visible' }}>
  <IconButton
    tooltip='Mark done'
    tooltipPosition={isLastRow ? 'top-center' : 'bottom-center'}
  >
    <DoneIcon />
  </IconButton>
</TableRowColumn>

@IzaGz
Copy link

IzaGz commented Jul 6, 2017

nothing works , tooltip still hidden

@sashion
Copy link

sashion commented Jul 11, 2017

@IzaGz <TableRowColumn style={{ overflow: 'visible' }}> should do the job.

@kr1stjans
Copy link

kr1stjans commented Jul 29, 2017

None of the solutions worked for my case.
tooltip

I have two cards next to each other and tooltip is hidden under the second card. I tried to change tooltip and container properties without any luck. Any advices?

Also this:
tooltip2
Top icon's tooltip is below bottom icon.

@oliviertassinari
Copy link
Member

Closing for #2230

@IzaGz
Copy link

IzaGz commented Sep 23, 2017

@oliviertassinari why is it closing? it does not work!

@oliviertassinari
Copy link
Member

@IzaGz, we moved our interest on the v1-beta branch where this issue is mostly fixed.

@ralphsmith80
Copy link

I'm noticing this with expansion panel on 1.0.0-beta.31.

Setting CollapseProps={{style:{ overflow: 'visible' }}} fixes the tooltip issue but cause the content to not be hidden when the panel is collapsed.

Here's a codesandbox example

I agree with the @octavioamu assessment that

The should most likely be render outside of the current DOM hierarchy.

Is it worth filing a new issue for this in v1 or is it on the roadmap already?

@MrEfrem
Copy link

MrEfrem commented Jun 4, 2018

Also look same issue. overflow: visible is helped.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 Something doesn't work
Projects
None yet
Development

Successfully merging a pull request may close this issue.