Changeset 9bd97cc in github

Timestamp:

Feb 27, 2012 5:00:00 AM (16 months ago)

Author:

alecpl <alec@…>

Branches:

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

Children:

b91f048

Parents:

2c193722

Message:

• Improve headers toggle button

Location:

skins/larry

Files:

• mail.css (modified) (2 diffs)
• templates/messagepreview.html (modified) (2 diffs)
• ui.js (modified) (1 diff)

skins/larry/mail.css

@@ -700 +700 @@
 }
 
-#messagepreviewheader .iconlink {
-        text-indent: -1000px;
-        padding: 1px 1px 1px 20px;
-}
-
-#messagepreviewheader .iconlink.add {
-        background-position: -5px -359px;
-}
-
-#messagepreviewheader .iconlink.delete {
-        background-position: -5px -339px;
+#previewheaderstoggle {
+    width: 16px;
+    height: 16px;
+    padding: 0;
+    margin-top: 5px;
+    position: absolute;
+    outline: none;
+}
+
+#previewheaderstoggle.iconlink.add {
+        background-position: -30px -241px;
+}
+
+#previewheaderstoggle.iconlink.remove {
+        background-position: -8px -241px;
 }
 
@@ -780 +784 @@
 
 #messagepreviewheader .headers-table {
-        margin: 0 -8px;
+        margin-left: 10px;
 }
 

skins/larry/templates/messagepreview.html

@@ -10 +10 @@
 <h3 class="subject"><roundcube:object name="messageHeaders" valueOf="subject" /></h3>
 
+<a href="#details" id="previewheaderstoggle" class="iconlink add"></a>
+
 <table class="headers-table" id="preview-shortheaders"><tbody><tr>
 <roundcube:if condition="env:mailbox == config:drafts_mbox || env:mailbox == config:sent_mbox">
@@ -20 +22 @@
         <td class="header-title"><roundcube:label name="date" /></td>
         <td class="header from"><roundcube:object name="messageHeaders" valueOf="date" /></td>
-        <td class="header"><a href="#details" id="previewheaderstoggle" class="iconlink add">Details</a>
 </tr></tbody></table>
 

skins/larry/ui.js

@@ -463 +463 @@
   {
     $('#preview-shortheaders').toggle();
-    var full = $('#preview-allheaders').toggle();
-    
+    var full = $('#preview-allheaders').toggle(),
+      button = $('a#previewheaderstoggle');
+
     // add toggle button to full headers table
-    if (!full.data('mod')) {
-      $('<a>').attr('href', '#hide')
-        .addClass('iconlink remove')
-        .html('Hide')
-        .appendTo($('<td>').appendTo($('tr:first', full)))
-        .click(function(){ toggle_preview_headers(this);return false });
-      full.data('mod', true);
-    }
+    if (full.is(':visible'))
+      button.attr('href', '#hide').removeClass('add').addClass('remove')
+    else
+      button.attr('href', '#details').removeClass('remove').addClass('add')
   }