Use css pseudo-classes instead of cycle("odd", "even") (#15361).

Patch by Marius BALTEANU.

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

app/views/admin/info.html.erb

@@ -4,7 +4,7 @@
 
 <table class="list">
 <% @checklist.each do |label, result| %>
-  <tr class="<%= cycle 'odd', 'even' %>">
+  <tr>
     <td class="name"><%= label.is_a?(Symbol) ? l(label) : label %></td>
     <td class="tick"><span class="icon-only <%= (result ? 'icon-ok' : 'icon-error') %>"></span></td>
   </tr>

app/views/admin/plugins.html.erb

@@ -3,7 +3,7 @@
 <% if @plugins.any? %>
 <table class="list plugins">
     <% @plugins.each do |plugin| %>
-        <tr id="plugin-<%= plugin.id %>" class="<%= cycle('odd', 'even') %>">
+        <tr id="plugin-<%= plugin.id %>">
         <td class="name"><span class="name"><%= plugin.name %></span>
             <%= content_tag('span', plugin.description, :class => 'description') unless plugin.description.blank? %>
             <%= content_tag('span', link_to(plugin.url, plugin.url), :class => 'url') unless plugin.url.blank? %>

app/views/admin/projects.html.erb

@@ -27,7 +27,7 @@
   </tr></thead>
   <tbody>
 <% project_tree(@projects, :init_level => true) do |project, level| %>
-  <tr class="<%= cycle("odd", "even") %> <%= project.css_classes %> <%= level > 0 ? "idnt idnt-#{level}" : nil %>">
+  <tr class="<%= project.css_classes %> <%= level > 0 ? "idnt idnt-#{level}" : nil %>">
   <td class="name"><span><%= link_to_project_settings(project, {}, :title => project.short_description) %></span></td>
   <td><%= checked_image project.is_public? %></td>
   <td><%= format_date(project.created_on) %></td>

app/views/auth_sources/index.html.erb

@@ -14,7 +14,7 @@
   </tr></thead>
   <tbody>
 <% for source in @auth_sources %>
-  <tr id="auth-source-<%= source.id %>" class="<%= cycle("odd", "even") %>">
+  <tr id="auth-source-<%= source.id %>">
     <td class="name"><%= link_to(source.name, :action => 'edit', :id => source)%></td>
     <td><%= source.auth_method_name %></td>
     <td><%= source.host %></td>

app/views/boards/index.html.erb

@@ -9,7 +9,7 @@
   </tr></thead>
   <tbody>
 <% Board.board_tree(@boards) do |board, level| %>
-  <tr class="<%= cycle 'odd', 'even' %>">
+  <tr>
     <td class="name" style="padding-left: <%= level * 18 %>px;">
       <%= link_to board.name, project_board_path(board.project, board), :class => "board" %><br />
       <%=h board.description %>

app/views/boards/show.html.erb

@@ -35,7 +35,7 @@
   </tr></thead>
   <tbody>
   <% @topics.each do |topic| %>
-    <tr id="message-<%= topic.id %>" class="message <%= cycle 'odd', 'even' %> <%= topic.sticky? ? 'sticky' : '' %> <%= topic.locked? ? 'locked' : '' %>">
+    <tr id="message-<%= topic.id %>" class="message <%= topic.sticky? ? 'sticky' : '' %> <%= topic.locked? ? 'locked' : '' %>">
       <td class="subject icon <%= 'icon-sticky' if topic.sticky? %> <%= 'icon-locked' if topic.locked? %>"><%= link_to topic.subject, board_message_path(@board, topic) %></td>
       <td class="author"><%= link_to_user(topic.author) %></td>
       <td class="created_on"><%= format_time(topic.created_on) %></td>

app/views/custom_fields/_index.html.erb

@@ -12,7 +12,7 @@
   <tbody>
   <% (@custom_fields_by_type[tab[:name]] || []).sort.each do |custom_field| -%>
     <% back_url = custom_fields_path(:tab => tab[:name]) %>
-    <tr class="<%= cycle("odd", "even") %>">
+    <tr>
       <td class="name"><%= link_to custom_field.name, edit_custom_field_path(custom_field) %></td>
       <td><%= l(custom_field.format.label) %></td>
       <td><%= checked_image custom_field.is_required? %></td>
@@ -25,6 +25,6 @@
         <%= delete_link custom_field_path(custom_field) %>
       </td>
     </tr>
-  <% end; reset_cycle %>
+  <% end %>
   </tbody>
 </table>

app/views/email_addresses/_index.html.erb

@@ -1,7 +1,7 @@
 <% if @addresses.present? %>
   <table class="list email_addresses">
   <% @addresses.each do |address| %>
-    <tr class="<%= cycle("odd", "even") %>">
+    <tr>
       <td class="email"><%= address.address %></td>
       <td class="buttons">
         <%= toggle_email_address_notify_link(address) %>

app/views/enumerations/index.html.erb

@@ -16,7 +16,7 @@
     <th></th>
 </tr></thead>
 <% enumerations.each do |enumeration| %>
-<tr class="<%= cycle('odd', 'even') %>">
+<tr>
     <td class="name"><%= link_to enumeration, edit_enumeration_path(enumeration) %></td>
     <td class="tick"><%= checked_image enumeration.is_default? %></td>
     <td class="tick"><%= checked_image enumeration.active? %></td>
@@ -27,7 +27,6 @@
 </tr>
 <% end %>
 </table>
-<% reset_cycle %>
 <% else %>
 <p class="nodata"><%= l(:label_no_data) %></p>
 <% end %>

app/views/files/index.html.erb

@@ -26,7 +26,7 @@
   </tr>
   <% end -%>
   <% container.attachments.each do |file| %>
-  <tr class="file <%= cycle("odd", "even") %>">
+  <tr class="file">
     <td class="filename"><%= link_to_attachment file, :download => true, :title => file.description %></td>
     <td class="created_on"><%= format_time(file.created_on) %></td>
     <td class="filesize"><%= number_to_human_size(file.filesize) %></td>
@@ -37,8 +37,7 @@
                                          :data => {:confirm => l(:text_are_you_sure)}, :method => :delete) if delete_allowed %>
     </td>
   </tr>
-  <% end
-  reset_cycle %>
+  <% end %>
 <% end %>
   </tbody>
 </table>

app/views/groups/_users.html.erb

@@ -8,7 +8,7 @@
     </tr></thead>
     <tbody>
     <% @group.users.sort.each do |user| %>
-      <tr id="user-<%= user.id %>" class="<%= cycle 'odd', 'even' %>">
+      <tr id="user-<%= user.id %>">
         <td class="name"><%= link_to_user user %></td>
         <td class="buttons">
           <%= delete_link group_user_path(@group, :user_id => user), :remote => true %>

app/views/groups/index.html.erb

@@ -24,7 +24,7 @@
   </tr></thead>
   <tbody>
 <% @groups.each do |group| %>
-  <tr id="group-<%= group.id %>" class="<%= cycle 'odd', 'even' %> <%= "builtin" if group.builtin? %>">
+  <tr id="group-<%= group.id %>" class="<%= "builtin" if group.builtin? %>">
     <td class="name"><%= link_to group, edit_group_path(group) %></td>
     <td class="user_count"><%= (@user_count_by_group_id[group.id] || 0) unless group.builtin? %></td>
     <td class="buttons"><%= delete_link group unless group.builtin? %></td>

app/views/issue_statuses/index.html.erb

@@ -16,7 +16,7 @@
   </tr></thead>
   <tbody>
 <% for status in @issue_statuses %>
-  <tr class="<%= cycle("odd", "even") %>">
+  <tr>
   <td class="name"><%= link_to status.name, edit_issue_status_path(status) %></td>
   <% if Issue.use_status_for_done_ratio? %>
   <td><%= status.default_done_ratio %></td>

app/views/issues/_changesets.html.erb

@@ -1,5 +1,5 @@
 <% changesets.each do |changeset| %>
-    <div class="changeset <%= cycle('odd', 'even') %>">
+    <div class="changeset">
     <p><%= link_to_revision(changeset, changeset.repository,
                             :text => "#{l(:label_revision)} #{changeset.format_identifier}") %>
         <% if changeset.filechanges.any? && User.current.allowed_to?(:browse_repository, changeset.project) %>

app/views/issues/_list.html.erb

@@ -1,7 +1,7 @@
 <%= form_tag({}, :data => {:cm_url => issues_context_menu_path}) do -%>
 <%= hidden_field_tag 'back_url', url_for(:params => request.query_parameters), :id => nil %>
 <div class="autoscroll">
-<table class="list issues <%= sort_css_classes %>">
+<table class="list issues odd-even <%= sort_css_classes %>">
   <thead>
     <tr>
       <th class="checkbox hide-when-print">

app/views/issues/_list_simple.html.erb

@@ -9,7 +9,7 @@
     </tr></thead>
     <tbody>
     <% for issue in issues %>
-    <tr id="issue-<%= h(issue.id) %>" class="hascontextmenu <%= cycle('odd', 'even') %> <%= issue.css_classes %>">
+    <tr id="issue-<%= h(issue.id) %>" class="hascontextmenu <%= issue.css_classes %>">
       <td class="id">
         <%= check_box_tag("ids[]", issue.id, false, :style => 'display:none;', :id => nil) %>
         <%= link_to("#{issue.tracker} ##{issue.id}", issue_path(issue)) %>

app/views/principal_memberships/_index.html.erb

@@ -13,7 +13,7 @@
   <tbody>
   <% memberships.each do |membership| %>
   <% next if membership.new_record? %>
-  <tr id="member-<%= membership.id %>" class="<%= cycle 'odd', 'even' %> class">
+  <tr id="member-<%= membership.id %>" class="class">
   <td class="project name">
     <%= link_to_project membership.project %>
   </td>
@@ -31,7 +31,7 @@
   </td>
       <%= call_table_row_hook principal, membership %>
   </tr>
-  <% end; reset_cycle %>
+  <% end %>
   </tbody>
 </table>
 <% else %>

app/views/projects/settings/_activities.html.erb

@@ -12,7 +12,7 @@
 
   <% @project.activities(true).each do |enumeration| %>
   <%= fields_for "enumerations[#{enumeration.id}]", enumeration do |ff| %>
-  <tr class="<%= cycle('odd', 'even') %>">
+  <tr>
     <td class="name">
       <%= ff.hidden_field :parent_id, :value => enumeration.id unless enumeration.project %>
       <%= enumeration %>

app/views/projects/settings/_boards.html.erb

@@ -8,7 +8,7 @@
     <div class="table-list-cell"><%= l(:label_board) %></div>
   </div>
   <%= render_boards_tree(@project.boards) do |board, level| %>
-  <div class="table-list-row <%= cycle 'odd', 'even' %>">
+  <div class="table-list-row">
     <div class="table-list-cell name" style="padding-left: <%= 2 + level * 16 %>px">
       <%= link_to board.name, project_board_path(@project, board) %>
     </div>

app/views/projects/settings/_issue_categories.html.erb

@@ -10,7 +10,7 @@
   <tbody>
 <% for category in @project.issue_categories %>
   <% unless category.new_record? %>
-  <tr class="<%= cycle 'odd', 'even' %>">
+  <tr>
     <td class="name"><%= category.name  %></td>
     <td><%= category.assigned_to.name if category.assigned_to %></td>
     <td class="buttons">

app/views/projects/settings/_members.html.erb

@@ -15,7 +15,7 @@
   <tbody>
   <% members.each do |member| %>
   <% next if member.new_record? %>
-  <tr id="member-<%= member.id %>" class="<%= cycle 'odd', 'even' %> member">
+  <tr id="member-<%= member.id %>" class="member">
   <td class="name icon icon-<%= member.principal.class.name.downcase %>"><%= link_to_user member.principal %></td>
   <td class="roles">
     <span id="member-<%= member.id %>-roles"><%= member.roles.sort.collect(&:to_s).join(', ') %></span>
@@ -32,7 +32,7 @@
   </td>
   <%= call_hook(:view_projects_settings_members_table_row, { :project => @project, :member => member}) %>
   </tr>
-<% end; reset_cycle %>
+<% end %>
   </tbody>
 </table>
 <% else %>

app/views/projects/settings/_repositories.html.erb

@@ -15,9 +15,9 @@
   </thead>
   <tbody>
   <% @project.repositories.sort.each do |repository| %>
-    <tr class="<%= cycle 'odd', 'even' %>">
+    <tr>
       <td class="name">
-      <%= link_to repository.identifier, 
+      <%= link_to repository.identifier,
             {:controller => 'repositories', :action => 'show',:id => @project, :repository_id => repository.identifier_param} if repository.identifier.present? %>
       </td>
       <td><%= checked_image repository.is_default? %></td>

app/views/projects/settings/_versions.html.erb

@@ -25,7 +25,7 @@
     </tr></thead>
   <tbody>
 <% @versions.sort.each do |version| %>
-    <tr class="version <%= cycle 'odd', 'even' %> <%=h version.status %> <%= 'shared' if version.project != @project %>">
+    <tr class="version <%=h version.status %> <%= 'shared' if version.project != @project %>">
     <td class="name <%= 'icon icon-shared' if version.project != @project %>"><%= link_to_version version %></td>
     <td class="date"><%= format_date(version.effective_date) %></td>
     <td class="description"><%= version.description %></td>
@@ -39,7 +39,7 @@
       <% end %>
      </td>
     </tr>
-<% end; reset_cycle %>
+<% end %>
     </tbody>
 </table>
 <% else %>

app/views/projects/show.html.erb

@@ -49,7 +49,7 @@
       </thead>
       <tbody>
       <% @trackers.each do |tracker| %>
-        <tr class="<%= cycle("odd", "even") %>">
+        <tr>
           <td class="name">
             <%= link_to tracker.name, project_issues_path(@project, :set_filter => 1, :tracker_id => tracker.id) %>
           </td>

app/views/queries/index.html.erb

@@ -9,7 +9,7 @@
 <% else %>
   <table class="list">
   <% @queries.each do |query| %>
-    <tr class="<%= cycle('odd', 'even') %>">
+    <tr>
       <td class="name">
         <%= link_to query.name, :controller => 'issues', :action => 'index', :project_id => @project, :query_id => query %>
       </td>

app/views/reports/_details.html.erb

@@ -13,7 +13,7 @@
 </tr></thead>
 <tbody>
 <% for row in rows %>
-<tr class="<%= cycle("odd", "even") %>">
+<tr>
   <td class="name"><%= link_to row.name, aggregate_path(@project, field_name, row) %></td>
   <% for status in @statuses %>
     <td><%= aggregate_link data, { field_name => row.id, "status_id" => status.id }, aggregate_path(@project, field_name, row, :status_id => status.id) %></td>
@@ -25,5 +25,4 @@
 <% end %>
 </tbody>
 </table>
-<% end
-  reset_cycle %>
+<% end %>

app/views/reports/_simple.html.erb

@@ -10,7 +10,7 @@
 </tr></thead>
 <tbody>
 <% for row in rows %>
-<tr class="<%= cycle("odd", "even") %>">
+<tr>
   <td class="name"><%= link_to row.name, aggregate_path(@project, field_name, row) %></td>
   <td><%= aggregate_link data, { field_name => row.id, "closed" => 0 }, aggregate_path(@project, field_name, row, :status_id => "o") %></td>
   <td><%= aggregate_link data, { field_name => row.id, "closed" => 1 }, aggregate_path(@project, field_name, row, :status_id => "c") %></td>
@@ -19,5 +19,4 @@
 <% end %>
 </tbody>
 </table>
-<% end
-  reset_cycle %>
+<% end %>

app/views/repositories/_revisions.html.erb

@@ -33,7 +33,7 @@ end %>
 <% show_diff = revisions.size > 1 %>
 <% line_num = 1 %>
 <% revisions.each do |changeset| %>
-<tr class="changeset <%= cycle 'odd', 'even' %>">
+<tr class="changeset">
 <% id_style = (show_revision_graph ? "padding-left:#{(graph_space + 1) * 20}px" : nil) %>
 <%= content_tag(:td, :class => 'id', :style => id_style) do %>
   <%= link_to_revision(changeset, @repository) %>

app/views/repositories/committers.html.erb

@@ -17,7 +17,7 @@
 <tbody>
 <% i = 0 -%>
 <% @committers.each do |committer, user_id| -%>
-  <tr class="<%= cycle 'odd', 'even' %>">
+  <tr>
     <td><%= committer %></td>
     <td>
       <%= hidden_field_tag "committers[#{i}][]", committer, :id => nil %>

app/views/roles/_form.html.erb

@@ -91,7 +91,7 @@
       <% end %>
     </tr>
     <% Tracker.sorted.all.each do |tracker| %>
-    <tr class="<%= cycle("odd", "even") %>">
+    <tr>
       <td class="name"><%= tracker.name %></td>
       <% permissions.each do |permission| %>
       <td class="<%= "#{permission}_shown" %>"><%= check_box_tag "role[permissions_tracker_ids][#{permission}][]",

app/views/roles/index.html.erb

@@ -12,7 +12,7 @@
   </tr></thead>
   <tbody>
 <% for role in @roles %>
-  <tr class="<%= cycle("odd", "even") %> <%= role.builtin? ? "builtin" : "givable" %>">
+  <tr class="<%= role.builtin? ? "builtin" : "givable" %>">
   <td class="name"><%= content_tag(role.builtin? ? 'em' : 'span', link_to(role.name, edit_role_path(role))) %></td>
   <td class="buttons">
     <%= reorder_handle(role) unless role.builtin? %>

app/views/roles/permissions.html.erb

@@ -34,7 +34,7 @@
     <% end %>
     <% perms_by_module[mod].each do |permission| %>
         <% humanized_perm_name = l_or_humanize(permission.name, :prefix => 'permission_') %>
-        <tr class="<%= cycle('odd', 'even') %> permission-<%= permission.name %>">
+        <tr class="permission-<%= permission.name %>">
         <td class="name">
             <%= link_to_function('',
                                  "toggleCheckboxesBySelector('.permission-#{permission.name} input')",

app/views/timelog/_list.html.erb

@@ -1,7 +1,7 @@
 <%= form_tag({}, :data => {:cm_url => time_entries_context_menu_path}) do -%>
 <%= hidden_field_tag 'back_url', url_for(:params => request.query_parameters), :id => nil %>
 <div class="autoscroll">
-<table class="list time-entries">
+<table class="list odd-even time-entries">
 <thead>
   <tr>
     <th class="checkbox hide-when-print">

app/views/timelog/_report_criteria.html.erb

@@ -1,7 +1,7 @@
 <% @report.hours.collect {|h| h[criterias[level]].to_s}.uniq.each do |value| %>
 <% hours_for_value = select_hours(hours, criterias[level], value) -%>
 <% next if hours_for_value.empty? -%>
-<tr class="<%= cycle('odd', 'even') %> <%= criterias.length > level+1 ? 'subtotal' : 'last-level' %>">
+<tr class="<%= criterias.length > level+1 ? 'subtotal' : 'last-level' %>">
 <%= ("<td></td>" * level).html_safe %>
 <td class="name"><%= format_criteria_value(@report.available_criteria[criterias[level]], value) %></td>
 <%= ("<td></td>" * (criterias.length - level - 1)).html_safe -%>

app/views/trackers/fields.html.erb

@@ -25,7 +25,7 @@
         </td>
       </tr>
       <% Tracker::CORE_FIELDS.each do |field| %>
-      <tr class="<%= cycle("odd", "even") %>">
+      <tr>
         <% field_name = l("field_#{field}".sub(/_id$/, '')) %>
         <td class="name">
           <%= link_to_function('', "toggleCheckboxesBySelector('input.core-field-#{field}')",
@@ -49,7 +49,7 @@
           </td>
         </tr>
         <% @custom_fields.each do |field| %>
-        <tr class="<%= cycle("odd", "even") %>">
+        <tr>
           <td class="name">
             <%= link_to_function('', "toggleCheckboxesBySelector('input.custom-field-#{field.id}')",
                                  :title => "#{l(:button_check_all)}/#{l(:button_uncheck_all)}",

app/views/trackers/index.html.erb

@@ -13,7 +13,7 @@
   </tr></thead>
   <tbody>
 <% for tracker in @trackers %>
-  <tr class="<%= cycle("odd", "even") %>">
+  <tr>
   <td class="name"><%= link_to tracker.name, edit_tracker_path(tracker) %></td>
   <td>
     <% unless tracker.workflow_rules.exists? %>

app/views/users/index.html.erb

@@ -37,7 +37,7 @@
   </tr></thead>
   <tbody>
 <% for user in @users -%>
-  <tr class="<%= user.css_classes %> <%= cycle("odd", "even") %>">
+  <tr class="<%= user.css_classes %>">
   <td class="username"><%= avatar(user, :size => "14") %><%= link_to user.login, edit_user_path(user) %></td>
   <td class="firstname"><%= user.firstname %></td>
   <td class="lastname"><%= user.lastname %></td>

app/views/wiki/history.html.erb

@@ -19,7 +19,7 @@
 <% show_diff = @versions.size > 1 %>
 <% line_num = 1 %>
 <% @versions.each do |ver| %>
-<tr class="wiki-page-version <%= cycle("odd", "even") %>">
+<tr class="wiki-page-version">
     <td class="id"><%= link_to ver.version, :action => 'show', :id => @page.title, :project_id => @page.project, :version => ver.version %></td>
     <td class="checkbox"><%= radio_button_tag('version', ver.version, (line_num==1), :id => "cb-#{line_num}", :onclick => "$('#cbto-#{line_num+1}').prop('checked', true);") if show_diff && (line_num < @versions.size) %></td>
     <td class="checkbox"><%= radio_button_tag('version_from', ver.version, (line_num==2), :id => "cbto-#{line_num}") if show_diff && (line_num > 1) %></td>

app/views/workflows/_form.html.erb

@@ -24,7 +24,7 @@
 <tbody>
   <% for old_status in [nil] + @statuses %>
   <% next if old_status.nil? && name != 'always' %>
-  <tr class="<%= cycle("odd", "even") %>">
+  <tr>
     <td class="name">
       <%= link_to_function('', "toggleCheckboxesBySelector('table.transitions-#{name} input.old-status-#{old_status.try(:id) || 0}')",
                            :title => "#{l(:button_check_all)}/#{l(:button_uncheck_all)}",

app/views/workflows/index.html.erb

@@ -17,7 +17,7 @@
 </thead>
 <tbody>
 <% @trackers.each do |tracker| -%>
-<tr class="<%= cycle('odd', 'even') %>">
+<tr>
   <td class="name"><%= tracker.name %></td>
   <% @roles.each do |role| -%>
   <% count = @workflow_counts[[tracker.id, role.id]] || 0 %>

app/views/workflows/permissions.html.erb

@@ -60,7 +60,7 @@
         </td>
       </tr>
       <% @fields.each do |field, name| %>
-      <tr class="<%= cycle("odd", "even") %>">
+      <tr>
         <td class="name">
           <%= name %> <%= content_tag('span', '*', :class => 'required') if field_required?(field) %>
         </td>
@@ -80,7 +80,7 @@
           </td>
         </tr>
         <% @custom_fields.each do |field| %>
-        <tr class="<%= cycle("odd", "even") %>">
+        <tr>
           <td class="name">
             <%= field.name %> <%= content_tag('span', '*', :class => 'required') if field_required?(field) %>
           </td>

public/javascripts/application.js

@@ -697,10 +697,6 @@ function beforeShowDatePicker(input, inst) {
           type: 'put',
           dataType: 'script',
           data: data,
-          success: function(data){
-            sortable.children(":even").removeClass("even").addClass("odd");
-            sortable.children(":odd").removeClass("odd").addClass("even");
-          },
           error: function(jqXHR, textStatus, errorThrown){
             alert(jqXHR.status);
             sortable.sortable("cancel");

public/stylesheets/application.css

@@ -320,6 +320,8 @@ table.permissions td.role {color:#999;font-size:90%;font-weight:normal !importan
 
 tr.wiki-page-version td.updated_on, tr.wiki-page-version td.author {text-align:center;}
 
+div.mypage-box table.time-entries tr.time-entry { background-color: #fff; }
+div.mypage-box table.time-entries tr.odd { background-color:#f6f7f8; }
 tr.time-entry { text-align: center; white-space: nowrap; }
 tr.time-entry td.issue, tr.time-entry td.comments, tr.time-entry td.subject, tr.time-entry td.activity { text-align: left; white-space: normal; }
 td.hours { text-align: right; font-weight: bold; padding-right: 0.5em; }
@@ -333,7 +335,7 @@ table.plugins span.url { display: block; font-size: 0.9em; }
 
 table.list.enumerations {table-layout: fixed; margin-bottom: 2em;}
 
-tr.group td { padding: 0.8em 0 0.5em 0.3em; border-bottom: 1px solid #ccc; text-align:left; }
+tr.group td { padding: 0.8em 0 0.5em 0.3em; border-bottom: 1px solid #ccc; text-align:left; background-color: #fff;}
 tr.group span.name {font-weight:bold;}
 tr.group span.count {font-weight:bold; position:relative; top:-1px; color:#fff; font-size:10px; background:#9DB9D5; padding:0px 6px 1px 6px; border-radius:3px; margin-left:4px;}
 tr.group span.totals {color: #aaa; font-size: 80%;}
@@ -346,8 +348,9 @@ table.list tbody tr:hover { background-color:#ffffdd; }
 table.list tbody tr.group:hover { background-color:inherit; }
 table td {padding:2px;}
 table p {margin:0;}
-.odd {background-color:#f6f7f8;}
-.even {background-color: #fff;}
+
+table.list tbody tr:nth-child(odd), table.list.odd-even tbody tr.odd, #issue-changesets div.changeset:nth-child(odd) { background-color:#f6f7f8; }
+table.list tbody tr:nth-child(even), table.list.odd-even tbody tr.even, #issue-changesets div.changeset:nth-child(even) { background-color: #fff; }
 
 tr.builtin td.name {font-style:italic;}