Redmine

The below changes fix the problem:

mariusbalteanu@Mariuss-MacBook-Pro redmine % git diff
diff --git a/lib/redmine/wiki_formatting/macros.rb b/lib/redmine/wiki_formatting/macros.rb
index 53276834f..0e1e4234c 100644
--- 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

Thanks @Popoki Tom for reporting the issue.

Go MAEDA wrote:

Thank you for posting the patch #32754#note-2 but it breaks some tests.

[...]

It can be fixed by applying 32754-update-test.patch.

Go MAEDA wrote:

Go MAEDA wrote:

Thank you for posting the patch #32754#note-2 but it breaks some tests.

[...]

It can be fixed by applying 32754-update-test.patch.

Please use this one, all tests pass: https://gitlab.com/redmine-org/redmine/pipelines/107443098

Committed the patch. Thank you for detecting and fixing the issue.

Michael Gerz wrote:

I think it hasn’t been backported, yet.

You are right, thank you for pointing it out. I have just merged the change to 4.1-stable in r19424.