merge :sbclose and viewSidebarMenu menuitem additions

NEWS

@@ -1,7 +1,7 @@
 <pre>
 2007-XX-XX:
 	* version 0.5.2
-	* added a new :sidebar command
+	* added new :sidebar and :sbclose commands
 	* added 'more' and standard more-prompt key mappings to control
 	  behaviour of the message list pager 
 	* added 'hlsearchstyle' option to allow for user CSS styling of the

TODO

@@ -19,8 +19,6 @@ BUGS:
 
 FEATURES:
 9 :command for new commands
-9 :sidebar {addons/bookmarks/history/downloads/...} command with tab completion
-  (and automatically getting possible sidebar elements from View->sidebar
 9 Use our own find-as-you-type mechanism (like conkeror does)
 9 make hints smarter, not only with characters from from hintchars, but use "NE" or "NP" for 'new posts' e.g. (might be too slow)
 8 middleclick in content == p, and if command line is open, paste there the clipboard buffer

chrome/content/vimperator/commands.js

@@ -1076,7 +1076,27 @@ function Commands() //{{{
             completer: function(filter) { return vimperator.completion.get_options_completions(filter); }
         }
     ));
-    addDefaultCommand(new Command(["sideb[ar]"],
+    // TODO: sclose instead?
+    addDefaultCommand(new Command(["sbcl[ose]"],
+        function(args)
+        {
+            if (args)
+            {
+                vimperator.echoerr("E488: Trailing characters");
+                return;
+            }
+
+            document.getElementById("sidebar-box").hidden = true;;
+            document.getElementById("sidebar-splitter").hidden = true;;
+        },
+        {
+            short_help: "Close the sidebar",
+            help: "TODO"
+        }
+    ));
+    // TODO: sopen instead? Separate :sidebar from :sbopen and make them behave
+    // more like :cw, :cope etc
+    addDefaultCommand(new Command(["sideb[ar]", "sb[ar]", "sbope[n]"],
         function(args)
         {
             if (!args)

chrome/content/vimperator/vimperator.xul

@@ -55,6 +55,7 @@ the terms of any one of the MPL, the GPL or the LGPL.
     <script type="application/x-javascript;version=1.7" src="ui.js"/>
 
     <window id="main-window">
+
         <toolbar id="vimperator-toolbar" hidden="false" align="center" fullscreentoolbar="true">
             <vbox id="vimperator-container" flex="1" hidden="false">
                 <listbox id="vimperator-bufferwindow" class="plain" rows="10" flex="1" hidden="true"
@@ -94,7 +95,6 @@ the terms of any one of the MPL, the GPL or the LGPL.
 
                 <iframe id="vimperator-multiline-output" src="about:blank" flex="1" height="10px" hidden="false" collapsed="true"
                     onclick="vimperator.commandline.onMultilineOutputEvent(event)"/>
-                    <!--onkeypress="vimperator.commandline.onMultilineOutputEvent(event)"/>-->
 
                 <hbox id="vimperator-commandline" flex="1" hidden="false">
                     <label class="plain" id="vimperator-commandline-prompt"  flex="0" crop="end" value="" collapsed="true"/>
@@ -122,6 +122,37 @@ the terms of any one of the MPL, the GPL or the LGPL.
         <box id="vimperator-visualbell" hidden="true"/>
 
     </window>
+
+    <menupopup id="viewSidebarMenu">
+        <menuitem observes="vimperator-viewAddonsSidebar" label="Add-ons" accesskey="A"/>
+        <menuitem observes="vimperator-viewPreferencesSidebar" label="Preferences" accesskey="P"/>
+        <menuitem observes="vimperator-viewDownloadsSidebar" label="Downloads" accesskey="D"/>
+    </menupopup>
+
+    <broadcasterset id="mainBroadcasterSet">
+        <broadcaster id="vimperator-viewAddonsSidebar"
+            autoCheck="false"
+            type="checkbox"
+            group="sidebar"
+            sidebarurl="chrome://mozapps/content/extensions/extensions.xul"
+            sidebartitle="Add-ons"
+            oncommand="toggleSidebar('vimperator-viewAddonsSidebar');"/>
+        <broadcaster id="vimperator-viewPreferencesSidebar"
+            autoCheck="false"
+            type="checkbox"
+            group="sidebar"
+            sidebarurl="about:config"
+            sidebartitle="Preferences"
+            oncommand="toggleSidebar('vimperator-viewPreferencesSidebar');"/>
+        <broadcaster id="vimperator-viewDownloadsSidebar"
+            autoCheck="false"
+            type="checkbox"
+            group="sidebar"
+            sidebarurl="chrome://mozapps/content/downloads/downloads.xul"
+            sidebartitle="Downloads"
+            oncommand="toggleSidebar('vimperator-viewDownloadsSidebar');"/>
+    </broadcasterset>
+
 </overlay>
 
 <!-- vim: set fdm=marker sw=4 ts=4 et: -->