Skip to content

Commit 3771688

Browse files
juno-nishizakiRedmine Patch Meetup
juno-nishizaki
authored and
Redmine Patch Meetup
committed
patched 14508.patch
1 parent 1fc0bf3 commit 3771688

4 files changed

Lines changed: 112 additions & 39 deletions

File tree

‎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
@@ -764,6 +764,75 @@ ul.properties li span {font-style:italic;}
764764

765765
#workflow_copy_form select { width: 200px; }
766766
table.transitions td.enabled {background: #bfb;}
767+
@media screen {
768+
table.fields_permissions td {
769+
min-width: 100px;
770+
}
771+
table.transitions td {
772+
min-width: 50px;
773+
}
774+
.sticky-table-wrapper {
775+
max-height: 500px;
776+
}
777+
}
778+
779+
.sticky-table-wrapper {
780+
table-layout: fixed;
781+
overflow-x: auto;
782+
margin-bottom: 1.2em;
783+
position: relative;
784+
}
785+
786+
.sticky-table-wrapper table {
787+
overflow-y: scroll;
788+
overflow-x: scroll;
789+
}
790+
.sticky-table-wrapper table tr.fixed-row td, .sticky-table-wrapper table tr.fixed-row th {
791+
position: -webkit-sticky;
792+
position: sticky;
793+
top: 0px;
794+
z-index: 10;
795+
}
796+
.sticky-table-wrapper table td.fixed-column, .sticky-table-wrapper table th.fixed-column {
797+
position: -webkit-sticky;
798+
position: sticky;
799+
left: 0px;
800+
z-index: 9;
801+
border-right: 1px solid #dfe8f1;
802+
}
803+
.sticky-table-wrapper table.workflows tr.fixed-row.status td {
804+
top: 23px;
805+
background-color: #fff;
806+
}
807+
.sticky-table-wrapper table.fields_permissions tr.fixed-row.group td {
808+
top: 40px;
809+
background-color: #fff;
810+
border-top: 1px solid #dfe8f1;
811+
}
812+
.sticky-table-wrapper table.transitions tr.fixed-row.status td:not(.fixed-column) {
813+
border-bottom: 1px solid #dfe8f1;
814+
}
815+
.sticky-table-wrapper table.workflows thead tr td.fixed-column, .sticky-table-wrapper table.workflows thead tr th.fixed-column {
816+
z-index: 11;
817+
white-space: nowrap;
818+
}
819+
.sticky-table-wrapper table.fields_permissions tr.fixed-row.group td.fixed-column {
820+
z-index: 11;
821+
white-space: nowrap;
822+
border-right: none;
823+
background-color: #fff;
824+
}
825+
.sticky-table-wrapper table tbody td.fixed-column, .sticky-table-wrapper table tbody th.fixed-column {
826+
background-color: #fff;
827+
}
828+
.sticky-table-wrapper table.workflows tr:nth-child(odd) td.fixed-column {
829+
background-color: #f6f7f8;
830+
}
831+
/* Only Safari */
832+
_::-webkit-full-page-media, _:future, :root .sticky-table-wrapper table.fields_permissions tr.fixed-row.group td {
833+
top: 0px;
834+
}
835+
767836
#workflow_form table select {font-size:90%; max-width:100px;}
768837
table.fields_permissions td.readonly {background:#ddd;}
769838
table.fields_permissions td.required {background:#d88;}
@@ -1814,6 +1883,7 @@ img.filecontent.image {background-image: url(../images/transparent.png);}
18141883
#wiki_add_attachment { display:none; }
18151884
.hide-when-print, .pagination ul.pages, .pagination .per-page { display: none !important; }
18161885
.autoscroll {overflow-x: visible;}
1886+
.sticky-table-wrapper table { overflow-x: visible; overflow-y: visible; }
18171887
table.list {margin-top:0.5em;}
18181888
table.list th, table.list td {border: 1px solid #aaa;}
18191889
}

0 commit comments

Comments
 (0)