corysanin/tubestation
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Bug 1897411 - Add keyboard shortcut for expanding/collapsing the new sidebar r=sidebar-reviewers,fluent-reviewers,bolsson,sclements

Browse Source 
Differential Revision: https://phabricator.services.mozilla.com/D229405
This commit is contained in:
Kelly Cochrane
2024-12-09 21:55:33 +00:00
parent 48543b66d6
commit 5750652024
7 changed files with 139 additions and 17 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
browser/components/sidebar/browser-sidebar.js
View File

@@ -381,6 +381,9 @@ var SidebarController = {
this.recordVisibilitySetting();
this.recordPositionSetting();
this.recordTabsLayoutSetting();
let sidebarToggleKey = document.getElementById("toggleSidebarKb");
const shortcut = ShortcutUtils.prettifyShortcut(sidebarToggleKey);
this.toolbarButton.dataset.l10nArgs = JSON.stringify({ shortcut });
} else {
this._switcherCloseButton = document.getElementById("sidebar-close");
if (!this._switcherListenersAdded) {
@@ -1097,15 +1100,15 @@ var SidebarController = {
// Toolbar button controls expanded state.
toolbarButton.checked = this.sidebarMain.expanded;
toolbarButton.dataset.l10nId = toolbarButton.checked
? "sidebar-widget-collapse-sidebar"
: "sidebar-widget-expand-sidebar";
? "sidebar-widget-collapse-sidebar2"
: "sidebar-widget-expand-sidebar2";
break;
case "hide-sidebar":
// Toolbar button controls hidden state.
toolbarButton.checked = !this.sidebarContainer.hidden;
toolbarButton.dataset.l10nId = toolbarButton.checked
? "sidebar-widget-hide-sidebar"
: "sidebar-widget-show-sidebar";
? "sidebar-widget-hide-sidebar2"
: "sidebar-widget-show-sidebar2";
break;
}
},