[tor-commits] [Git][tpo/applications/tor-browser][base-browser-115.3.1esr-13.0-1] fixup! Bug 32308: use direct browser sizing for letterboxing. Bug 30556: align...

[ma1 (@ma1)]

ma1 pushed to branch base-browser-115.3.1esr-13.0-1 at The Tor Project / Applications / Tor Browser


Commits:
4ff109fa by hackademix at 2023-10-09T20:47:34+02:00
fixup! Bug 32308: use direct browser sizing for letterboxing. Bug 30556: align letterboxing with 200x100 new win width stepping

Bug 41259: Do not interfere with devtools responsive mode sizing.

- - - - -


2 changed files:

- browser/base/content/browser.css
- toolkit/components/resistfingerprinting/RFPHelper.sys.mjs


Changes:

=====================================
browser/base/content/browser.css
=====================================
@@ -140,7 +140,7 @@ body {
   Never modify the following selector without synchronizing
   LETTERBOX_CSS_SELECTOR in RFPHelper.jsm!
 **/
-.letterboxing .browserStack:not(.exclude-letterboxing) > browser {
+.letterboxing .browserContainer:not(.responsive-mode) > .browserStack:not(.exclude-letterboxing) > browser {
   /* width & height to be dynamically set by RFPHelper.jsm */
   outline: 1px solid var(--chrome-content-separator-color);
 }
@@ -149,7 +149,7 @@ body {
   outline: initial;
 }
 
-:root:not([inDOMFullscreen]) .letterboxing.letterboxing-ready .browserStack:not(.exclude-letterboxing) {
+:root:not([inDOMFullscreen]) .letterboxing.letterboxing-ready .browserContainer:not(.responsive-mode) > .browserStack:not(.exclude-letterboxing) {
   place-content: start center;
 }
 


=====================================
toolkit/components/resistfingerprinting/RFPHelper.sys.mjs
=====================================
@@ -437,7 +437,7 @@ class _RFPHelper {
       // preemptively in order to beat race conditions on tab/window creation
       const LETTERBOX_CSS_URL = "chrome://browser/content/browser.css";
       const LETTERBOX_CSS_SELECTOR =
-        ".letterboxing .browserStack:not(.exclude-letterboxing) > browser";
+        ".letterboxing .browserContainer:not(.responsive-mode) > .browserStack:not(.exclude-letterboxing) > browser";
       for (let ss of document.styleSheets) {
         if (ss.href !== LETTERBOX_CSS_URL) {
           continue;
@@ -557,8 +557,8 @@ class _RFPHelper {
 
     const roundDimensions = (aWidth, aHeight) => {
       const r = (aWidth, aHeight) => ({
-        width: `${aWidth}px`,
-        height: `${aHeight}px`,
+        width: `var(--rdm-width, ${aWidth}px)`,
+        height: `var(--rdm-height, ${aHeight}px)`,
       });
 
       let result;



View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser/-/commit/4ff109faf88bd2e22d800fb719a36e1327f3301b

-- 
View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser/-/commit/4ff109faf88bd2e22d800fb719a36e1327f3301b
You're receiving this email because of your account on gitlab.torproject.org.


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.torproject.org/pipermail/tor-commits/attachments/20231009/b2b64ebb/attachment-0001.htm>