`AnnotoriousPlugin` component doesn't reapply the `plugin` on its change about annotorious

[oleksandr-danylchenko]

Oh-oh... I think this issue appeared in the 1st place because I was using the Annotorious + TextAnnotator components in a wrong way... 👁️👁️👁️

I was trying to use the same annotator instance across mutliple pages at the same time. I thought that I can use filter to display only relevant annotations on a certain page, but I didn't consider that filter doens't remove the annos from the positions recalculation! So each page change my customly made ranges were getting needlesly revived on irrelevant content and destroyed 🤯
And I haven't mentioned the jittering that was happening on a page change when the filter was trying to remove the old highligths and display the new ones not quickly enough...
Morevover, that approach didn't allow me to use a static plugin, leading to all sorts of magic I described above...

So... The solution I found was to force the re-render of the Annotorious! It destroy the annotator instance and allows each page work with the annotator from scratch! No more monkey patching of the already monkey patched methods or redundant revivals 🎉

[rsimon]

Ah - sorry. Need to find some time later to read through this in detail. Just briefly (if this is of interest at all!). If you need to work with multiple anno instances in parallel - register common selection or lifecycle handlers, delete/update annotations regardless on which annotator they live etc. - there's a utility that can handle most of the plumbing for you:
https://github.com/annotorious/annotorious/tree/main/packages/annotorious-react-manifold

But as I said: not sure if this is even a use case for you.

[oleksandr-danylchenko]

Fortunately, I think that having just a single annotator instance per page should be enought. And it makes the code more straightforward anyway 😅

[oleksandr-danylchenko]

I was just wondering "why does the A9S have 2 React implementations?" 😅
But turns out the manifold should be used for handle multiple annotator instances instead...