Utilized sendMsgToActiveTab() in openFeedbackModalOrReviewPage()

adamlui · adamlui · commit e15b3774a196 · 2025-10-13T21:21:24.000-07:00
diff --git a/chatgpt-auto-continue/chromium/extension/popup/controller.js b/chatgpt-auto-continue/chromium/extension/popup/controller.js
@@ -164,9 +164,7 @@
     }
 
     function openFeedbackModalOrReviewPage() {
-        if (env.onMatchedPage)
-            chrome.tabs.query({ active: true, currentWindow: true }, tabs =>
-                chrome.tabs.sendMessage(tabs[0].id, { action: 'showFeedback' }))
+        if (env.onMatchedPage) sendMsgToActiveTab('showFeedback')
         else open(app.urls.review[app.sourceWebStore])
         close()
     }
diff --git a/chatgpt-auto-continue/firefox/extension/popup/controller.js b/chatgpt-auto-continue/firefox/extension/popup/controller.js
@@ -164,9 +164,7 @@
     }
 
     function openFeedbackModalOrReviewPage() {
-        if (env.onMatchedPage)
-            chrome.tabs.query({ active: true, currentWindow: true }, tabs =>
-                chrome.tabs.sendMessage(tabs[0].id, { action: 'showFeedback' }))
+        if (env.onMatchedPage) sendMsgToActiveTab('showFeedback')
         else open(app.urls.review[app.sourceWebStore])
         close()
     }
diff --git a/chatgpt-infinity/chromium/extension/popup/controller.js b/chatgpt-infinity/chromium/extension/popup/controller.js
@@ -206,9 +206,7 @@
     }
 
     function openFeedbackModalOrReviewPage() {
-        if (env.onMatchedPage)
-            chrome.tabs.query({ active: true, currentWindow: true }, tabs =>
-                chrome.tabs.sendMessage(tabs[0].id, { action: 'showFeedback' }))
+        if (env.onMatchedPage) sendMsgToActiveTab('showFeedback')
         else open(app.urls.review[app.sourceWebStore])
         close()
     }
diff --git a/chatgpt-infinity/firefox/extension/popup/controller.js b/chatgpt-infinity/firefox/extension/popup/controller.js
@@ -207,9 +207,7 @@
     }
 
     function openFeedbackModalOrReviewPage() {
-        if (env.onMatchedPage)
-            chrome.tabs.query({ active: true, currentWindow: true }, tabs =>
-                chrome.tabs.sendMessage(tabs[0].id, { action: 'showFeedback' }))
+        if (env.onMatchedPage) sendMsgToActiveTab('showFeedback')
         else open(app.urls.review[app.sourceWebStore])
         close()
     }
diff --git a/chatgpt-widescreen/chromium/extension/popup/controller.js b/chatgpt-widescreen/chromium/extension/popup/controller.js
@@ -171,9 +171,7 @@
     }
 
     function openFeedbackModalOrReviewPage() {
-        if (env.onMatchedPage)
-            chrome.tabs.query({ active: true, currentWindow: true }, tabs =>
-                chrome.tabs.sendMessage(tabs[0].id, { action: 'showFeedback' }))
+        if (env.onMatchedPage) sendMsgToActiveTab('showFeedback')
         else open(app.urls.review[app.sourceWebStore])
         close()
     }
diff --git a/chatgpt-widescreen/firefox/extension/popup/controller.js b/chatgpt-widescreen/firefox/extension/popup/controller.js
@@ -171,9 +171,7 @@
     }
 
     function openFeedbackModalOrReviewPage() {
-        if (env.onMatchedPage)
-            chrome.tabs.query({ active: true, currentWindow: true }, tabs =>
-                chrome.tabs.sendMessage(tabs[0].id, { action: 'showFeedback' }))
+        if (env.onMatchedPage) sendMsgToActiveTab('showFeedback')
         else open(app.urls.review[app.sourceWebStore])
         close()
     }

Original file line number	Diff line number	Diff line change
`@@ -164,9 +164,7 @@`
`164`	`164`	`}`
`165`	`165`
`166`	`166`	`function openFeedbackModalOrReviewPage() {`
`167`		`- if (env.onMatchedPage)`
`168`		`- chrome.tabs.query({ active: true, currentWindow: true }, tabs =>`
`169`		`- chrome.tabs.sendMessage(tabs[0].id, { action: 'showFeedback' }))`
	`167`	`+ if (env.onMatchedPage) sendMsgToActiveTab('showFeedback')`
`170`	`168`	`else open(app.urls.review[app.sourceWebStore])`
`171`	`169`	`close()`
`172`	`170`	`}`
Original file line number	Diff line number	Diff line change
`@@ -206,9 +206,7 @@`
`206`	`206`	`}`
`207`	`207`
`208`	`208`	`function openFeedbackModalOrReviewPage() {`
`209`		`- if (env.onMatchedPage)`
`210`		`- chrome.tabs.query({ active: true, currentWindow: true }, tabs =>`
`211`		`- chrome.tabs.sendMessage(tabs[0].id, { action: 'showFeedback' }))`
	`209`	`+ if (env.onMatchedPage) sendMsgToActiveTab('showFeedback')`
`212`	`210`	`else open(app.urls.review[app.sourceWebStore])`
`213`	`211`	`close()`
`214`	`212`	`}`
Original file line number	Diff line number	Diff line change
`@@ -207,9 +207,7 @@`
`207`	`207`	`}`
`208`	`208`
`209`	`209`	`function openFeedbackModalOrReviewPage() {`
`210`		`- if (env.onMatchedPage)`
`211`		`- chrome.tabs.query({ active: true, currentWindow: true }, tabs =>`
`212`		`- chrome.tabs.sendMessage(tabs[0].id, { action: 'showFeedback' }))`
	`210`	`+ if (env.onMatchedPage) sendMsgToActiveTab('showFeedback')`
`213`	`211`	`else open(app.urls.review[app.sourceWebStore])`
`214`	`212`	`close()`
`215`	`213`	`}`
Original file line number	Diff line number	Diff line change
`@@ -171,9 +171,7 @@`
`171`	`171`	`}`
`172`	`172`
`173`	`173`	`function openFeedbackModalOrReviewPage() {`
`174`		`- if (env.onMatchedPage)`
`175`		`- chrome.tabs.query({ active: true, currentWindow: true }, tabs =>`
`176`		`- chrome.tabs.sendMessage(tabs[0].id, { action: 'showFeedback' }))`
	`174`	`+ if (env.onMatchedPage) sendMsgToActiveTab('showFeedback')`
`177`	`175`	`else open(app.urls.review[app.sourceWebStore])`
`178`	`176`	`close()`
`179`	`177`	`}`