Refactor browser-dependent UI CSS.

hackademix · hackademix · commit daa69b22c216 · 2024-10-19T09:44:13.000+02:00
diff --git a/src/nscl b/src/nscl
@@ -1 +1 @@
-Subproject commit 095d75584b9c19203bada0b09d6b05e5f1f15edb
+Subproject commit 6190ad1d4288385aa1062af35fd62807c2f6ab2d
diff --git a/src/ui/ui.js b/src/ui/ui.js
@@ -46,11 +46,9 @@ var UI = (() => {
         "/nscl/common/Permissions.js",
         "/nscl/common/Policy.js",
       ];
+
       this.mobile = UA.mobile;
-      let root = document.documentElement;
-      if (this.mobile) {
-        root.classList.add("mobile");
-      }
+
       await include(scripts);
 
       let inited = new Promise(resolve => {
@@ -71,6 +69,8 @@ var UI = (() => {
                 debug = () => {}; // be quiet!
               }
               if (UI.local.isTorBrowser) {
+                Sites.onionSecure = true;
+                // TODO: replace the following with pure CSS :content in common.css
                 const label = document.querySelector("span.tor");
                 if (label) {
                   const browserName = (await browser.runtime.getBrowserInfo()).name;
@@ -80,8 +80,6 @@ var UI = (() => {
                     );
                   }
                 }
-                document.documentElement.classList.add("tor");
-                Sites.onionSecure = true;
               }
             }
             resolve();

Original file line number	Diff line number	Diff line change
`@@ -46,11 +46,9 @@ var UI = (() => {`
`46`	`46`	`"/nscl/common/Permissions.js",`
`47`	`47`	`"/nscl/common/Policy.js",`
`48`	`48`	`];`
	`49`	`+`
`49`	`50`	`this.mobile = UA.mobile;`
`50`		`- let root = document.documentElement;`
`51`		`- if (this.mobile) {`
`52`		`- root.classList.add("mobile");`
`53`		`- }`
	`51`	`+`
`54`	`52`	`await include(scripts);`
`55`	`53`
`56`	`54`	`let inited = new Promise(resolve => {`
`@@ -71,6 +69,8 @@ var UI = (() => {`
`71`	`69`	`debug = () => {}; // be quiet!`
`72`	`70`	`}`
`73`	`71`	`if (UI.local.isTorBrowser) {`
	`72`	`+ Sites.onionSecure = true;`
	`73`	`+ // TODO: replace the following with pure CSS :content in common.css`
`74`	`74`	`const label = document.querySelector("span.tor");`
`75`	`75`	`if (label) {`
`76`	`76`	`const browserName = (await browser.runtime.getBrowserInfo()).name;`
`@@ -80,8 +80,6 @@ var UI = (() => {`
`80`	`80`	`);`
`81`	`81`	`}`
`82`	`82`	`}`
`83`		`- document.documentElement.classList.add("tor");`
`84`		`- Sites.onionSecure = true;`
`85`	`83`	`}`
`86`	`84`	`}`
`87`	`85`	`resolve();`