Project

General

Profile

Actions

Patch #22278

closed

Revision Graph and Table should work with vertical-align: middle

Added by Gregor Schmidt over 8 years ago. Updated over 8 years ago.

Status:
Closed
Priority:
Normal
Category:
UI
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:

Description

The revision list and the Raphael based graph next to it, are currently assumed to be vertical-align: top, i.e. the dots representing the commits are always positioned at the top of the table cell. This is a limitation for theme authors. Planio e.g. restyled the table to use vertical-align: middle but now the dots are misplaced.

The patch attached inspects the currently used css properties to properly calculate the dot's position. The patch is targeted at current trunk r15251.


Files

0001-Check-for-vertical-align-when-computing-positions-in.patch (2.28 KB) 0001-Check-for-vertical-align-when-computing-positions-in.patch Gregor Schmidt, 2016-03-17 15:52
theme-without-patch.png (67.3 KB) theme-without-patch.png Gregor Schmidt, 2016-03-25 12:53
theme-with-patch.png (67.7 KB) theme-with-patch.png Gregor Schmidt, 2016-03-25 12:54
default-with-patch.png (67.1 KB) default-with-patch.png Gregor Schmidt, 2016-03-25 12:54
planio-with-patch.png (49.3 KB) planio-with-patch.png Gregor Schmidt, 2016-03-25 13:01
Actions #1

Updated by Go MAEDA over 8 years ago

  • Category changed from Themes to UI

Thanks for the patch. Could you show screenshots of the effect of this patch?

Actions #2

Updated by Gregor Schmidt over 8 years ago

Sure, here we go!

For "educational purpose" I've created a simple theme with the following application.css

@import url(../../../stylesheets/application.css);

table.changesets td {
  vertical-align: middle;
}

Using this theme and the current Redmine trunk, I get the following revision table

Using the patch, I get the following

while the default theme still looks as expected

This example theme does not make a lot of sense on its own. It was just created for testing. But e.g. Planio's new design uses vertical-align: middle for its tables. (Planio already has the patch applied.)

Actions #3

Updated by Go MAEDA over 8 years ago

  • Target version set to 3.2.2

Thanks, I understand.
Setting target version to 3.2.2.

Actions #4

Updated by Jean-Philippe Lang over 8 years ago

  • Status changed from New to Closed
  • Assignee set to Jean-Philippe Lang

Committed, thanks.

Actions

Also available in: Atom PDF