Magic-Mirror/gecko-dev
1
0
Fork 0
mirror of https://github.com/mozilla/gecko-dev.git synced 2024-10-09 19:35:51 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Make View->Sidebar a type=checkbox menuitem (r=slamm)

Browse Source
This commit is contained in:
shaver%netscape.com 1999-10-15 21:37:53 +00:00
parent ba5179289d
commit c09d70aace
6 changed files with 22 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
suite/browser/navigatorOverlay.xul
View File

@ -204,7 +204,9 @@ Contributor(s): ______________________________________. -->
</menupopup>
</menu>
<menuitem value="&sidebarCmd.label;" oncommand="sidebarShowHide();"/>
<!-- shaver asks: shouldn't this be in sidebarOverlay? -->
<menuitem id="menu_sidebar" type="checkbox" value="&sidebarCmd.label;"
oncommand="sidebarShowHide();"/>
<menuseparator />
<menuitem value="&enlargeTextSizeCmd.label;"
oncommand="BrowserReload();"/>

3
suite/common/sidebar/sidebarOverlay.js
View File

@ -76,10 +76,13 @@ function sidebarOverlayInit(usersidebar)
}
var sidebar_element = document.getElementById('sidebar-box')
var sidebar_menuitem = document.getElementById('menu_sidebar')
if (sidebar_element.getAttribute('hidden') == 'true') {
sidebar_element.setAttribute('style', 'display:none')
sidebar_menuitem.setAttribute('checked', 'false')
return
}
sidebar_menuitem.setAttribute('checked', 'true')
//dump("sidebar = " + sidebar + "\n")
//dump("sidebar.resource = " + sidebar.resource + "\n")

9
suite/common/sidebar/sidebarOverlay.xul
View File

@ -6,10 +6,6 @@
xmlns:html="http://www.w3.org/TR/REC-html40"
xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul">
<html:script language="JavaScript"
src="chrome://sidebar/content/sidebarOverlay.js"/>
<html:script language="JavaScript" src="chrome://sidebar/content/flash.js"/>
<!-- Overlay the sidebar panels -->
<box id="sidebar-box" align="vertical" persist="hidden width">
<splitter id="title-box" collapse="after"
@ -81,5 +77,10 @@
<!-- Some splitter attritutes -->
<splitter id="sidebar-splitter" collapse="before" persist="state hidden"
chromeclass="extrachrome" onclick="sidebarSaveState(this)"/>
<!-- Scripts go last, because they peek at state to tweak menus -->
<html:script language="JavaScript"
src="chrome://sidebar/content/sidebarOverlay.js"/>
<html:script language="JavaScript" src="chrome://sidebar/content/flash.js"/>
</overlay>

4
xpfe/browser/resources/content/navigatorOverlay.xul
View File

@ -204,7 +204,9 @@ Contributor(s): ______________________________________. -->
</menupopup>
</menu>
<menuitem value="&sidebarCmd.label;" oncommand="sidebarShowHide();"/>
<!-- shaver asks: shouldn't this be in sidebarOverlay? -->
<menuitem id="menu_sidebar" type="checkbox" value="&sidebarCmd.label;"
oncommand="sidebarShowHide();"/>
<menuseparator />
<menuitem value="&enlargeTextSizeCmd.label;"
oncommand="BrowserReload();"/>

3
xpfe/components/sidebar/resources/sidebarOverlay.js
View File

@ -76,10 +76,13 @@ function sidebarOverlayInit(usersidebar)
}
var sidebar_element = document.getElementById('sidebar-box')
var sidebar_menuitem = document.getElementById('menu_sidebar')
if (sidebar_element.getAttribute('hidden') == 'true') {
sidebar_element.setAttribute('style', 'display:none')
sidebar_menuitem.setAttribute('checked', 'false')
return
}
sidebar_menuitem.setAttribute('checked', 'true')
//dump("sidebar = " + sidebar + "\n")
//dump("sidebar.resource = " + sidebar.resource + "\n")

9
xpfe/components/sidebar/resources/sidebarOverlay.xul
View File

@ -6,10 +6,6 @@
xmlns:html="http://www.w3.org/TR/REC-html40"
xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul">
<html:script language="JavaScript"
src="chrome://sidebar/content/sidebarOverlay.js"/>
<html:script language="JavaScript" src="chrome://sidebar/content/flash.js"/>
<!-- Overlay the sidebar panels -->
<box id="sidebar-box" align="vertical" persist="hidden width">
<splitter id="title-box" collapse="after"
@ -81,5 +77,10 @@
<!-- Some splitter attritutes -->
<splitter id="sidebar-splitter" collapse="before" persist="state hidden"
chromeclass="extrachrome" onclick="sidebarSaveState(this)"/>
<!-- Scripts go last, because they peek at state to tweak menus -->
<html:script language="JavaScript"
src="chrome://sidebar/content/sidebarOverlay.js"/>
<html:script language="JavaScript" src="chrome://sidebar/content/flash.js"/>
</overlay>