Commit d7620dd8 authored by henry's avatar henry Committed by Pier Angelo Vendrame
Browse files

fixup! BB 32308: Use direct browser sizing for letterboxing. 

TB 44214: Move the letterboxing classes one element up from tabpanels to
tabbox.

This is because we need to restyle the tabbox.
parent 1a262a27

toolkit/components/resistfingerprinting/RFPHelper.sys.mjs

+4 −4
Original line number Diff line number Diff line
@@ -605,8 +605,8 @@ class _RFPHelper { 
  _updateSizeForTabsInWindow(aWindow) {

    let tabBrowser = aWindow.gBrowser;



    tabBrowser.tabpanels?.classList.add("letterboxing");

    tabBrowser.tabpanels?.classList.toggle(

    tabBrowser.tabbox.classList.add("letterboxing");

    tabBrowser.tabbox.classList.toggle(

      "letterboxing-vcenter",

      Services.prefs.getBoolPref(kPrefLetterboxingVcenter, false)

    );
@@ -618,7 +618,7 @@ class _RFPHelper { 
    // We need to add this class late because otherwise new windows get

    // maximized.

    aWindow.setTimeout(() => {

      tabBrowser.tabpanels?.classList.add("letterboxing-ready");

      tabBrowser.tabbox.classList.add("letterboxing-ready");

    });

  }
@@ -669,7 +669,7 @@ class _RFPHelper { 
    aWindow.removeEventListener("TabOpen", this);



    // revert tabpanel's style to default

    tabBrowser.tabpanels?.classList.remove("letterboxing");

    tabBrowser.tabbox.classList.remove("letterboxing");



    // and restore default size on each browser element

    for (let tab of tabBrowser.tabs) {

toolkit/components/resistfingerprinting/content/letterboxing.css

+1 −1
Original line number Diff line number Diff line
@@ -17,7 +17,7 @@ 
  }

}



.letterboxing {

#tabbrowser-tabbox.letterboxing {

  --letterboxing-bgcolor: var(--tabpanel-background-color);

  --letterboxing-border-radius: 8px;

  --letterboxing-border-top-radius: 0;