Completely ignore size given by htmlwidgets

pfh · Nov 1, 2023 · a52e427 · a52e427
1 parent 0d2d0a1
commit a52e427
Showing 1 changed file with 16 additions and 10 deletions.
diff --git a/inst/htmlwidgets/langevitour.js b/inst/htmlwidgets/langevitour.js
@@ -4,16 +4,16 @@ HTMLWidgets.widget({
     factory: function(el, width, height) {
         // Supplied width and height are based on getBoundingClientRect.
         // Wrong eg when using revealjs and slide is scaled.
-        // Ignore if we seem to be scaled.
-        let trustSize = () => {
-            let rect = el.getBoundingClientRect();
-            return rect.width == el.offsetWidth && rect.height == el.offsetHeight;
-        };
+        //let trustSize = () => {
+        //    let rect = el.getBoundingClientRect();
+        //    return rect.width == el.offsetWidth && rect.height == el.offsetHeight;
+        //};
 
-        if (!trustSize()) {
-            width = el.offsetWidth;
-            height = el.offsetHeight;
-        }
+        // May as will just never trust the width and height we are given.
+        //if (!trustSize()) {
+        width = el.offsetWidth;
+        height = el.offsetHeight;
+        //}
 
         let tour = new langevitour.Langevitour(el, width, height);
 
@@ -139,9 +139,15 @@ HTMLWidgets.widget({
                 // Supplied width and height are based on getBoundingClientRect.
                 // Wrong eg when using revealjs and slide is scaled.
                 // Spurious events also happen if we are full screen.
-                if (!trustSize() || tour.fullscreen || document.fullscreenElement)
+                if (tour.fullscreen || document.fullscreenElement)
                     return;
 
+                // May as will just never trust the width and height we are given.
+                //if (!trustSize()) {
+                width = el.offsetWidth;
+                height = el.offsetHeight;
+                //}
+
                 tour.resize(width, height);
             }
         };