Merge pull request #1948 from bh462007/fix/giscus-contentwindow-null-check

sanjay-kv · web-flow · commit 5acfba4bd543 · 2026-07-10T09:50:57.000+10:00
fix: guard against undefined contentWindow in GiscusComments
diff --git a/src/components/giscus.tsx b/src/components/giscus.tsx
@@ -130,7 +130,7 @@ const GiscusComments: React.FC<GiscusCommentsProps> = ({
       "iframe.giscus-frame",
     );
 
-    if (!iframe) {
+    if (!iframe || !iframe.contentWindow) {
       return;
     }
 

Original file line number	Diff line number	Diff line change
`@@ -130,7 +130,7 @@ const GiscusComments: React.FC<GiscusCommentsProps> = ({`
`130`	`130`	`"iframe.giscus-frame",`
`131`	`131`	`);`
`132`	`132`
`133`		`- if (!iframe) {`
	`133`	`+ if (!iframe \|\| !iframe.contentWindow) {`
`134`	`134`	`return;`
`135`	`135`	`}`
`136`	`136`