Defect #8809

Table in timelog report overflows

Added by Hans Schmidt over 6 years ago. Updated almost 6 years ago.

Status:ClosedStart date:2011-07-14
Priority:NormalDue date:
Assignee:Jean-Philippe Lang% Done:

0%

Category:UI
Target version:1.2.3
Resolution:Fixed Affected version:1.2.1

Description

I think a screenshot says more than words: Spent_time_bug.PNG

The fix Spent_time_fix.PNG needs a <div> around the table.

File /{redmine_root}/app/views/time_entry_reports/report.rhtml

<p><%= l(:label_total) %>: <%= html_hours(l_hours(@total_hours)) %></p>
</div>

<% unless @hours.empty? %>

Line 41: + <div class="overflow-auto">

<table class="list" id="time-report">
<thead>
<tr>
<% @criterias.each do |criteria| %>
.
.
.
  <td class="hours"><%= html_hours("%.2f" % total) if total > 0 %></td>
  </tr>
</tbody>
</table>

Line 69: + </div>

<% other_formats_links do |f| %>
    <%= f.link_to 'CSV', :url => params %>
<% end %>
<% end %>
<% end %>

Add the CSS-Class "overflow-auto" to the theme

div.overflow-auto {
    overflow: auto;
}

Spent_time_bug.PNG (11 KB) Hans Schmidt, 2011-07-14 09:55

Spent_time_fix.PNG (8.71 KB) Hans Schmidt, 2011-07-14 09:55

Associated revisions

Revision 7982
Added by Jean-Philippe Lang almost 6 years ago

Adds an autoscroll div around time entry list and report (#8809).

History

#1 Updated by Toshi MARUYAMA over 6 years ago

  • Subject changed from Table in timelog report is to long to Table in timelog report is too long
  • Resolution deleted (Fixed)

#2 Updated by Etienne Massip over 6 years ago

  • Target version set to Candidate for next minor release
  • Affected version (unused) changed from 1.1.2 to 1.2.1
  • Affected version changed from 1.1.2 to 1.2.1

#3 Updated by Jean-Philippe Lang almost 6 years ago

  • Subject changed from Table in timelog report is too long to Table in timelog report overflows
  • Status changed from New to Resolved
  • Assignee set to Jean-Philippe Lang
  • Target version changed from Candidate for next minor release to 1.2.3
  • Resolution set to Fixed

Fixed in r7982.

#4 Updated by Jean-Philippe Lang almost 6 years ago

  • Status changed from Resolved to Closed

Merged in r7996.

Also available in: Atom PDF