From 9ba15fd679802990665df7c8f7304f1e1c259695 Mon Sep 17 00:00:00 2001
From: "google-labs-jules[bot]"
 <161369871+google-labs-jules[bot]@users.noreply.github.com>
Date: Sat, 22 Aug 2026 00:45:41 +0000
Subject: [PATCH] =?UTF-8?q?=E2=9A=A1=20optimize=20feedback=20button=20quer?=
 =?UTF-8?q?y=20in=20sidepanel?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Scope `querySelectorAll('.feedback-btn')` to the `container` element instead of the entire `document` within the `renderResults` function. This prevents re-scanning the entire page DOM every time results are rendered. Added `node_modules` to `.gitignore`.

Co-authored-by: savvides <1580637+savvides@users.noreply.github.com>
---
 .gitignore                       | 1 +
 extension/sidepanel/sidepanel.js | 2 +-
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/.gitignore b/.gitignore
index 9ba7a56..ad84674 100644
--- a/.gitignore
+++ b/.gitignore
@@ -12,3 +12,4 @@ _site/
 build/
 dist/
 *.zip
+node_modules
diff --git a/extension/sidepanel/sidepanel.js b/extension/sidepanel/sidepanel.js
index 7b383c9..759c5c0 100644
--- a/extension/sidepanel/sidepanel.js
+++ b/extension/sidepanel/sidepanel.js
@@ -173,7 +173,7 @@ export function renderResults(data) {
     });
   }
 
-  document.querySelectorAll('.feedback-btn').forEach(btn => {
+  container.querySelectorAll('.feedback-btn').forEach(btn => {
     btn.addEventListener('click', (e) => {
       const parent = e.target.parentElement;
       if (parent) {