[PATCH 25/51] patch-list: Re-design the main list of patches with Bootsrap

Finucane, Stephen stephen.finucane at intel.com
Thu Sep 10 01:13:11 AEST 2015


> From: Belén Barros Peña <belen.barros.pena at intel.com>
> 
> A bit more spacing everywhere. Also opted for the highlight on hover
> instead of the alternating row background color.
> 
> v2: Squash the patch fixing the unit tests (Jeremy Kerr)
> 
> Signed-off-by: Belén Barros Pena <belen.barros.pena at intel.com>
> Signed-off-by: Damien Lespiau <damien.lespiau at intel.com>

Looks good but I can imagine this being contentious. This change means we can display a lot less information on the screen just so things look a little prettier. I'm hesitant about this, but I'll allow it.

If I may, though, I'd like to make suggest one change that would make a huge difference though: monospace font for the subject. A monospace font makes it very easy to visually distinguish groups of related changes, given the way that these subjects are formatted (with the leading tags, etc.). Just a thought.

Acked-by: Stephen Finucane <stephen.finucane at intel.com>


More information about the Patchwork mailing list