[bugfix/LEMS-2805-blockquote-hidden-in-explanation-on-mobile] Add a c…

…omment to clarify "parentElement" usage.
Khan · Feb 4, 2025 · cefeed6 · cefeed6
1 parent ec925c8
commit cefeed6
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/packages/perseus/src/components/zoomable.tsx b/packages/perseus/src/components/zoomable.tsx
@@ -197,6 +197,9 @@ class Zoomable extends React.Component<Props, State> {
         let parentNode = this._node;
         let currentNode: HTMLElement | null = parentNode;
         while (currentNode && currentNode.offsetWidth === 0) {
+            // When traversing up the DOM tree, we need to start at the "parentElement"
+            //      because "closest()" can match the current element,
+            //      and if it does so, then we are stuck in an infinite loop.
             currentNode =
                 currentNode.parentElement?.closest(".perseus-renderer") ?? null;
         }