Changeset 4101 in subversion

Timestamp:

Oct 17, 2010 1:43:05 PM (3 years ago)

Author:

alec

Message:

• Add separate column for message status icon (#1486665)

Location:

trunk/roundcubemail

Files:

• CHANGELOG (modified) (1 diff)
• config/main.inc.php.dist (modified) (1 diff)
• program/js/app.js (modified) (4 diffs)
• program/localization/en_US/labels.inc (modified) (1 diff)
• program/localization/pl_PL/labels.inc (modified) (1 diff)
• program/steps/mail/func.inc (modified) (5 diffs)
• skins/default/ie6hacks.css (modified) (1 diff)
• skins/default/mail.css (modified) (6 diffs)
• skins/default/templates/mail.html (modified) (1 diff)

trunk/roundcubemail/CHANGELOG

@@ -32 +32 @@
 - Display IMAP errors for LIST/THREAD/SEARCH commands (#1486905)
 - Add LITERAL+ (IMAP4 non-synchronizing literals) support (RFC2088)
+- Add separate column for message status icon (#1486665)
 
 RELEASE 0.4.2

trunk/roundcubemail/config/main.inc.php.dist

@@ -294 +294 @@
 
 // These cols are shown in the message list. Available cols are:
-// subject, from, to, cc, replyto, date, size, flag, attachment
-$rcmail_config['list_cols'] = array('subject', 'from', 'date', 'size', 'flag', 'attachment');
+// subject, from, to, cc, replyto, date, size, status, flag, attachment
+$rcmail_config['list_cols'] = array('subject', 'status', 'from', 'date', 'size', 'flag', 'attachment');
 
 // the default locale setting (leave empty for auto-detection)

trunk/roundcubemail/program/js/app.js

@@ -1551 +1551 @@
       $.extend(row, this.env.messages[uid]);
 
-    // set eventhandler to message icon
-    if (this.env.subject_col != null && (row.icon = document.getElementById('msgicn'+row.uid))) {
+    row.msgicon = document.getElementById('msgicn'+row.uid);
+
+    // set eventhandler to message status icon
+    if (row.icon = document.getElementById('statusicn'+row.uid)) {
       row.icon._row = row.obj;
       row.icon.onmousedown = function(e) { self.command('toggle_status', this); rcube_event.cancel(e); };
@@ -1619 +1621 @@
     row.className = css_class;
 
-    // message status icon
+    // message status icons
     css_class = 'msgicon';
     if (!flags.unread && flags.unread_children > 0)
       css_class += ' unreadchildren';
-    if (flags.deleted)
-      css_class += ' deleted';
-    else if (flags.replied || flags.forwarded) {
-      if (flags.replied)
-        css_class += ' replied';
-      if (flags.forwarded)
-        css_class += ' forwarded';
-    }
-    else if (flags.unread)
-      css_class += ' unread';
+    if (flags.replied)
+      css_class += ' replied';
+    if (flags.forwarded)
+      css_class += ' forwarded';
 
     // update selection
@@ -1693 +1689 @@
         else
           html = ' ';
+      }
+      else if (c == 'status') {
+        if (flags.deleted)
+          css_class = 'deleted';
+        else if (flags.unread)
+          css_class = 'unread';
+        else
+          css_class = 'msgicon';
+        html = '<span id="statusicn'+uid+'" class="'+css_class+'">&nbsp;</span>';
       }
       else if (c == 'threads')
@@ -2222 +2227 @@
 
     if (rows[uid].icon) {
+      if (rows[uid].deleted)
+        css_class = 'deleted';
+      else if (rows[uid].unread)
+        css_class = 'unread';
+      else
+        css_class = 'msgicon';
+
+      rows[uid].icon.className = css_class;
+    }
+
+    if (rows[uid].msgicon) {
       css_class = 'msgicon';
       if (!rows[uid].unread && rows[uid].unread_children)
         css_class += ' unreadchildren';
-      if (rows[uid].deleted)
-        css_class += ' deleted';
-      else if (rows[uid].replied || rows[uid].forwarded) {
-        if (rows[uid].replied)
-          css_class += ' replied';
-        if (rows[uid].forwarded)
-          css_class += ' forwarded';
-      }
-      else if (rows[uid].unread)
-        css_class += ' unread';
-
-      rows[uid].icon.className = css_class;
+      if (rows[uid].replied)
+        css_class += ' replied';
+      if (rows[uid].forwarded)
+        css_class += ' forwarded';
+
+      rows[uid].msgicon.className = css_class;
     }
 

trunk/roundcubemail/program/localization/en_US/labels.inc

@@ -53 +53 @@
 $labels['priority'] = 'Priority';
 $labels['organization'] = 'Organization';
+$labels['readstatus'] = 'Read status';
 
 // aliases

trunk/roundcubemail/program/localization/pl_PL/labels.inc

@@ -47 +47 @@
 $labels['date'] = 'Data';
 $labels['size'] = 'Rozmiar';
+$labels['readstatus'] = 'Przeczytano';
 $labels['priority'] = 'Priorytet';
 $labels['organization'] = 'Organizacja';

trunk/roundcubemail/program/steps/mail/func.inc

@@ -256 +256 @@
     return;
 
-  // remove 'threads', 'attachment', 'flag' columns, we don't need them here
-  if (($key = array_search('attachment', $a_show_cols)) !== FALSE)
-    unset($a_show_cols[$key]);
-  if (($key = array_search('flag', $a_show_cols)) !== FALSE)
-    unset($a_show_cols[$key]);
-  if (($key = array_search('threads', $a_show_cols)) !== FALSE)
-    unset($a_show_cols[$key]);
+  // remove 'threads', 'attachment', 'flag', 'status' columns, we don't need them here
+  foreach (array('threads', 'attachment', 'flag', 'status') as $col) {
+    if (($key = array_search($col, $a_show_cols)) !== FALSE)
+      unset($a_show_cols[$key]);
+  }
+
   // loop through message headers
-
-  foreach ($a_headers as $n => $header)
-    {
+  foreach ($a_headers as $n => $header) {
     if (empty($header))
       continue;
@@ -276 +273 @@
 
     // format each col; similar as in rcmail_message_list()
-    foreach ($a_show_cols as $col)
-      {
+    foreach ($a_show_cols as $col) {
       if (in_array($col, array('from', 'to', 'cc', 'replyto')))
         $cont = Q(rcmail_address_string($header->$col, 3), 'show');
-      else if ($col=='subject')
-        {
+      else if ($col=='subject') {
         $cont = abbreviate_string(trim($IMAP->decode_header($header->$col)), 160);
         if (!$cont) $cont = rcube_label('nosubject');
         $cont = Q($cont);
-        }
+      }
       else if ($col=='size')
         $cont = show_bytes($header->$col);
@@ -294 +289 @@
 
       $a_msg_cols[$col] = $cont;
-      }
+    }
 
     if ($header->depth)
@@ -333 +328 @@
       $a_msg_flags,
       $insert_top);
-    }
-
-    if ($IMAP->threading) {
-      $OUTPUT->command('init_threads', (array) $roots);
-    }
+  }
+
+  if ($IMAP->threading) {
+    $OUTPUT->command('init_threads', (array) $roots);
+  }
 }
 
@@ -381 +376 @@
         break;
       case 'attachment':
-        $col_name = '<span class="attachment">&nbsp;</span>';
+      case 'status':
+        $col_name = '<span class="' . $col .'">&nbsp;</span>';
         break;
       case 'threads':

trunk/roundcubemail/skins/default/ie6hacks.css

@@ -129 +129 @@
 #messagelist tr td.flag span.flagged,
 #messagelist tr td.flag span.unflagged:hover,
+#messagelist tr td.status span.status,
+#messagelist tr td.status span.msgicon,
+#messagelist tr td.status span.unread,
+#messagelist tr td.status span.deleted,
 #messagelist tr td.subject span.msgicon,
 #messagelist tr td.subject span.msgicon.replied,
 #messagelist tr td.subject span.msgicon.forwarded,
-#messagelist tr td.subject span.msgicon.deleted,
-#messagelist tr td.subject span.msgicon.unread,
 #messagelist tr td.subject span.msgicon.unreadchildren
 {

trunk/roundcubemail/skins/default/mail.css

@@ -702 +702 @@
 }
 
-#messagelist tbody tr td.flag
+#messagelist tbody tr td.flag,
+#messagelist tbody tr td.status
 {
   cursor: pointer;
@@ -708 +709 @@
 
 #messagelist tr td.flag span,
+#messagelist tr td.status span,
 #messagelist tr td.attachment span
 {
@@ -720 +722 @@
 #messagelist tr td.flag span.flagged,
 #messagelist tr td.flag span.unflagged:hover,
+#messagelist tr td.status span.status,
+#messagelist tr td.status span.msgicon,
+#messagelist tr td.status span.deleted,
+#messagelist tr td.status span.unread,
 #messagelist tr td.subject span.msgicon,
 #messagelist tr td.subject span.msgicon.replied,
 #messagelist tr td.subject span.msgicon.forwarded,
-#messagelist tr td.subject span.msgicon.deleted,
-#messagelist tr td.subject span.msgicon.unread,
 #messagelist tr td.subject span.msgicon.unreadchildren
 {
@@ -758 +762 @@
   background-position: 0 -51px;
   margin: 0 2px;
-  cursor: pointer;
 }
 
@@ -776 +779 @@
 }
 
-#messagelist tr td.subject span.msgicon.deleted
+#messagelist tr td.status span.msgicon
+{
+  background-position: 0 -51px;
+}
+
+#messagelist tr td.status span.deleted
 {
   background-position: 0 -187px;
 }
 
-#messagelist tr td.subject span.msgicon.unread
+#messagelist tr td.status span.status,
+#messagelist tr td.status span.unread
 {
   background-position: 0 -119px;
@@ -822 +831 @@
 #messagelist tr td.attachment,
 #messagelist tr td.threads,
+#messagelist tr td.status,
 #messagelist tr td.flag
 {
-  width: 19px;
-  padding: 0 2px;
+  width: 17px;
+  padding: 0 0 0 2px;
 }
 

trunk/roundcubemail/skins/default/templates/mail.html

@@ -190 +190 @@
     <li><input type="checkbox" name="list_col[]" value="date" id="cols_date" /><label for="cols_date"><roundcube:label name="date" /></label></li>
     <li><input type="checkbox" name="list_col[]" value="size" id="cols_size" /><label for="cols_size"><roundcube:label name="size" /></label></li>
+    <li><input type="checkbox" name="list_col[]" value="status" id="cols_status" /><label for="cols_status"><roundcube:label name="readstatus" /></label></li>
     <li><input type="checkbox" name="list_col[]" value="attachment" id="cols_attachment" /><label for="cols_attachment"><roundcube:label name="attachment" /></label></li>
     <li><input type="checkbox" name="list_col[]" value="flag" id="cols_flag" /><label for="cols_flag"><roundcube:label name="flag" /></label></li>