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

Far too slow on initial render of pages with > 100 tooltips #100

Closed
kybishop opened this issue Dec 4, 2017 · 1 comment
Closed

Far too slow on initial render of pages with > 100 tooltips #100

kybishop opened this issue Dec 4, 2017 · 1 comment
Milestone

Comments

@kybishop
Copy link
Owner

@kybishop kybishop commented Dec 4, 2017

This is even a problem with lazy rendering enabled. I believe the issue is that we aren't lazy enough. Will have to see about pushing lazy-rendering and visibility toggling down a layer into ember-popper.

It's quite possible a separate and/or custom impl of ember-popper is needed for tooltip libraries; ember-popper might already be too far down the road of supporting popovers alone for people to accept the burden of adding hide/show logic to the addon.

@kybishop

This comment has been minimized.

Copy link
Owner Author

@kybishop kybishop commented Dec 20, 2017

The lazy-rendering of #102 is unfortunately not a11y friendly since information is hidden until a tooltip is rendered. I'll have to look more into a11y to see if there is a known solution to this for lazy rendering.

If there is no known solution for screen readers, I plan to implement a barebones tooltip component after #102 to have an a11y-friendly solution. The idea is to prioritize performance over all else, and not offer any options besides hideOn, showOn, and isVisible.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant
You can’t perform that action at this time.