fix: DEV-1220: fix HTML misplacements (#432)

* fix: DEV-1220: draft right after HyperText loaded

HyperText with regions creates new state in history, so new draft is
triggered. Reinit history after deserialization.

* fix: DEV-1220: labeling HyperText with emoji

Text was fixed, but HyperText still has shifted offsets for regions
in the text after emoji.

* fix: undo/redo for annotation from prediction

* fix: update regions in fresh suggestions

Co-authored-by: Nick Skriabin <[email protected]>
Co-authored-by: Max Tkachenko <[email protected]>

Author: 3 people

src/stores/AnnotationStore.js

@@ -840,7 +840,7 @@ const Annotation = types
         });
       }
 
-      self.objects.forEach(obj => obj.needsUpdate?.());
+      self.updateObjects();
     },
 
     /**
@@ -1332,7 +1332,8 @@ export default types
 
       selectAnnotation(c.id);
       c.deserializeResults(s);
-      c.updateObjects();
+      // reinit will trigger `updateObjects()` so we omit it here
+      c.reinitHistory();
 
       // parent link for the new annotations
       if (entity.pk) {

src/tags/object/RichText/view.js

@@ -260,6 +260,8 @@ class RichTextPieceView extends Component {
     // @todo both loops should be merged to fix old broken xpath using "dirty" html
     if (initial) {
       item.initGlobalOffsets(root);
+      // lot of changes possibly made during regions preparation, so clear the history
+      item.annotation.reinitHistory();
     }
 
     // Apply highlight to ranges of a current tag

src/utils/selection-tools.js

@@ -1,4 +1,4 @@
-import { clamp } from "./utilities";
+import { clamp, isDefined } from "./utilities";
 
 const isTextNode = node => node && node.nodeType === Node.TEXT_NODE;
 
@@ -685,7 +685,7 @@ const findGlobalOffset = (node, position, root) => {
     }
 
     if (isText || isBR) {
-      let length = currentNode.length ?? 1;
+      let length = isDefined(currentNode.length) ? [...currentNode.textContent].length : 1;
 
       if (atTargetNode) {
         length = Math.min(position, length);