Feature #9846

Provide the way for theme developers to set the row height in SCM history view

Added by Alex Pogodin over 10 years ago. Updated over 10 years ago.

Status:ClosedStart date:
Priority:NormalDue date:
Assignee:-% Done:


Target version:-


It's not correct to rely on the row height 20px set in source:branches/1.3-stable/public/stylesheets/application.css#L165, cause, for example on my system (using Gnome/Linux) only checkbox takes 15px, plus cell padding etc. yields 26px total row height.

It'll be very good if we give theme authors the way to pass their settings in the branchGraph function. The 20px value for ystep is hardcoded (source:branches/1.3-stable/public/javascripts/revision_graph.js#L29) now. I managed to override the entire function (!) in theme.js, but I consider it is the wrong way.

BTW. I think the script /javascripts/revision_graph.js is written in awfull manner: it polutes global namespace, uses window.onload assignment instead of adding load listener...

00.png - Graph is drawn incorrectly (36.9 KB) Alex Pogodin, 2011-12-20 15:15

Related issues

Duplicates Redmine - Defect #9832: Revision graph height should depend on height of rows in ... Closed


#1 Updated by Alex Pogodin over 10 years ago

As seen on the screenshot attached, graph nodes are not aligned with table rows

#2 Updated by Toshi MARUYAMA over 10 years ago

Try #9832 patch.

#3 Updated by Toshi MARUYAMA over 10 years ago

  • Status changed from New to Closed
  • Resolution set to Duplicate

#9832 fixed revision graph height by JavaScript.

Also available in: Atom PDF