From 6afbe6ac92c835eaeeb28190f07f9fa5092c9672 Mon Sep 17 00:00:00 2001
From: Kyryl <91323838+kirizdev@users.noreply.github.com>
Date: Mon, 13 Feb 2023 18:46:42 +0000
Subject: Refactor createList function

---
 src/pages/options/index.js | 125 +++++++++++++++++++++++----------------------
 1 file changed, 63 insertions(+), 62 deletions(-)

(limited to 'src')

diff --git a/src/pages/options/index.js b/src/pages/options/index.js
index f122f3fc..3f37d829 100644
--- a/src/pages/options/index.js
+++ b/src/pages/options/index.js
@@ -181,68 +181,69 @@ async function processCustomInstances(frontend, document) {
 }
 
 function createList(frontend, networks, document, redirects, blacklist) {
-	for (const network in networks) {
-		if (redirects[frontend]) {
-			if (redirects[frontend][network].length > 0) {
-				document.getElementById(frontend).getElementsByClassName("custom-instance")[0].placeholder = redirects[frontend].clearnet[0]
-				document.getElementById(frontend)
-					.getElementsByClassName(network)[0]
-					.getElementsByClassName("checklist")[0]
-					.innerHTML = [
-						`<div class="some-block option-block">
-						<h4>${utils.camelCase(network)}</h4>
-					</div>`,
-						...redirects[frontend][network]
-							.sort((a, b) =>
-								(blacklist.cloudflare.includes(a) && !blacklist.cloudflare.includes(b))
-							)
-							.map(x => {
-								const cloudflare = blacklist.cloudflare.includes(x) ?
-									` <a target="_blank" href="https://libredirect.github.io/docs.html#instances">
-							 		<span style="color:red;">cloudflare</span>
-								</a>` : ""
-
-								const warnings = [cloudflare].join(" ")
-								return `<div>
-										<x>
-											<a href="${x}" target="_blank">${x}</a>${warnings}
-										</x>
-										<button class="add add-${x}">
-											<svg xmlns="https://www.w3.org/2000/svg" height="20px" viewBox="0 0 24 24" width="20px" fill="currentColor">
-												<path d="M19 13h-6v6h-2v-6H5v-2h6V5h2v6h6v2z" />
-											</svg>
-										</button>
-						  			</div>`
-							}),
-						'<br>'
-					].join("\n<hr>\n")
-
-				for (const x of redirects[frontend][network]) {
-					document.getElementById(frontend)
-						.getElementsByClassName(network)[0]
-						.getElementsByClassName("checklist")[0]
-						.getElementsByClassName(`add-${x}`)[0]
-						.addEventListener("click", async () => {
-							let options = await utils.getOptions()
-							let customInstances = options[frontend]
-							if (!customInstances.includes(x)) {
-								customInstances.push(x)
-								options = await utils.getOptions()
-								options[frontend] = customInstances
-								browser.storage.local.set({ options }, () => {
-									calcCustomInstances(frontend)
-								})
-							}
-						})
-				}
-			}
-		} else {
-			document.getElementById(frontend).getElementsByClassName(network)[0].getElementsByClassName("checklist")[0].innerHTML =
-				`<div class="some-block option-block">No instances found.</div>`
-			break
-		}
+    for (const network in networks) {
+        const checklist = document.getElementById(frontend)
+            .getElementsByClassName(network)[0]
+            .getElementsByClassName("checklist")[0]
 
-	}
+        if (!redirects[frontend]) {
+            checklist.innerHTML = '<div class="some-block option-block">No instances found.</div>'
+            break
+        }
+        const instances = redirects[frontend][network]
+        if (!instances || instances.length === 0) continue
+
+        document.getElementById(frontend)
+            .getElementsByClassName("custom-instance")[0]
+            .placeholder = redirects[frontend].clearnet[0]
+
+        const sortedInstances = instances
+            .sort((a, b) => (blacklist.cloudflare.includes(a) && !blacklist.cloudflare.includes(b)))
+
+        const content = sortedInstances
+            .map(x => {
+                const cloudflare = blacklist.cloudflare.includes(x) ?
+                    `<a target="_blank" href="https://libredirect.github.io/docs.html#instances">
+                        <span style="color:red;">cloudflare</span>
+                    </a>` : ""
+
+                const warnings = [cloudflare].join(" ")
+                return `<div class="frontend">
+                            <x>
+                                <a href="${x}" target="_blank">${x}</a>${warnings}
+                            </x>
+                            <button class="add add-${x}">
+                                <svg xmlns="https://www.w3.org/2000/svg" height="20px" viewBox="0 0 24 24" width="20px" fill="currentColor">
+                                    <path d="M19 13h-6v6h-2v-6H5v-2h6V5h2v6h6v2z" />
+                                </svg>
+                            </button>
+                        </div>`
+            })
+
+        checklist.innerHTML = [
+            `<div class="some-block option-block">
+                <h4>${utils.camelCase(network)}</h4>
+            </div>`,
+            ...content,
+            "<br>"
+        ].join("\n<hr>\n")
+
+        for (const instance of instances) {
+            checklist.getElementsByClassName(`add-${instance}`)[0]
+                .addEventListener("click", async () => {
+                    let options = await utils.getOptions()
+                    let customInstances = options[frontend]
+                    if (!customInstances.includes(instance)) {
+                        customInstances.push(instance)
+                        options = await utils.getOptions()
+                        options[frontend] = customInstances
+                        browser.storage.local.set({options}, () => {
+                            calcCustomInstances(frontend)
+                        })
+                    }
+                })
+        }
+    }
 }
 
 const r = window.location.href.match(/#(.*)/)
@@ -288,4 +289,4 @@ async function ping(frontend) {
 		span.innerHTML = `<span style="color:${color};">${text}</span>`
 
 	}
-}
\ No newline at end of file
+}
-- 
cgit 1.4.1


From f94d79229f06e1a0befda43f6db7e9b56606fc28 Mon Sep 17 00:00:00 2001
From: Kyryl <91323838+kirizdev@users.noreply.github.com>
Date: Mon, 13 Feb 2023 18:54:40 +0000
Subject: Remove unused import

---
 src/pages/options/index.js | 1 -
 1 file changed, 1 deletion(-)

(limited to 'src')

diff --git a/src/pages/options/index.js b/src/pages/options/index.js
index 3f37d829..2cb28bd8 100644
--- a/src/pages/options/index.js
+++ b/src/pages/options/index.js
@@ -1,5 +1,4 @@
 import utils from "../../assets/javascripts/utils.js"
-import localise from "../../assets/javascripts/localise.js"
 
 let config,
 	options,
-- 
cgit 1.4.1


From 6df1197bd1489f9934970ce8d5497abfbc3f3baf Mon Sep 17 00:00:00 2001
From: ManeraKai <manerakai@protonmail.com>
Date: Tue, 14 Feb 2023 20:03:06 +0300
Subject: small tweak

---
 src/pages/options/index.js | 94 +++++++++++++++++++++++-----------------------
 1 file changed, 46 insertions(+), 48 deletions(-)

(limited to 'src')

diff --git a/src/pages/options/index.js b/src/pages/options/index.js
index 2cb28bd8..1256c9b1 100644
--- a/src/pages/options/index.js
+++ b/src/pages/options/index.js
@@ -180,34 +180,37 @@ async function processCustomInstances(frontend, document) {
 }
 
 function createList(frontend, networks, document, redirects, blacklist) {
-    for (const network in networks) {
-        const checklist = document.getElementById(frontend)
-            .getElementsByClassName(network)[0]
-            .getElementsByClassName("checklist")[0]
-
-        if (!redirects[frontend]) {
-            checklist.innerHTML = '<div class="some-block option-block">No instances found.</div>'
-            break
-        }
-        const instances = redirects[frontend][network]
-        if (!instances || instances.length === 0) continue
-
-        document.getElementById(frontend)
-            .getElementsByClassName("custom-instance")[0]
-            .placeholder = redirects[frontend].clearnet[0]
-
-        const sortedInstances = instances
-            .sort((a, b) => (blacklist.cloudflare.includes(a) && !blacklist.cloudflare.includes(b)))
-
-        const content = sortedInstances
-            .map(x => {
-                const cloudflare = blacklist.cloudflare.includes(x) ?
-                    `<a target="_blank" href="https://libredirect.github.io/docs.html#instances">
+	for (const network in networks) {
+		const checklist = document.getElementById(frontend)
+			.getElementsByClassName(network)[0]
+			.getElementsByClassName("checklist")[0]
+
+		if (!redirects[frontend]) {
+			checklist.innerHTML = '<div class="some-block option-block">No instances found.</div>'
+			break
+		}
+
+		const instances = redirects[frontend][network]
+		if (!instances || instances.length === 0) continue
+
+		document.getElementById(frontend)
+			.getElementsByClassName("custom-instance")[0]
+			.placeholder = redirects[frontend].clearnet[0]
+
+		const sortedInstances = instances
+			.sort((a, b) => {
+				return (blacklist.cloudflare.includes(a) && !blacklist.cloudflare.includes(b))
+			})
+
+		const content = sortedInstances
+			.map(x => {
+				const cloudflare = blacklist.cloudflare.includes(x) ?
+					`<a target="_blank" href="https://libredirect.github.io/docs.html#instances">
                         <span style="color:red;">cloudflare</span>
                     </a>` : ""
 
-                const warnings = [cloudflare].join(" ")
-                return `<div class="frontend">
+				const warnings = [cloudflare].join(" ")
+				return `<div class="frontend">
                             <x>
                                 <a href="${x}" target="_blank">${x}</a>${warnings}
                             </x>
@@ -217,32 +220,27 @@ function createList(frontend, networks, document, redirects, blacklist) {
                                 </svg>
                             </button>
                         </div>`
-            })
+			})
 
-        checklist.innerHTML = [
-            `<div class="some-block option-block">
+		checklist.innerHTML = [
+			`<div class="some-block option-block">
                 <h4>${utils.camelCase(network)}</h4>
             </div>`,
-            ...content,
-            "<br>"
-        ].join("\n<hr>\n")
-
-        for (const instance of instances) {
-            checklist.getElementsByClassName(`add-${instance}`)[0]
-                .addEventListener("click", async () => {
-                    let options = await utils.getOptions()
-                    let customInstances = options[frontend]
-                    if (!customInstances.includes(instance)) {
-                        customInstances.push(instance)
-                        options = await utils.getOptions()
-                        options[frontend] = customInstances
-                        browser.storage.local.set({options}, () => {
-                            calcCustomInstances(frontend)
-                        })
-                    }
-                })
-        }
-    }
+			...content,
+			"<br>"
+		].join("\n<hr>\n")
+
+		for (const instance of instances) {
+			checklist.getElementsByClassName(`add-${instance}`)[0]
+				.addEventListener("click", async () => {
+					let options = await utils.getOptions()
+					if (!options[frontend].includes(instance)) {
+						options[frontend].push(instance)
+						browser.storage.local.set({ options }, () => calcCustomInstances(frontend))
+					}
+				})
+		}
+	}
 }
 
 const r = window.location.href.match(/#(.*)/)
-- 
cgit 1.4.1