Skip to content

Commit 5292f60

Browse files
juno-nishizakiRedmine Patch Meetup
juno-nishizaki
authored and
Redmine Patch Meetup
committed
patched 14508.patch
1 parent 8d061a1 commit 5292f60

File tree

4 files changed

+112
-39
lines changed

4 files changed

+112
-39
lines changed

‎app/views/workflows/_form.html.erb

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,16 +1,16 @@
11
<table class="list workflows transitions transitions-<%= name %>">
22
<thead>
3-
<tr>
4-
<th>
3+
<tr class='fixed-row'>
4+
<th class='fixed-column'>
55
<%= link_to_function('', "toggleCheckboxesBySelector('table.transitions-#{name} input[type=checkbox]:not(:disabled)')",
66
:title => "#{l(:button_check_all)}/#{l(:button_uncheck_all)}",
77
:class => 'no-tooltip icon-only icon-checked') %>
88
<%=l(:label_current_status)%>
99
</th>
1010
<th colspan="<%= @statuses.length %>"><%=l(:label_new_statuses_allowed)%></th>
1111
</tr>
12-
<tr>
13-
<td></td>
12+
<tr class='fixed-row status'>
13+
<td class='fixed-column'></td>
1414
<% for new_status in @statuses %>
1515
<td style="width:<%= 75 / @statuses.size %>%;">
1616
<%= link_to_function('', "toggleCheckboxesBySelector('table.transitions-#{name} input[type=checkbox]:not(:disabled).new-status-#{new_status.id}')",
@@ -26,7 +26,7 @@
2626
<% for old_status in [nil] + @statuses %>
2727
<% next if old_status.nil? && name != 'always' %>
2828
<tr>
29-
<td class="name">
29+
<td class="name fixed-column">
3030
<%= link_to_function('', "toggleCheckboxesBySelector('table.transitions-#{name} input[type=checkbox]:not(:disabled).old-status-#{old_status.try(:id) || 0}')",
3131
:title => "#{l(:button_check_all)}/#{l(:button_uncheck_all)}",
3232
:class => 'no-tooltip icon-only icon-checked') %>

‎app/views/workflows/edit.html.erb

Lines changed: 20 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -33,28 +33,28 @@
3333

3434
<% if @trackers && @roles && @statuses.any? %>
3535
<%= form_tag workflows_path, method: :patch, id: 'workflow_form' do %>
36-
<%= @trackers.map {|tracker| hidden_field_tag 'tracker_id[]', tracker.id, :id => nil}.join.html_safe %>
37-
<%= @roles.map {|role| hidden_field_tag 'role_id[]', role.id, :id => nil}.join.html_safe %>
38-
<%= hidden_field_tag 'used_statuses_only', params[:used_statuses_only], :id => nil %>
39-
<div class="autoscroll">
36+
<div class="sticky-table-wrapper">
37+
<%= @trackers.map {|tracker| hidden_field_tag 'tracker_id[]', tracker.id, :id => nil}.join.html_safe %>
38+
<%= @roles.map {|role| hidden_field_tag 'role_id[]', role.id, :id => nil}.join.html_safe %>
39+
<%= hidden_field_tag 'used_statuses_only', params[:used_statuses_only], :id => nil %>
4040
<%= render :partial => 'form', :locals => {:name => 'always', :workflows => @workflows['always']} %>
41-
42-
<fieldset class="collapsible" style="padding: 0; margin-top: 0.5em;">
43-
<legend onclick="toggleFieldset(this);" class="icon icon-collapsed"><%= l(:label_additional_workflow_transitions_for_author) %></legend>
44-
<div id="author_workflows" style="margin: 0.5em 0 0.5em 0;">
45-
<%= render :partial => 'form', :locals => {:name => 'author', :workflows => @workflows['author']} %>
46-
</div>
47-
</fieldset>
48-
<%= javascript_tag "hideFieldset($('#author_workflows'))" unless @workflows['author'].present? %>
49-
50-
<fieldset class="collapsible" style="padding: 0;">
51-
<legend onclick="toggleFieldset(this);" class="icon icon-collapsed"><%= l(:label_additional_workflow_transitions_for_assignee) %></legend>
52-
<div id="assignee_workflows" style="margin: 0.5em 0 0.5em 0;">
53-
<%= render :partial => 'form', :locals => {:name => 'assignee', :workflows => @workflows['assignee']} %>
54-
</div>
55-
</fieldset>
56-
<%= javascript_tag "hideFieldset($('#assignee_workflows'))" unless @workflows['assignee'].present? %>
5741
</div>
42+
43+
<fieldset class="collapsible" style="padding: 0; margin-top: 0.5em;">
44+
<legend onclick="toggleFieldset(this);" class="icon icon-collapsed"><%= l(:label_additional_workflow_transitions_for_author) %></legend>
45+
<div id="author_workflows" class="sticky-table-wrapper" style="margin: 0.5em 0 0.5em 0;">
46+
<%= render :partial => 'form', :locals => {:name => 'author', :workflows => @workflows['author']} %>
47+
</div>
48+
</fieldset>
49+
<%= javascript_tag "hideFieldset($('#author_workflows'))" unless @workflows['author'].present? %>
50+
51+
<fieldset class="collapsible" style="padding: 0;">
52+
<legend onclick="toggleFieldset(this);" class="icon icon-collapsed"><%= l(:label_additional_workflow_transitions_for_assignee) %></legend>
53+
<div id="assignee_workflows" class="sticky-table-wrapper" style="margin: 0.5em 0 0.5em 0;">
54+
<%= render :partial => 'form', :locals => {:name => 'assignee', :workflows => @workflows['assignee']} %>
55+
</div>
56+
</fieldset>
57+
<%= javascript_tag "hideFieldset($('#assignee_workflows'))" unless @workflows['assignee'].present? %>
5858
<%= submit_tag l(:button_save) %>
5959
<% end %>
6060
<% end %>

‎app/views/workflows/permissions.html.erb

Lines changed: 17 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -34,16 +34,17 @@
3434
<%= @trackers.map {|tracker| hidden_field_tag 'tracker_id[]', tracker.id, :id => nil}.join.html_safe %>
3535
<%= @roles.map {|role| hidden_field_tag 'role_id[]', role.id, :id => nil}.join.html_safe %>
3636
<%= hidden_field_tag 'used_statuses_only', params[:used_statuses_only], :id => nil %>
37-
<div class="autoscroll">
37+
<div class="sticky-table-wrapper">
3838
<table class="list workflows fields_permissions">
3939
<thead>
40-
<tr>
41-
<th>
40+
<tr class='fixed-row'>
41+
<th class='fixed-column'>
42+
<%=l(:label_issue_status)%>
4243
</th>
43-
<th colspan="<%= @statuses.length %>"><%=l(:label_issue_status)%></th>
44+
<th colspan="<%= @statuses.length %>"></th>
4445
</tr>
45-
<tr>
46-
<td></td>
46+
<tr class='fixed-row status'>
47+
<td class='fixed-column'></td>
4748
<% for status in @statuses %>
4849
<td style="width:<%= 75 / @statuses.size %>%;">
4950
<%= status.name %>
@@ -52,15 +53,16 @@
5253
</tr>
5354
</thead>
5455
<tbody>
55-
<tr class="group open">
56-
<td colspan="<%= @statuses.size + 1 %>">
57-
<span class="expander icon icon-expanded" onclick="toggleRowGroup(this);">&nbsp;</span>
56+
<tr class="group open fixed-row">
57+
<td class='fixed-column'>
58+
<span class="expander icon icon-expanded" onclick="toggleRowGroup(this);">&nbsp;</span>
5859
<%= l(:field_core_fields) %>
5960
</td>
61+
<td colspan="<%= @statuses.size %>"></td>
6062
</tr>
6163
<% @fields.each do |field, name| %>
6264
<tr>
63-
<td class="name">
65+
<td class="name fixed-column">
6466
<%= name %> <%= content_tag('span', '*', :class => 'required') if field_required?(field) %>
6567
</td>
6668
<% for status in @statuses -%>
@@ -72,15 +74,16 @@
7274
</tr>
7375
<% end %>
7476
<% if @custom_fields.any? %>
75-
<tr class="group open">
76-
<td colspan="<%= @statuses.size + 1 %>">
77-
<span class="expander icon icon-expanded" onclick="toggleRowGroup(this);">&nbsp;</span>
77+
<tr class="group open fixed-row">
78+
<td class='fixed-column'>
79+
<span class="expander icon icon-expended" onclick="toggleRowGroup(this);">&nbsp;</span>
7880
<%= l(:label_custom_field_plural) %>
7981
</td>
82+
<td colspan="<%= @statuses.size %>"></td>
8083
</tr>
8184
<% @custom_fields.each do |field| %>
8285
<tr>
83-
<td class="name">
86+
<td class="name fixed-column">
8487
<%= field.name %> <%= content_tag('span', '*', :class => 'required') if field_required?(field) %>
8588
</td>
8689
<% for status in @statuses -%>

‎public/stylesheets/application.css

Lines changed: 70 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -749,6 +749,75 @@ ul.properties li span {font-style:italic;}
749749

750750
#workflow_copy_form select { width: 200px; }
751751
table.transitions td.enabled {background: #bfb;}
752+
@media screen {
753+
table.fields_permissions td {
754+
min-width: 100px;
755+
}
756+
table.transitions td {
757+
min-width: 50px;
758+
}
759+
.sticky-table-wrapper {
760+
max-height: 500px;
761+
}
762+
}
763+
764+
.sticky-table-wrapper {
765+
table-layout: fixed;
766+
overflow-x: auto;
767+
margin-bottom: 1.2em;
768+
position: relative;
769+
}
770+
771+
.sticky-table-wrapper table {
772+
overflow-y: scroll;
773+
overflow-x: scroll;
774+
}
775+
.sticky-table-wrapper table tr.fixed-row td, .sticky-table-wrapper table tr.fixed-row th {
776+
position: -webkit-sticky;
777+
position: sticky;
778+
top: 0px;
779+
z-index: 10;
780+
}
781+
.sticky-table-wrapper table td.fixed-column, .sticky-table-wrapper table th.fixed-column {
782+
position: -webkit-sticky;
783+
position: sticky;
784+
left: 0px;
785+
z-index: 9;
786+
border-right: 1px solid #dfe8f1;
787+
}
788+
.sticky-table-wrapper table.workflows tr.fixed-row.status td {
789+
top: 23px;
790+
background-color: #fff;
791+
}
792+
.sticky-table-wrapper table.fields_permissions tr.fixed-row.group td {
793+
top: 40px;
794+
background-color: #fff;
795+
border-top: 1px solid #dfe8f1;
796+
}
797+
.sticky-table-wrapper table.transitions tr.fixed-row.status td:not(.fixed-column) {
798+
border-bottom: 1px solid #dfe8f1;
799+
}
800+
.sticky-table-wrapper table.workflows thead tr td.fixed-column, .sticky-table-wrapper table.workflows thead tr th.fixed-column {
801+
z-index: 11;
802+
white-space: nowrap;
803+
}
804+
.sticky-table-wrapper table.fields_permissions tr.fixed-row.group td.fixed-column {
805+
z-index: 11;
806+
white-space: nowrap;
807+
border-right: none;
808+
background-color: #fff;
809+
}
810+
.sticky-table-wrapper table tbody td.fixed-column, .sticky-table-wrapper table tbody th.fixed-column {
811+
background-color: #fff;
812+
}
813+
.sticky-table-wrapper table.workflows tr:nth-child(odd) td.fixed-column {
814+
background-color: #f6f7f8;
815+
}
816+
/* Only Safari */
817+
_::-webkit-full-page-media, _:future, :root .sticky-table-wrapper table.fields_permissions tr.fixed-row.group td {
818+
top: 0px;
819+
}
820+
752821
#workflow_form table select {font-size:90%; max-width:100px;}
753822
table.fields_permissions td.readonly {background:#ddd;}
754823
table.fields_permissions td.required {background:#d88;}
@@ -1805,6 +1874,7 @@ img.filecontent.image {background-image: url(../images/transparent.png);}
18051874
#wiki_add_attachment { display:none; }
18061875
.hide-when-print, .pagination ul.pages, .pagination .per-page { display: none !important; }
18071876
.autoscroll {overflow-x: visible;}
1877+
.sticky-table-wrapper table { overflow-x: visible; overflow-y: visible; }
18081878
table.list {margin-top:0.5em;}
18091879
table.list th, table.list td {border: 1px solid #aaa;}
18101880
}

0 commit comments

Comments
 (0)