Set and remove element aria-describedby base on popper

[tuanthieu2993]

Summary

To fix aria attribute do not have valid value, issue: https://github.com/tvkhoa/react-tippy/issues/89

As describedby need to point to a valid id, that means id that exists, while current implementation set it from beginning and keep it there even when the popper is hidden - removed from DOM.

This update will help solve issue as we only set it when show and remove when hide the popper

Screenshots

When page is init with tooltipped elements

When popper is shown

When popper is hidden

[carlosparreno]

Hi @tvkhoa, this fix was approved. Will this be merged and released soon? Thanks!

[kjchiu]

bueller? This is breaking aXe tests and I'd like to re-enable them

[EmaSuriano]

Hello guys, I recommend switching to https://github.com/atomiks/tippy.js-react It's the official package and it's totally a11y compliance!