#185 Fixed popup rendering on Android

[oleksandr-danylchenko]

Issues

See - #185 (comment)

In the #123 I made the popup presence check resilient to the underlying collapsed ranges in the target's selectors.

However, it caused a timing regression on Android. When the annotation is processed in the popup's useEffect, the target and its bounds aren't yet populated.

text-annotator-js/packages/text-annotator-react/src/TextAnnotationPopup/TextAnnotationPopup.tsx

Lines 95 to 98 in 5b9abad

	useEffect(() => {
	if (annotation?.id) {
	const bounds = r?.state.store.getAnnotationBounds(annotation.id);
	setOpen(Boolean(bounds));

Also, the timing of the selection change event that populates the target differs on Android:

Android	Mac

Changes Made

I added a check for the selectors' presence on the annotation's target. That way, the effect will react properly to their addition and re-run the bounds querying.

[rsimon]

I need to look at this tomorrow with a fresh mind (it's been a long day). But whatever you did - it's working, thanks! :-)