From 2fe5bd29131a4a9114bdc95804a31a84d4c1586f Mon Sep 17 00:00:00 2001
From: Doug Kearns <dougkearns@gmail.com>
Date: Tue, 16 Sep 2008 15:45:26 +0000
Subject: [PATCH] pull some overlay height/width attributes out in to the CSS
 file

---
 content/vimperator.xul |  8 ++++----
 skin/vimperator.css    | 14 +++++++++++++-
 2 files changed, 17 insertions(+), 5 deletions(-)

diff --git a/content/vimperator.xul b/content/vimperator.xul
index d0116eaf..15579845 100644
--- a/content/vimperator.xul
+++ b/content/vimperator.xul
@@ -81,12 +81,12 @@ the terms of any one of the MPL, the GPL or the LGPL.
         <!-- As of Firefox 3.1pre, <iframe>.height changes do not seem to have immediate effect,
              therefore we need to put them into a <vbox> for which that works just fine -->
         <vbox class="liberator-container" hidden="false" collapsed="true">
-            <iframe id="liberator-multiline-output" src="about:blank" flex="1" height="10px" hidden="false" collapsed="false"
+            <iframe id="liberator-multiline-output" src="about:blank" flex="1" hidden="false" collapsed="false"
                 onclick="liberator.commandline.onMultilineOutputEvent(event)"/>
         </vbox>
 
         <vbox class="liberator-container" hidden="false" collapsed="true">
-            <iframe id="liberator-completions" src="about:blank" flex="1" height="250px" hidden="false" collapsed="false"
+            <iframe id="liberator-completions" src="about:blank" flex="1" hidden="false" collapsed="false"
                 onclick="liberator.commandline.onMultilineOutputEvent(event)"/>
         </vbox>
 
@@ -108,10 +108,10 @@ the terms of any one of the MPL, the GPL or the LGPL.
     </window>
 
     <statusbar id="status-bar" class="hl-StatusLine">
-        <hbox insertbefore="statusbar-display" id="liberator-statusline" flex="1" height="10" hidden="false" align="center">
+        <hbox insertbefore="statusbar-display" id="liberator-statusline" flex="1" hidden="false" align="center">
             <textbox class="plain" id="liberator-statusline-field-url" readonly="false" flex="1" crop="end"/>
             <label class="plain" id="liberator-statusline-field-inputbuffer"    flex="0"/>
-            <label class="plain" id="liberator-statusline-field-progress"       flex="0" width="200px"/>
+            <label class="plain" id="liberator-statusline-field-progress"       flex="0"/>
             <label class="plain" id="liberator-statusline-field-tabcount"       flex="0"/>
             <label class="plain" id="liberator-statusline-field-bufferposition" flex="0"/>
         </hbox>
diff --git a/skin/vimperator.css b/skin/vimperator.css
index 1dcaf2d9..fedb1997 100644
--- a/skin/vimperator.css
+++ b/skin/vimperator.css
@@ -59,6 +59,7 @@ the terms of any one of the MPL, the GPL or the LGPL.
 }
 #liberator-statusline {
     font-family: monospace;
+    height: 10px;
     margin: 0px;
 }
 #liberator-statusline > label {
@@ -72,6 +73,10 @@ the terms of any one of the MPL, the GPL or the LGPL.
     color: inherit;
 }
 
+liberator-statusline-field-progress {
+    width: 200px;
+}
+
 /* no longer at the window's bottom right corner */
 .statusbar-resizerpanel {
     display: none;
@@ -172,6 +177,14 @@ a.hl-URL:hover {
 
 /* MOW */
 
+#liberator-completions {
+    height: 250px;
+}
+
+#liberator-multiline-output {
+    height: 10px;
+}
+
 #liberator-completions, #liberator-multiline-output {
     overflow: hidden;
     background-color: white;
@@ -203,5 +216,4 @@ a.hl-URL:hover {
     border: 0px;
 }
 
-
 /* vim: set fdm=marker sw=4 ts=4 et: */