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

[Uptime] Waterfall: Add fixed width to the URL label tooltip to accommodate for longer URLs displayed in the content #90909

Closed
formgeist opened this issue Feb 10, 2021 · 2 comments · Fixed by #100147
Assignees
Labels
enhancement New value added to drive a business result Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability test-plan test-plan-ok issue has passed test plan v7.14.0

Comments

@formgeist
Copy link
Contributor

formgeist commented Feb 10, 2021

Summary

1. Add fixed width to the tooltip to improve the readability for commonly truncated long labels

Screenshot 2021-02-09 at 14 38 30

We should set the fixed width to 570px to keep within the last breakpoint we have for responsive viewports. Any more characters will make the tooltip wrap the content.

Waterfall - Tooltip width

2. Change the delay of when the tooltip is displayed

EUI provides an option to add a longer delay for when the tooltip is displayed, which is especially useful for repeating items like this. We can set the delay="long" on the tooltip.

@formgeist formgeist added enhancement New value added to drive a business result Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability labels Feb 10, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/uptime (Team:uptime)

@justinkambic
Copy link
Contributor

I had some offline discussion about this, but I think simply using something like 90vw for the tooltip max-width will be nicer than using a fixed width. This seems like a pretty common UX expectation.

Example:

image

image

image

image

@justinkambic justinkambic removed their assignment Jul 16, 2021
@dominiqueclarke dominiqueclarke self-assigned this Jul 19, 2021
@dominiqueclarke dominiqueclarke added the test-plan-ok issue has passed test plan label Jul 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New value added to drive a business result Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability test-plan test-plan-ok issue has passed test plan v7.14.0
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants