mirror of
https://github.com/mozilla/gecko-dev.git
synced 2024-10-14 13:55:43 +00:00
Fix 22184: F9 should toggle sidebar visibility. r=jag a=ben
This commit is contained in:
parent
ef83335246
commit
a3051c83c3
@ -825,6 +825,7 @@ function SidebarShowHide() {
|
||||
var sidebar_panels_splitter = document.getElementById('sidebar-panels-splitter');
|
||||
var sidebar_panels_splitter_box = document.getElementById('sidebar-panels-splitter-box');
|
||||
var sidebar_splitter = document.getElementById('sidebar-splitter');
|
||||
var sidebar_menu_item = document.getElementById('sidebar-menu');
|
||||
|
||||
if (sidebar_is_hidden()) {
|
||||
debug("Showing the sidebar");
|
||||
@ -837,6 +838,7 @@ function SidebarShowHide() {
|
||||
sidebar_panels_splitter.removeAttribute('hidden');
|
||||
}
|
||||
sidebar_overlay_init();
|
||||
sidebar_menu_item.setAttribute('checked', 'true');
|
||||
} else {
|
||||
debug("Hiding the sidebar");
|
||||
var hide_everything = sidebar_panels_splitter.getAttribute('hidden') == 'true';
|
||||
@ -849,6 +851,7 @@ function SidebarShowHide() {
|
||||
}
|
||||
title_box.setAttribute('hidden', 'true');
|
||||
sidebar_panels_splitter_box.setAttribute('hidden', 'true');
|
||||
sidebar_menu_item.setAttribute('checked', 'false');
|
||||
}
|
||||
// Immediately save persistent values
|
||||
document.persist('sidebar-title-box', 'hidden');
|
||||
|
@ -26,6 +26,12 @@
|
||||
xmlns:html="http://www.w3.org/1999/xhtml"
|
||||
xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul">
|
||||
|
||||
<broadcasterset id="broadcasterset">
|
||||
<broadcaster id="toggleSidebar" oncommand="SidebarShowHide();"/>
|
||||
</broadcasterset>
|
||||
<keyset id="keyset">
|
||||
<key id="showHideSidebar" keycode="VK_F9" observes="toggleSidebar"/>
|
||||
</keyset>
|
||||
<!-- Overlay the sidebar panels -->
|
||||
<box id="sidebar-box" orient="vertical" persist="hidden width collapsed">
|
||||
<splitter id="sidebar-panels-splitter" collapse="after"
|
||||
@ -148,7 +154,8 @@
|
||||
<menuitem id="sidebar-menu" type="checkbox"
|
||||
value="&sidebarCmd.label;"
|
||||
accesskey="&sidebarCmd.accesskey;"
|
||||
oncommand="SidebarShowHide();"
|
||||
observes="toggleSidebar"
|
||||
key="showHideSidebar"
|
||||
position="2"/>
|
||||
</menupopup>
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user