Revert "Scroll height fix"

Rajat · Rajat · commit 38f3edd66f26 · 2026-02-08T16:15:40.000+05:30
This reverts commit e385f39.
diff --git a/packages/page-blocks/src/blocks/embed/widget.tsx b/packages/page-blocks/src/blocks/embed/widget.tsx
@@ -7,6 +7,7 @@ import { Section } from "@courselit/page-primitives";
 import { SandboxedEmbed } from "../../components";
 
 export default function Widget({
+    id,
     settings: {
         contentType,
         content,
@@ -29,10 +30,8 @@ export default function Widget({
 
     const formattedHeight = `${height}px`;
 
-    // Check if content is "script" type but only contains an iframe (no actual scripts)
-    const hasScript = contentType === "script" && /<script[\s>]/i.test(content);
-    const isPureIframe =
-        contentType === "script" && !hasScript && /<iframe[\s>]/i.test(content);
+    // Check if content is "script" or "iframe" (not a direct URL)
+    const isEmbedCode = contentType === "script";
 
     const containerStyle =
         aspectRatio && aspectRatio !== "default"
@@ -41,7 +40,7 @@ export default function Widget({
                   width: "100%",
                   paddingTop: `calc(100% / (${aspectRatio.split(":")[0]} / ${aspectRatio.split(":")[1]}))`,
               } as React.CSSProperties)
-            : hasScript
+            : isEmbedCode
               ? ({} as React.CSSProperties)
               : ({ height: formattedHeight } as React.CSSProperties);
 
@@ -54,29 +53,21 @@ export default function Widget({
                   width: "100%",
                   height: "100%",
               } as React.CSSProperties)
-            : hasScript
+            : isEmbedCode
               ? ({} as React.CSSProperties)
               : ({ height: "100%" } as React.CSSProperties);
 
     const renderContent = () => {
-        if (hasScript) {
-            // Content has actual script tags - use sandboxed embed
+        if (isEmbedCode) {
+            // Content is a script or iframe code - use sandboxed embed for dynamic height
             return (
                 <SandboxedEmbed
+                    id={id}
                     content={content}
                     className="w-full"
                     style={iframeStyle}
                 />
             );
-        } else if (isPureIframe) {
-            // Content is a pure iframe without scripts - render directly
-            return (
-                <div
-                    className="w-full"
-                    style={iframeStyle}
-                    dangerouslySetInnerHTML={{ __html: content }}
-                />
-            );
         } else {
             // URL-based content
             return (
diff --git a/packages/page-blocks/src/components/sandboxed-embed.tsx b/packages/page-blocks/src/components/sandboxed-embed.tsx
@@ -38,8 +38,8 @@ export const SandboxedEmbed = ({
         <html>
         <head>
             <style>
-                html, body { margin: 0; padding: 0; overflow: hidden; }
-                #content-wrapper { overflow: hidden; height: auto; }
+                html, body { margin: 0; padding: 0; height: auto; min-height: 100%; }
+                #content-wrapper { overflow: auto; height: auto; }
             </style>
         </head>
         <body>
@@ -91,7 +91,6 @@ export const SandboxedEmbed = ({
                 width: "100%",
                 height: height,
                 border: "none",
-                overflow: "hidden",
                 ...style,
             }}
             sandbox="allow-scripts allow-popups allow-forms allow-same-origin"
