From fe94c10a8cd567c798f2de642aaca97c2449eaa7 Mon Sep 17 00:00:00 2001
From: DovydasGrb1 <dovydasgrabauskas@gmail.com>
Date: Wed, 9 Apr 2025 13:32:26 +0200
Subject: [PATCH 1/2] Fix issue #308: added functionality where a user can
 select whether he wants to see random tips

---
 src/popup/qrcode.html |  4 +++
 src/popup/qrcode.js   | 70 ++++++++++++++++++++++++++++++++-----------
 2 files changed, 56 insertions(+), 18 deletions(-)

diff --git a/src/popup/qrcode.html b/src/popup/qrcode.html
index b8f80503..b6ca1f0a 100644
--- a/src/popup/qrcode.html
+++ b/src/popup/qrcode.html
@@ -47,6 +47,10 @@
 				</div>
 			</div>
 		</div>
+		<label>
+			<input type="checkbox" id="disable-tips-checkbox">
+			Disable random tips
+		  </label>		  
 		<textarea id="qrcodetext"
 					class="browser-style"
 					placeholder="Enter text for QR code here to generate it."
diff --git a/src/popup/qrcode.js b/src/popup/qrcode.js
index d7cbc811..12afc027 100644
--- a/src/popup/qrcode.js
+++ b/src/popup/qrcode.js
@@ -1,18 +1,52 @@
-/**
- * Starter module for QR code popup.
- *
- * @module qrcode
- * @requires modules/RandomTips
- * @requires modules/InitQrCode
- */
-"use strict";
-
-import { tips } from "/common/modules/data/Tips.js";
-import * as RandomTips from "/common/modules/RandomTips/RandomTips.js";
-
-import "./modules/InitQrCode.js";
-
-RandomTips.init(tips).then(() => {
-    RandomTips.setContext("popup");
-    RandomTips.showRandomTipIfWanted();
-});
+// /**
+//  * Starter module for QR code popup.
+//  *
+//  * @module qrcode
+//  * @requires modules/RandomTips
+//  * @requires modules/InitQrCode
+//  */
+// "use strict";
+
+// import { tips } from "/common/modules/data/Tips.js";
+// import * as RandomTips from "/common/modules/RandomTips/RandomTips.js";
+
+// import "./modules/InitQrCode.js";
+
+const CHECKBOX_ID = "disable-tips-checkbox";
+
+/*Check browser's storage if user wants to receive the tips (default = true)*/
+async function ShowTips() {
+    const result = await browser.storage.local.get("showTips");
+    return result.showTips !== false;
+}
+
+// Save user preference when checkbox is changed
+function setupCheckbox() {
+    const checkbox = document.getElementById(CHECKBOX_ID);
+    if (!checkbox) return;
+
+    // Load initial state
+    browser.storage.local.get("showTips").then((result) => {
+        checkbox.checked = result.showTips === false;
+    });
+
+    // Listen for changes
+    checkbox.addEventListener("change", () => {
+        const show = !checkbox.checked;
+        browser.storage.local.set({ showTips: show });
+    });
+}
+
+async function init() 
+{
+    setupCheckbox();
+    if (await shouldShowTips()) 
+    {
+        RandomTips.init(tips).then(() => {
+            RandomTips.setContext("popup");
+            RandomTips.showRandomTipIfWanted();
+        });
+    }
+}
+
+init();
\ No newline at end of file

From bb6b8fa2194409d510fdc7bf3fb366471da775d1 Mon Sep 17 00:00:00 2001
From: DovydasGrb1 <dovydasgrabauskas@gmail.com>
Date: Wed, 9 Apr 2025 16:21:08 +0200
Subject: [PATCH 2/2] Fix: uncommented imports

---
 src/popup/qrcode.js | 35 +++++++++++++++++------------------
 1 file changed, 17 insertions(+), 18 deletions(-)

diff --git a/src/popup/qrcode.js b/src/popup/qrcode.js
index 12afc027..29f5bae6 100644
--- a/src/popup/qrcode.js
+++ b/src/popup/qrcode.js
@@ -1,36 +1,35 @@
-// /**
-//  * Starter module for QR code popup.
-//  *
-//  * @module qrcode
-//  * @requires modules/RandomTips
-//  * @requires modules/InitQrCode
-//  */
-// "use strict";
+/**
+ * Starter module for QR code popup.
+ *
+ * @module qrcode
+ * @requires modules/RandomTips
+ * @requires modules/InitQrCode
+ */
+"use strict";
 
-// import { tips } from "/common/modules/data/Tips.js";
-// import * as RandomTips from "/common/modules/RandomTips/RandomTips.js";
+import { tips } from "/common/modules/data/Tips.js";
+import * as RandomTips from "/common/modules/RandomTips/RandomTips.js";
 
-// import "./modules/InitQrCode.js";
+import "./modules/InitQrCode.js";
 
 const CHECKBOX_ID = "disable-tips-checkbox";
 
-/*Check browser's storage if user wants to receive the tips (default = true)*/
+/*Check browser's storage if user wants to receive the tips*/
 async function ShowTips() {
     const result = await browser.storage.local.get("showTips");
     return result.showTips !== false;
 }
 
-// Save user preference when checkbox is changed
-function setupCheckbox() {
+/*Functionality for checkbox changes */
+function SetupCheckbox() {
     const checkbox = document.getElementById(CHECKBOX_ID);
     if (!checkbox) return;
 
-    // Load initial state
+    /*Load initial state whether users wants to see tips */
     browser.storage.local.get("showTips").then((result) => {
         checkbox.checked = result.showTips === false;
     });
 
-    // Listen for changes
     checkbox.addEventListener("change", () => {
         const show = !checkbox.checked;
         browser.storage.local.set({ showTips: show });
@@ -39,8 +38,8 @@ function setupCheckbox() {
 
 async function init() 
 {
-    setupCheckbox();
-    if (await shouldShowTips()) 
+    SetupCheckbox();
+    if (await ShowTips()) 
     {
         RandomTips.init(tips).then(() => {
             RandomTips.setContext("popup");