Commit bd128dd5 authored by henry's avatar henry Committed by morgan
Browse files

fixup! BB 43902: Modify the new sidebar for Base Browser. 

BB 45169: Allow the history item in the sidebar, but hide it by default.
parent 3fe797bc

browser/components/sidebar/SidebarManager.sys.mjs

+5 −1
Original line number Diff line number Diff line
@@ -15,7 +15,11 @@ const PINNED_PROMO_PREF = "sidebar.verticalTabs.dragToPinPromo.dismissed"; 
// New panels that are ready to be introduced to new sidebar users should be added to this list;

// ensure your feature flag is enabled at the same time you do this and that its the same value as

// what you added to .

const DEFAULT_LAUNCHER_TOOLS = "aichat,syncedtabs,history,bookmarks,opentabs";

// By default, we only include "bookmarks". Most options are non-functional in

// Tor Browser. For "history", by default the user will not have any history nor

// will they accumulate any in permanent private browsing. Though, a user may

// still customize the sidebar to add "history" back. tor-browser#45169.

const DEFAULT_LAUNCHER_TOOLS = "bookmarks";

const lazy = {};

ChromeUtils.defineESModuleGetters(lazy, {

  BrowserWindowTracker: "resource:///modules/BrowserWindowTracker.sys.mjs",

browser/components/sidebar/browser-sidebar.js

+0 −3
Original line number Diff line number Diff line
@@ -140,9 +140,6 @@ var SidebarController = { 
          gleanEvent: Glean.history.sidebarToggle,

          gleanClickEvent: Glean.sidebar.historyIconClick,

          recordSidebarVersion: true,

          // In permanent private browsing, the history panel can be opened, but

          // we hide the sidebar button to control this. tor-browser#43902.

          visible: !PrivateBrowsingUtils.permanentPrivateBrowsing,

        }),

      ],

      [