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 causes overflow #21

Open
supermario opened this issue Oct 3, 2023 · 2 comments
Open

Tooltip causes overflow #21

supermario opened this issue Oct 3, 2023 · 2 comments
Labels
good first issue Good for newcomers help wanted Extra attention is needed

Comments

@supermario
Copy link
Member

supermario commented Oct 3, 2023

It seems the moveLeft trick for the tooltip rendering is causing an issue, because the pre-moved positioning seems to be affecting the width calculation of the page on mobile resolutions:

Screenshot 2023-10-03 at 13 18 34

Can be seen clearly when commenting out this line:

, moveLeft (tooltipWidth - 30 - 10)

How do we fix this?

@supermario supermario added good first issue Good for newcomers help wanted Extra attention is needed labels Oct 3, 2023
@VivekChatterjee
Copy link

Hi @supermario, so what is the expected behaviour for this?

@supermario
Copy link
Member Author

Hey @VivekChatterjee – I would expect:

  • The tooltip continues to render as it does now (this part is already correct)
  • The page doesn't end up with an unnecessary overflow-x scroll (this is the annoying part)

Also I'm not fussed for this to be solved purely with elm-ui, I'd be happy with a solution that uses Element.html with regular html/css.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants