Cherry-picked rev 84a7acd which implemented CSS sprites into branch npk.

2 files changed, 35 insertions, 56 deletions

diff --git a/roundcubemail/program/steps/mail/func.inc b/roundcubemail/program/steps/mail/func.inc
index 89c566fba..e69551c1c 100644
--- a/roundcubemail/program/steps/mail/func.inc
+++ b/roundcubemail/program/steps/mail/func.inc
@@ -174,10 +174,10 @@ function rcmail_message_list($attrib)
     switch ($col)
       {
       case 'flag':
-        $col_name = sprintf($image_tag, $skin_path, $attrib['unflaggedicon'], '');
+        $col_name = '<div class="icon-holder flag"></div>';
         break;
       case 'attachment':
-        $col_name = sprintf($image_tag, $skin_path, $attrib['attachmenticon'], '');
+        $col_name = '<div class="icon-holder attachment"></div>';
         break;
       default:
         $col_name = Q(rcube_label($col));
@@ -268,32 +268,29 @@ function rcmail_message_list($attrib)
     $js_row_arr['unread_children'] = $header->unread_children;
 
     // set message icon
-    if ($header->seen && $attrib['unreadchildrenicon'] && $header->unread_children > 0) 
+    if ($header->seen && $header->unread_children > 0) 
       $message_icon = $attrib['unreadchildrenicon'];
-    else if ($attrib['deletedicon'] && $header->deleted)
-      $message_icon = $attrib['deletedicon'];
-    else if ($attrib['repliedicon'] && $header->answered)
+    else if ($header->deleted)
+      $message_icon = '<div class="icon-holder deleted"></div>';
+    else if ($header->answered)
       {
-      if ($attrib['forwardedrepliedicon'] && $header->forwarded)
-        $message_icon = $attrib['forwardedrepliedicon'];
+      if ($header->forwarded)
+        $message_icon = '<div class="icon-holder forwardedreplied"></div>';
       else
-        $message_icon = $attrib['repliedicon'];
+        $message_icon = '<div class="icon-holder replied"></div>';
       }
-    else if ($attrib['forwardedicon'] && $header->forwarded)
-      $message_icon = $attrib['forwardedicon'];
-    else if ($attrib['unreadicon'] && !$header->seen)
-      $message_icon = $attrib['unreadicon'];
-    else if ($attrib['messageicon'])
-      $message_icon = $attrib['messageicon'];
+    else if ($header->forwarded)
+      $message_icon = '<div class="icon-holder forwarded"></div>';
+    else if (!$header->seen)
+      $message_icon = '<div class="icon-holder unread"></div>';
+    else
+      $message_icon = '<div class="icon-holder message"></div>';
 
-    if ($attrib['flaggedicon'] && $header->flagged)
-      $flagged_icon = $attrib['flaggedicon'];
-    else if ($attrib['unflaggedicon'] && !$header->flagged)
-      $flagged_icon = $attrib['unflaggedicon'];
-    
-    // set attachment icon
-    if ($attrib['attachmenticon'] && preg_match("/multipart\/m/i", $header->ctype))
-      $attach_icon = $attrib['attachmenticon'];
+    // set attachment icons
+    if (preg_match("/multipart\/m/i", $header->ctype))
+      $attach_icon = '<div class="icon-holder attachment"></div>';
+    else
+      $attach_icon = '&nbsp';
         
     $out .= sprintf('<tr id="rcmrow%d" class="message%s%s%s%s"%s>'."\n",
                     $header->uid,
@@ -315,7 +312,7 @@ function rcmail_message_list($attrib)
         $js_row_arr['expanded'] = true;
       $tree .= $header->has_children?'<div id="rcmexpando' . $header->uid . '" class="' . $toggle_expanded . '">&nbsp;</div>':'<div class="leaf">&nbsp;</div>';
       }
-    $tree .= $message_icon ? sprintf($image_tag, $skin_path, $message_icon, '') : '';
+    $tree .= sprintf("<td class=\"icon\">%s</td>\n", $message_icon);
 
     $IMAP->set_charset(!empty($header->charset) ? $header->charset : $CONFIG['default_charset']);
   
@@ -334,7 +331,7 @@ function rcmail_message_list($attrib)
         $cont = sprintf('<a href="%s" onclick="return rcube_event.cancel(event)">%s</a>', Q(rcmail_url($action, array($uid_param=>$header->uid, '_mbox'=>$mbox))), Q($cont));
         }
       else if ($col=='flag')
-        $cont = $flagged_icon ? sprintf($image_tag, $skin_path, $flagged_icon, '') : '';
+        $cont = '<div class="icon-holder flag"></div>';
       else if ($col=='size')
         $cont = show_bytes($header->$col);
       else if ($col=='date')
@@ -349,7 +346,7 @@ function rcmail_message_list($attrib)
       if ($col!='attachment')
         $out .= '<td class="'.$col.'">' . $cont . "</td>\n";
       else
-        $out .= sprintf("<td class=\"icon\">%s</td>\n", $attach_icon ? sprintf($image_tag, $skin_path, $attach_icon, '') : '&nbsp;');
+        $out .= '<td class="icon">' . $attach_icon . "</td>\n";
       }
 
     $out .= "</tr>\n";
@@ -372,27 +369,16 @@ function rcmail_message_list($attrib)
   $OUTPUT->set_env('sort_col', $sort_col);
   $OUTPUT->set_env('sort_order', $sort_order);
   
-  if ($attrib['messageicon'])
-    $OUTPUT->set_env('messageicon', $skin_path . $attrib['messageicon']);
-  if ($attrib['deletedicon'])
-    $OUTPUT->set_env('deletedicon', $skin_path . $attrib['deletedicon']);
-  if ($attrib['unreadicon'])
-    $OUTPUT->set_env('unreadicon', $skin_path . $attrib['unreadicon']);
-  if ($attrib['repliedicon'])
-    $OUTPUT->set_env('repliedicon', $skin_path . $attrib['repliedicon']);
-  if ($attrib['forwardedicon'])
-    $OUTPUT->set_env('forwardedicon', $skin_path . $attrib['forwardedicon']);
-  if ($attrib['forwardedrepliedicon'])
-    $OUTPUT->set_env('forwardedrepliedicon', $skin_path . $attrib['forwardedrepliedicon']);
-  if ($attrib['attachmenticon'])
-    $OUTPUT->set_env('attachmenticon', $skin_path . $attrib['attachmenticon']);
-  if ($attrib['flaggedicon'])
-    $OUTPUT->set_env('flaggedicon', $skin_path . $attrib['flaggedicon']);
-  if ($attrib['unflaggedicon'])
-    $OUTPUT->set_env('unflaggedicon', $skin_path . $attrib['unflaggedicon']);
+  $OUTPUT->set_env('messageicon', '<div class="icon-holder message"></div>');
+  $OUTPUT->set_env('deletedicon', '<div class="icon-holder deleted"></div>');
+  $OUTPUT->set_env('unreadicon', '<div class="icon-holder unread"></div>');
+  $OUTPUT->set_env('repliedicon', '<div class="icon-holder replied"></div>');
+  $OUTPUT->set_env('forwardedicon', '<div class="icon-holder forwarded"></div>');
+  $OUTPUT->set_env('forwardedrepliedicon', '<div class="icon-holder forwardedreplied"></div>');
+  $OUTPUT->set_env('attachmenticon', '<div class="icon-holder attachment"></div>');
+  $OUTPUT->set_env('flagicon', '<div class="icon-holder flag"></div>');
   if ($attrib['unreadchildrenicon'])
     $OUTPUT->set_env('unreadchildrenicon', $skin_path . $attrib['unreadchildrenicon']);
-  
   $OUTPUT->set_env('messages', $a_js_message_arr);
   $OUTPUT->set_env('coltypes', $a_show_cols);
   
diff --git a/roundcubemail/program/steps/settings/manage_folders.inc b/roundcubemail/program/steps/settings/manage_folders.inc
index 80a6b7c65..4adeca4c7 100644
--- a/roundcubemail/program/steps/settings/manage_folders.inc
+++ b/roundcubemail/program/steps/settings/manage_folders.inc
@@ -244,15 +244,8 @@ function rcube_subscription_form($attrib)
     'onclick' => JS_OBJECT_NAME.".command(this.checked?'enable-threading':'disable-threading',this.value)",
   ));
   
-  if (!empty($attrib['deleteicon']))
-    $del_button = html::img(array('src' => $CONFIG['skin_path'] . $attrib['deleteicon'], 'alt' => rcube_label('delete')));
-  else
-    $del_button = rcube_label('delete');
-
-  if (!empty($attrib['renameicon']))
-    $edit_button = html::img(array('src' => $CONFIG['skin_path'] . $attrib['renameicon'], 'alt' => rcube_label('rename')));
-  else
-    $edit_button = rcube_label('rename');
+  $del_button = '<div class="icon-holder delete"></div>';
+  $edit_button = '<div class="icon-holder rename"></div>';
     
   // create list of available folders
   foreach ($list_folders as $i => $folder) {
@@ -281,8 +274,8 @@ function rcube_subscription_form($attrib)
     
     // add rename and delete buttons
     if (!$protected && !$folder['virtual']) {
-      $table->add('rename', html::a(array('href' => "#rename", 'title' => rcube_label('renamefolder')), $edit_button));
-      $table->add('delete', html::a(array('href' => "#delete", 'title' => rcube_label('deletefolder')), $del_button));
+      $table->add('rename', html::div('icon-holder rename', html::a(array('href' => "#rename", 'title' => rcube_label('renamefolder'), 'style' => 'display: block; width: 16px; height: 16px;'), '')));
+      $table->add('delete', html::div('icon-holder delete', html::a(array('href' => "#delete", 'title' => rcube_label('deletefolder'), 'style' => 'display: block; width: 16px; height: 16px;'), '')));
     }
     else {
       $table->add('rename', '&nbsp;');