[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