Fix cell semantics

The last cell is actually not the row header. Thus, 'th' must be
replaced by 'td' to restore table semantics.
pull/6827/head
Hagen Schink 11 years ago
parent 5d9e71712a
commit cf5b84a6c8
  1. 8
      lib/assets/stylesheets/reporting_engine/reporting.css.erb
  2. 2
      lib/widget/table/report_table.rb

@ -81,6 +81,14 @@ Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
text-align: right;
}
.report .odd td.inner {
background-color: #e8e8e8;
}
.report .even td.inner {
background-color: #e3e3e3;
}
.report tr.even:hover .inner,
.report tr.even:hover .bottom,
.report tr.even:hover .empty,

@ -33,7 +33,7 @@ class Widget::Table::ReportTable < Widget::Table
@walker.for_final_row do |row, cells|
html = "<th scope='row' class='normal inner left'>#{show_row row}#{debug_fields(row)}</th>"
html << cells.join
html << "<th class='normal inner right'>#{show_result(row)}#{debug_fields(row)}</th>"
html << "<td class='normal inner right'>#{show_result(row)}#{debug_fields(row)}</th>"
html.html_safe
end

Loading…
Cancel
Save