Project

General

Profile

Actions

Patch #14068

closed

Increase base height of author lines on 'Commits per author' graph

Added by Mischa The Evil almost 11 years ago. Updated almost 11 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
SCM
Target version:
-
Start date:
Due date:
% Done:

0%

Estimated time:

Description

In r11723 for #13486 the height of "Commits per author" graph is made flexible. This is done by taking a base height for each author (15) and multiplying that with the number of authors.

The current base height value seems to be a bit on the low side which makes the graph pretty difficult to read. I'll add some examples further on.

I tried several values and came to the conclusion that a value of 301 is working best for me. What do you think?

Examples
Repo Base height 15 Base height 30
Redmine rm_15.png rm_30.png
Apache httpd httpd_15.png httpd_30.png

1 in source:/trunk/app/controllers/repositories_controller.rb@11845#L414:

Index: app/controllers/repositories_controller.rb
===================================================================
--- app/controllers/repositories_controller.rb  (revision 11845)
+++ app/controllers/repositories_controller.rb  (working copy)
@@ -411,7 +411,7 @@
     fields = fields.collect {|c| c.gsub(%r{<.+@.+>}, '') }

     graph = SVG::Graph::BarHorizontal.new(
-      :height => 15 * commits_data.length,
+      :height => 30 * commits_data.length,
       :width => 800,
       :fields => fields,
       :stack => :side,


Files

rm_15.png (7.1 KB) rm_15.png Redmine repo graph with base height of 15 Mischa The Evil, 2013-05-16 02:39
rm_30.png (11.2 KB) rm_30.png Redmine repo graph with base height of 30 Mischa The Evil, 2013-05-16 02:39
httpd_15.png (15.4 KB) httpd_15.png Apache httpd repo graph with base height of 15 Mischa The Evil, 2013-05-16 02:40
httpd_30.png (25.3 KB) httpd_30.png Apache httpd repo graph with base height of 30 Mischa The Evil, 2013-05-16 02:40

Related issues

Related to Redmine - Defect #1983: statistics get rather cramped with more than 15 or so contributersClosed2008-10-03

Actions
Actions #1

Updated by Toshi MARUYAMA almost 11 years ago

  • Tracker changed from Feature to Patch
  • Status changed from New to Closed

I committed r11847 which changes from 15 to 30, thanks.

Actions

Also available in: Atom PDF