sidebar panel preview skinnability

xpfe/components/sidebar/resources/customize-panel.xul

@@ -27,8 +27,7 @@
   xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul"
   onload="Init();">
 
-  <html:script src="chrome://communicator/content/sidebar/customize-panel.js" />
-  <html:iframe id="customize_frame" type="content" src="about:blank"
-    flex="100%"/>
+  <script src="chrome://communicator/content/sidebar/customize-panel.js" />
+  <iframe id="customize_frame" type="content" src="about:blank" flex="1"/>
 
 </window>

xpfe/components/sidebar/resources/customize.js

@@ -210,10 +210,8 @@ function PreviewPanel()
     var preview_URL  = get_attr(database, rdfNode, 'content');
     if (!preview_URL || !preview_name) break;
 
-    var preview = window.open("chrome://communicator/content/sidebar/preview.xul",
-                              "_blank", "chrome,resizable");
-    preview.panel_name = preview_name;
-    preview.panel_URL = preview_URL;
+    var preview = window.openDialog("chrome://communicator/content/sidebar/preview.xul",
+                              "_blank", "chrome,resizable", preview_name, preview_URL);
   }
 }
 

xpfe/components/sidebar/resources/preview.js

@@ -21,8 +21,10 @@
 
 function Init()
 {
+  var panel_name = window.arguments[0];
+  var panel_URL = window.arguments[1];
   var panel_title = document.getElementById('paneltitle');
   var preview_frame = document.getElementById('previewframe');
-  panel_title.setAttribute('value', panel_name);
+  panel_title.description = panel_name;  
   preview_frame.setAttribute('src', panel_URL);
 }

xpfe/components/sidebar/resources/preview.xul

@@ -18,9 +18,11 @@
 
 -->
 
-<?xml-stylesheet href="chrome://communicator/skin/sidebar/sidebar.css" type="text/css"?>
+<?xml-stylesheet href="chrome://communicator/skin/" type="text/css"?>
 <?xml-stylesheet href="chrome://communicator/skin/sidebar/preview.css" type="text/css"?>
 
+<?xul-overlay href="chrome://global/content/dialogOverlay.xul"?>
+
 <!DOCTYPE window SYSTEM "chrome://communicator/locale/sidebar/preview.dtd" >
 
 <window
@@ -29,26 +31,21 @@
   xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul"
   onload="Init();"
   class="dialog"
-  align="vertical">
+  title="&sidebar.preview.title.label;"
+  style="width: 26em;"
+  orient="vertical">
 
-<html:script src="chrome://communicator/content/sidebar/preview.js" />
+  <script src="chrome://communicator/content/sidebar/preview.js" />
 
-  <html:div class="title">
-    &sidebar.preview.title.label;
-  </html:div>
+  <box class="box-smallheader" id="paneltitle" title="&sidebar.preview.title.label;" description=""/>
 
-  <html:div>
-    <html:hr />
-  </html:div>
+  <iframe class="inset" id="previewframe" type="content" src="about:blank" flex="1"/>
 
-  <titledbutton id="paneltitle" class="plain paneltitle" align="left"/>
-
-  <html:iframe id="previewframe" type="content" src="about:blank" flex="1"/>
-
-  <box align="horizontal">
-    <spring flex="50%"/>
-    <titledbutton onclick="window.close()" class="push dialog"
-       value="&sidebar.preview.close.label;" />
-    <spring flex="50%"/>
+  <separator class="groove"/>
+  
+  <box halign="right">
+    <spring flex="1"/>
+    <button class="dialog" value="&sidebar.preview.close.label;" oncommand="window.close();"/>
   </box>
+
 </window>