Changeset e1eb70b in github

Timestamp:

Aug 29, 2008 9:20:12 AM (5 years ago)

Author:

thomascube <thomas@…>

Branches:

master, HEAD, courier-fix, dev-browser-capabilities, pdo, release-0.6, release-0.7, release-0.8

Children:

835a0cab

Parents:

940fc13

Message:

Fix position of collapse/expand icons (#1485302)

Files:

• program/include/main.inc (modified) (2 diffs)
• skins/default/mail.css (modified) (4 diffs)

program/include/main.inc

@@ -1054 +1054 @@
         'class' => join(' ', $classes),
         'noclose' => true),
-      html::div(array(
-        'class' => empty($folder['folders']) ? 'nocollapse' : ($collapsed ? 'collapsed' : 'expanded'),
-        'onclick' => sprintf("%s.command('collapse-folder', '%s')", JS_OBJECT_NAME, $js_name)
-      ), ' ') .
       html::a(array(
         'href' => rcmail_url('', array('_mbox' => $folder['id'])),
@@ -1065 +1061 @@
         'onmouseup' => sprintf("return %s.folder_mouse_up('%s')", JS_OBJECT_NAME, $js_name),
         'title' => $title,
-      ), Q($foldername)));
+      ), Q($foldername)) .
+      (!empty($folder['folders']) ? html::div(array(
+        'class' => ($collapsed ? 'collapsed' : 'expanded'),
+        'style' => "position:absolute",
+        'onclick' => sprintf("%s.command('collapse-folder', '%s')", JS_OBJECT_NAME, $js_name)
+      ), ' ') : ''));
     
     if (!empty($folder['folders']))

skins/default/mail.css

@@ -267 +267 @@
 #mailboxlist
 {
+  position:relative;
   width: 100%;
   height: auto;
@@ -279 +280 @@
 #mailboxlist li
 {
+  display: block;
+  position: relative;
   font-size: 11px;
   background: url(images/icons/folder-closed.png) no-repeat;
@@ -288 +291 @@
 {
   position: absolute;
-  padding-left: 10px;
-  padding-top: 2px;
-  padding-bottom: 2px;
-  margin-right: 2px;
-  width: 12px;
+  left: 8px !important;
+  left: -16px;
+  top: 2px;
+  width: 14px;
+  height: 16px;
 }
 
@@ -340 +343 @@
   cursor: default;
   display: block;
+  position: relative;
   padding-left: 25px;
   padding-top: 2px;
   padding-bottom: 2px;
   text-decoration: none;
-  /* css hack for IE */
-  margin-top: expression('-12px');
 }