Redmine

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...