Merge pull request #615 from mathjax/issue2530

Make sure hitbox is translated along with the hit-able element. (mathjax/MathJax#2530)
mathjax · Mar 8, 2021 · 6b98ce7 · 6b98ce7
2 parents cc5ba3a + 310f046
commit 6b98ce7
Showing 1 changed file with 10 additions and 3 deletions.
diff --git a/ts/output/svg/Wrapper.ts b/ts/output/svg/Wrapper.ts
@@ -243,12 +243,19 @@ CommonWrapper<
    */
   public place(x: number, y: number, element: N = null) {
     if (!(x || y)) return;
+    const adaptor = this.adaptor;
+    const translate = 'translate(' + this.fixed(x) + ', ' + this.fixed(y) + ')';
     if (!element) {
       element = this.element;
+      if (this.node.attributes.get('href')) {
+        const rect = adaptor.previous(element);
+        if (rect && adaptor.kind(rect) === 'rect' && adaptor.getAttribute(rect, 'data-hitbox')) {
+          adaptor.setAttribute(rect, 'transform', translate);
+        }
+      }
     }
-    let transform = this.adaptor.getAttribute(element, 'transform') || '';
-    transform = 'translate(' + this.fixed(x) + ', ' + this.fixed(y) + ')' + (transform ? ' ' + transform : '');
-    this.adaptor.setAttribute(element, 'transform', transform);
+    let transform = adaptor.getAttribute(element, 'transform') || '';
+    adaptor.setAttribute(element, 'transform', translate + (transform ? ' ' + transform : ''));
   }
 
   /**