From: Michael M Slusarz <slusarz@curecanti.org>
Date: Mon, 30 Mar 2009 19:26:35 +0000 (-0600)
Subject: Don't use separate separator div.
X-Git-Url: https://git.internetallee.de/?a=commitdiff_plain;h=3e277f2890a82954a480e3782bcab6fc502ed6ed;p=horde.git

Don't use separate separator div.
---

diff --git a/imp/js/src/DimpBase.js b/imp/js/src/DimpBase.js
index 654bfd265..ad65d655a 100644
--- a/imp/js/src/DimpBase.js
+++ b/imp/js/src/DimpBase.js
@@ -494,11 +494,7 @@ var DimpBase = {
                      * elsewhere. */
                     tmp = [ $('button_deleted') ].compact().invoke('up', 'SPAN');
                     [ 'ctx_message_', 'ctx_draft_' ].each(function(c) {
-                        var t = $(c + 'setflag');
-                        if (t) {
-                            tmp = tmp.concat([ t, t.previous('DIV.sep'), t.next('DIV.sep') ]);
-                        }
-                        tmp = tmp.concat($(c + 'deleted', c + 'undeleted'));
+                        tmp = tmp.concat($(c + 'deleted', c + 'setflag', c + 'undeleted'));
                     });
 
                     if (this.viewport.getMetaData('readonly')) {
@@ -759,9 +755,9 @@ var DimpBase = {
             break;
 
         case 'ctx_otheractions':
-            tmp = $('oa_sep1', 'oa_blacklist', 'oa_whitelist', 'oa_sep2', 'oa_undeleted');
+            tmp = $('oa_blacklist', 'oa_whitelist', 'oa_undeleted');
             if (this.viewport.getMetaData('readonly')) {
-                $('oa_setflag').hide().previous('DIV.sep').hide();
+                $('oa_setflag').hide();
             } else {
                 tmp.push($('oa_setflag'));
             }
diff --git a/imp/templates/index/index-dimp.inc b/imp/templates/index/index-dimp.inc
index 36dd71f7c..3b1f90aaf 100644
--- a/imp/templates/index/index-dimp.inc
+++ b/imp/templates/index/index-dimp.inc
@@ -339,11 +339,9 @@ function _simpleButton($id, $text, $image, $imagedir = null)
  <a id="ctx_folder_create"><span class="contextImg"></span><?php echo _("Create subfolder") ?></a>
  <a id="ctx_folder_rename"><span class="contextImg"></span><?php echo _("Rename Folder") ?></a>
  <a id="ctx_folder_empty"><span class="contextImg"></span><?php echo _("Empty Folder") ?></a>
- <a id="ctx_folder_delete"><span class="contextImg"></span><?php echo _("Delete Folder") ?></a>
- <div id="ctx_folder_seen_sep" class="sep"></div>
+ <a id="ctx_folder_delete" class="sep"><span class="contextImg"></span><?php echo _("Delete Folder") ?></a>
  <a id="ctx_folder_seen"><span class="contextImg"></span><?php echo _("Mark all as Read") ?></a>
- <a id="ctx_folder_unseen"><span class="contextImg"></span><?php echo _("Mark all as New") ?></a>
- <div id="ctx_folder_poll_sep" class="sep"></div>
+ <a id="ctx_folder_unseen" class="sep"><span class="contextImg"></span><?php echo _("Mark all as New") ?></a>
  <a id="ctx_folder_poll"><span class="contextImg"></span><?php echo _("Check for New Mail") ?></a>
  <a id="ctx_folder_nopoll"><span class="contextImg"></span><?php echo _("Do Not Check for New Mail") ?></a>
 </div>
@@ -355,10 +353,8 @@ function _simpleButton($id, $text, $image, $imagedir = null)
 
 <div class="context" id="ctx_message" style="display:none">
  <a id="ctx_message_reply"><span class="contextImg"></span><?php echo _("Reply") ?></a>
- <a id="ctx_message_forward"><span class="contextImg"></span><?php echo _("Forward") ?></a>
- <div class="sep"></div>
- <a id="ctx_message_setflag"><span class="contextImg"></span><?php echo _("Mark as") ?></a>
- <div class="sep"></div>
+ <a id="ctx_message_forward" class="sep"><span class="contextImg"></span><?php echo _("Forward") ?></a>
+ <a id="ctx_message_setflag" class="sep"><span class="contextImg"></span><?php echo _("Mark as") ?></a>
 <?php if (!empty($conf['spam']['reporting'])): ?>
  <a id="ctx_message_spam"><span class="contextImg"></span><?php echo _("Mark as Spam") ?></a>
 <?php endif; ?>
@@ -378,10 +374,8 @@ function _simpleButton($id, $text, $image, $imagedir = null)
 </div>
 
 <div class="context" id="ctx_draft" style="display:none">
- <a id="ctx_draft_resume"><span class="contextImg"></span><?php echo _("Resume Draft") ?></a>
- <div class="sep"></div>
- <a id="ctx_draft_setflag"><span class="contextImg"></span><?php echo _("Mark as") ?></a>
- <div class="sep"></div>
+ <a id="ctx_draft_resume" class="sep"><span class="contextImg"></span><?php echo _("Resume Draft") ?></a>
+ <a id="ctx_draft_setflag" class="sep"><span class="contextImg"></span><?php echo _("Mark as") ?></a>
  <a id="ctx_draft_deleted"><span class="contextImg"></span><?php echo _("Delete") ?></a>
 <?php if (!$usetrash): ?>
  <a id="ctx_draft_undeleted"><span class="contextImg"></span><?php echo _("Undelete") ?></a>
@@ -395,22 +389,18 @@ function _simpleButton($id, $text, $image, $imagedir = null)
 </div>
 
 <div class="context" id="ctx_otheractions" style="display:none">
- <a id="previewtoggle"><span class="contextImg"></span><?php echo ($prefs->getValue('dimp_show_preview') ? _("Hide Preview") : _("Show Preview")); ?></a>
- <div class="sep"></div>
- <a id="oa_setflag"><span class="contextImg"></span><?php echo _("Mark as") ?></a>
- <div class="sep" id="oa_sep1"></div>
+ <a id="previewtoggle" class="sep"><span class="contextImg"></span><?php echo ($prefs->getValue('dimp_show_preview') ? _("Hide Preview") : _("Show Preview")); ?></a>
+ <a id="oa_setflag" class="sep"><span class="contextImg"></span><?php echo _("Mark as") ?></a>
 <?php if ($has_blacklist || $has_whitelist): ?>
 <?php if ($has_blacklist): ?>
- <a id="oa_blacklist"><span class="contextImg"></span><?php echo _("Blacklist") ?></a>
+ <a id="oa_blacklist"<?php if (!$has_whitelist): ?> class="sep"<?php endif; ?>><span class="contextImg"></span><?php echo _("Blacklist") ?></a>
 <?php endif; ?>
 <?php if ($has_whitelist): ?>
- <a id="oa_whitelist"><span class="contextImg"></span><?php echo _("Whitelist") ?></a>
+ <a id="oa_whitelist" class="sep"><span class="contextImg"></span><?php echo _("Whitelist") ?></a>
 <?php endif; ?>
- <div class="sep" id="oa_sep2"></div>
 <?php endif; ?>
- <a id="oa_selectall"><span class="contextImg"></span><?php echo _("Select All") ?></a>
+ <a id="oa_selectall"<?php if (!$usetrash): ?> class="sep" <?php endif; ?>><span class="contextImg"></span><?php echo _("Select All") ?></a>
 <?php if (!$usetrash): ?>
- <div class="sep"></div>
  <a id="oa_purge_deleted"><span class="contextImg"></span><?php echo _("Purge Deleted") ?></a>
  <a id="oa_undeleted"><span class="contextImg"></span><?php echo _("Undelete") ?></a>
 <?php endif; ?>
diff --git a/imp/themes/screen-dimp.css b/imp/themes/screen-dimp.css
index 0c4c43ef2..072abbe74 100644
--- a/imp/themes/screen-dimp.css
+++ b/imp/themes/screen-dimp.css
@@ -810,7 +810,7 @@ span.attachName:hover, .nojserror {
     color: #fff;
     cursor: pointer;
 }
-.context div.sep {
+.context .sep {
     border-bottom: 1px #d4d4d4 solid;
 }
 .contextSubmenu {