Some fixes for r18214 (#31496):

* incorrect icon-toggle-plus and icon-toggle-minus background rules
* broken CSS rule in .toggle-multiselect and removes some unnecessary rules
* some cases where clicking on toggle-multiselect element adds the icon class to "a" element instead of "span"
* icon switch in workflow permissions tab

Patch by Marius BALTEANU.

git-svn-id: http://svn.redmine.org/redmine/trunk@18224 e93f8b46-1217-0410-a6f0-8f06a7374b81

diff --git a/app/views/workflows/edit.html.erb b/app/views/workflows/edit.html.erb
index 66eb11a861b447d2d4d51a05ed5ed9841526f873..3f3e3c2f589fc060f00f6a3d1c7c2f5e1b6614fe 100644 (file)
--- a/app/views/workflows/edit.html.erb
+++ b/app/views/workflows/edit.html.erb
@@ -64,11 +64,11 @@ $("a[data-expands]").click(function(e){
   e.preventDefault();
   var target = $($(this).attr("data-expands"));
   if (target.attr("multiple")) {
-    $(event.target).switchClass('icon-toggle-minus', 'icon-toggle-plus');
+    $(this).find('span').switchClass('icon-toggle-minus', 'icon-toggle-plus');
     target.attr("multiple", false);
     target.find("option[value=all]").show();
   } else {
-    $(event.target).switchClass('icon-toggle-plus', 'icon-toggle-minus');
+    $(this).find('span').switchClass('icon-toggle-plus', 'icon-toggle-minus');
     target.attr("multiple", true);
     target.find("option[value=all]").attr("selected", false).hide();
   }

diff --git a/app/views/workflows/permissions.html.erb b/app/views/workflows/permissions.html.erb
index a9af3d20825052589ca2968ea664ee02a4b01aec..60ad1c73c188b47445e426ceeb0157fa70fc52c2 100644 (file)
--- a/app/views/workflows/permissions.html.erb
+++ b/app/views/workflows/permissions.html.erb
@@ -112,9 +112,11 @@ $("a[data-expands]").click(function(e){
   e.preventDefault();
   var target = $($(this).attr("data-expands"));
   if (target.attr("multiple")) {
+    $(this).find('span').switchClass('icon-toggle-minus', 'icon-toggle-plus');
     target.attr("multiple", false);
     target.find("option[value=all]").show();
   } else {
+    $(this).find('span').switchClass('icon-toggle-plus', 'icon-toggle-minus');
     target.attr("multiple", true);
     target.find("option[value=all]").attr("selected", false).hide();
   }

diff --git a/public/stylesheets/application.css b/public/stylesheets/application.css
index 9981efc4bbcff5e050c1fc3d39458a28d52d0e97..a4c77d1188a1bcbf3bb33722220749c88ae38731 100644 (file)
--- a/public/stylesheets/application.css
+++ b/public/stylesheets/application.css
@@ -531,7 +531,7 @@ fieldset#filters td.values select {min-width:130px;}
 .add-filter {width:35%; float:right; text-align: right; vertical-align: top;}
 
 #issue_is_private_wrap {float:right; margin-right:1em;}
-.toggle-multiselect { no-repeat 0% 40%; padding-left:16px; margin-left:0; margin-right:5px; cursor:pointer;}
+.toggle-multiselect { margin-right:5px; cursor:pointer;}
 .buttons { font-size: 0.9em; margin-bottom: 1.4em; margin-top: 1em; }
 
 div#issue-changesets {float:right; width:45%; margin-left: 1em; margin-bottom: 1em; background: #fff; padding-left: 1em; font-size: 90%;}
@@ -1502,8 +1502,8 @@ td.gantt_selected_column .gantt_hdr,.gantt_selected_column_container {
 .icon-bookmark-off { background-image: url(../images/tag_blue_add.png); }
 .icon-sorted-asc { background-image: url(../images/arrow_down.png); }
 .icon-sorted-desc { background-image: url(../images/arrow_up.png); }
-.icon-toggle-plus { background: url(../images/bullet_toggle_plus.png) }
-.icon-toggle-minus { background: url(../images/bullet_toggle_minus.png) }
+.icon-toggle-plus { background-image: url(../images/bullet_toggle_plus.png) }
+.icon-toggle-minus { background-image: url(../images/bullet_toggle_minus.png) }
 
 .icon-file { background-image: url(../images/files/default.png); }
 .icon-file.text-plain { background-image: url(../images/files/text.png); }