Defect #33724

Selected gantt columns are not displayed with MS Edge Legacy

Added by Yuichi HARADA about 1 month ago. Updated about 1 month ago.

Status:ClosedStart date:
Priority:NormalDue date:
Assignee:Go MAEDA% Done:

0%

Category:Gantt
Target version:4.1.2
Resolution:Fixed Affected version:

Description

#27672 makes it possible to select the display columns for the Gantt.
However, if you use MS Edge Legacy (old version), you can select columns, but only the default columns (Status, Priority, Assignee, Updated) are displayed and other selected columns (e.g. Spent time) are not displayed.

The following patch resolves this issue.

diff --git a/app/views/queries/_columns.html.erb b/app/views/queries/_columns.html.erb
index c5d14190e..18443de57 100644
--- a/app/views/queries/_columns.html.erb
+++ b/app/views/queries/_columns.html.erb
@@ -36,7 +36,7 @@
 <%= javascript_tag do %>
 $(document).ready(function(){
   $('.query-columns').closest('form').submit(function(){
-    $('#<%= selected_tag_id %> option').prop('selected', true);
+    $('#<%= selected_tag_id %> option:not(:disabled)').prop('selected', true);
   });
 });
 <% end %>

Related issues

Related to Redmine - Feature #27672: Show selected columns in gantt chart Closed

Associated revisions

Revision 19899
Added by Go MAEDA about 1 month ago

Fix that selected gantt columns are not displayed with MS Edge Legacy (#33724).

Patch by Yuichi HARADA.

Revision 19900
Added by Go MAEDA about 1 month ago

Merged r19899 from trunk to 4.1-stable (#33724).

History

#1 Updated by Go MAEDA about 1 month ago

  • Subject changed from Gantt: Not displayed even if the columns are selected (MS Edge Legacy) to Selected gantt columns are not displayed with MS Edge Legacy
  • Status changed from New to Confirmed
  • Target version set to 4.1.2

I have confirmed the issue and the attached patch fixes it. Setting the target version to 4.1.2.

#2 Updated by Go MAEDA about 1 month ago

  • Related to Feature #27672: Show selected columns in gantt chart added

#3 Updated by Go MAEDA about 1 month ago

  • Status changed from Confirmed to Resolved
  • Assignee set to Go MAEDA
  • Resolution set to Fixed

Committed the fix. Thank you.

#4 Updated by Go MAEDA about 1 month ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF