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] Typeahead search updates the content for every character entered #94421

Closed
formgeist opened this issue Mar 11, 2021 · 1 comment · Fixed by #99675
Closed

[Uptime] Typeahead search updates the content for every character entered #94421

formgeist opened this issue Mar 11, 2021 · 1 comment · Fixed by #99675
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

Summary

I was trying out the new typeahead search for monitors in Uptime, but I found that the experience a little jarring as the typeahead updates the page content for every character immediately. I would like to suggest that we add a small delay to avoid the slowed-down typing experience and optionally only update the content on submit (the user will need to hit enter to confirm their search). I also imagine it's firing a request for every single keyword entered which seems wasteful.

2021-03-11.at.13.44.05.-.Amber.Deer.mp4
@formgeist formgeist added enhancement New value added to drive a business result Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability labels Mar 11, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/uptime (Team:uptime)

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