Fix missing arrow icon of collapse macro (#32754).

Patch by Marius BALTEANU.

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

diff --git a/lib/redmine/wiki_formatting/macros.rb b/lib/redmine/wiki_formatting/macros.rb
index 53276834fc49f030f64c9c8f4e5921b6f87a8802..0e1e4234c813800d77380db7b7bd39eb28de8da4 100644 (file)
--- a/lib/redmine/wiki_formatting/macros.rb
+++ b/lib/redmine/wiki_formatting/macros.rb
@@ -234,8 +234,8 @@ module Redmine
         hide_label = args[1] || args[0] || l(:button_hide)
         js = "$('##{html_id}-show, ##{html_id}-hide').toggle(); $('##{html_id}').fadeToggle(150);"
         out = ''.html_safe
-        out << link_to_function(show_label, js, :id => "#{html_id}-show", :class => 'collapsible collapsed')
-        out << link_to_function(hide_label, js, :id => "#{html_id}-hide", :class => 'collapsible', :style => 'display:none;')
+        out << link_to_function(show_label, js, :id => "#{html_id}-show", :class => 'icon icon-collapsed collapsible')
+        out << link_to_function(hide_label, js, :id => "#{html_id}-hide", :class => 'icon icon-expended collapsible', :style => 'display:none;')
         out << content_tag('div', textilizable(text, :object => obj, :headings => false, :inline_attachments => @@inline_attachments), :id => html_id, :class => 'collapsed-text', :style => 'display:none;')
         out
       end

diff --git a/test/unit/lib/redmine/wiki_formatting/macros_test.rb b/test/unit/lib/redmine/wiki_formatting/macros_test.rb
index ceda727c81d8e22b67293cb1ed2f70498ec8a75f..214e7786802eb0022536800afc454ff40adb1b03 100644 (file)
--- a/test/unit/lib/redmine/wiki_formatting/macros_test.rb
+++ b/test/unit/lib/redmine/wiki_formatting/macros_test.rb
@@ -215,8 +215,8 @@ class Redmine::WikiFormatting::MacrosTest < Redmine::HelperTest
 
       assert_select_in result, 'div.collapsed-text'
       assert_select_in result, 'strong', :text => 'Collapsed'
-      assert_select_in result, 'a.collapsible.collapsed', :text => 'Show'
-      assert_select_in result, 'a.collapsible', :text => 'Hide'
+      assert_select_in result, 'a.collapsible.icon-collapsed', :text => 'Show'
+      assert_select_in result, 'a.collapsible.icon-expended', :text => 'Hide'
     end
   end
 
@@ -226,8 +226,8 @@ class Redmine::WikiFormatting::MacrosTest < Redmine::HelperTest
 
     assert_select_in result, 'div.collapsed-text'
     assert_select_in result, 'strong', :text => 'Collapsed'
-    assert_select_in result, 'a.collapsible.collapsed', :text => 'Example'
-    assert_select_in result, 'a.collapsible', :text => 'Example'
+    assert_select_in result, 'a.collapsible.icon-collapsed', :text => 'Example'
+    assert_select_in result, 'a.collapsible.icon-expended', :text => 'Example'
   end
 
   def test_macro_collapse_with_two_args
@@ -236,8 +236,8 @@ class Redmine::WikiFormatting::MacrosTest < Redmine::HelperTest
 
     assert_select_in result, 'div.collapsed-text'
     assert_select_in result, 'strong', :text => 'Collapsed'
-    assert_select_in result, 'a.collapsible.collapsed', :text => 'Show example'
-    assert_select_in result, 'a.collapsible', :text => 'Hide example'
+    assert_select_in result, 'a.collapsible.icon-collapsed', :text => 'Show example'
+    assert_select_in result, 'a.collapsible.icon-expended', :text => 'Hide example'
   end
 
   def test_macro_collapse_should_not_break_toc