update.

themes/modern/communicator/directory/directory.css

@@ -28,10 +28,6 @@
 
 @import url(chrome://communicator/skin/);
 
-treecol[sortActive="true"] {
-  background-color: lightgray;
-}
-
 /* Hide the twisty when type != DIRECTORY (XXX is there a '!=' in CSS?) */
 treeitem[type="FILE"] > treerow > treecell > .twisty {
   list-style-image: none;

themes/modern/communicator/search/search.css

@@ -54,10 +54,6 @@ titledbutton#allEngines {
   margin-right: 12px;
 }
 
-treecol[sortActive="true"] {
- background-color: lightgray;
-}
-
 titledbutton.engine[loading="true"] {
  list-style-image: url("resource:/res/rdf/loading.gif") ! important ;
 }
@@ -139,4 +135,4 @@ tree[singleclick="true"] treeitem[selected="true"] > treerow {
 
 titledbutton.push {
   background-color: #CCCCCC;
-}
+}

themes/modern/communicator/sidebar/sidebarOverlay.css

@@ -149,5 +149,5 @@ box#title-box {
 }
 
 *[collapsed="true"] {
-  visibility: collapsed;
+  visibility: collapse;
 }

themes/modern/editor/EditorDialog.css

@@ -96,7 +96,8 @@ titledbox {
 /* temporary -- we need a simple box-based list defined in XBL */
 tree.list {
   border: 1px inset #CCCCCC;
-  margin: 1px 4px 2px 4px;
+  /* same as in menulist.css */
+  margin: 1px 5px 2px 5px;
   width: 10em;
   height: 3em;
   /* use rows="#" in XUL to define height */

themes/modern/editor/editor.css

@@ -21,9 +21,21 @@
  */
 
 @import url(chrome://communicator/skin/);
+/* Note: most of our styles are in EditorToolbars.css 
+   in order to share with all users of editor
+*/
 
 iframe#content-frame { 
   min-width: 10px; 
   min-height: 10px;
   height: 400px;
 } 
+
+.source-editor, .source-editor:focus {
+  margin: 0px 5px 5px 0px;
+  border: 0px;
+  border-top: 1px solid black;
+  /* Scroll bars are in content, 
+     so we can't use right and bottom padding! */
+  padding: 5px 0px 0px 5px;
+}

themes/modern/global/menu.css

@@ -167,3 +167,19 @@ menuitem[checked="true"][menuactive="true"]
   {
     list-style-image      : url("chrome://global/skin/menu-arrow-hover.gif");
   }
+
+/** Tooltips (tooltip <popup> element)
+ *  class="tooltip"
+ **/
+.tooltip
+  {
+    background-color      : #FFFFCC;
+    border                : 1px outset #FFFFCC;
+    color                 : #000000;
+    font                  : smaller;
+    padding               : 3px;
+  }
+  
+  
+  
+  

themes/modern/messenger/mailWidgets.xml

@@ -133,7 +133,6 @@
         </body>
       </method>
     </interface>
-    <handler type="command" value="dump('onCommand!\n');"/>
   </binding>
 
   <!-- searchattribute - Subject, Sender, To, CC, etc. -->
@@ -259,11 +258,17 @@
       <!-- the three tags that make up a term - to use, set the
            attribute in the XUL to the ID of the term.
         -->
-      <property name="searchattribute" readonly="true" onget="return document.getElementById(this.getAttribute('searchattribute'));"/>
+      <property name="searchattribute" 
+                onget="return document.getElementById(this.getAttribute('searchattribute'));"
+                onset="this.setAttribute('searchattribute',val.id)"/>
 
-      <property name="searchoperator" readonly="true" onget="return document.getElementById(this.getAttribute('searchoperator'));"/>
+      <property name="searchoperator" 
+                onget="return document.getElementById(this.getAttribute('searchoperator'));"
+                onset="this.setAttribute('searchoperator',val.id)"/>
 
-      <property name="searchvalue" readonly="true" onget="return document.getElementById(this.getAttribute('searchvalue'));"/>
+      <property name="searchvalue" 
+                onget="return document.getElementById(this.getAttribute('searchvalue'));"
+                onset="this.setAttribute('searchvalue',val.id)"/>
 
       <!-- takes an nsIMsgSearchValue and initializes the appropriate widget
            with the right value
@@ -285,12 +290,6 @@
             var searchOperator=this.searchoperator;
             var searchValue=this.searchvalue;
 
-            dump("searchAttribute=" + searchAttribute + "\n");
-            dump("searchOperator=" + searchOperator + "\n");
-            dump("searchValue=" + searchValue + "\n");
-
-            dump("Setting " + attrib.value + ", " +
-                 op.value + ", " + value.value + "\n");
             if (searchAttribute) searchAttribute.value = attrib.value;
             if (searchOperator) searchOperator.value = op.value;
             if (searchValue) searchValue.value = value.value;