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

Finucane, Stephen stephen.finucane at intel.com
Thu Sep 10 18:40:27 AEST 2015

> On 09/09/2015 16:13, "Finucane, Stephen" <stephen.finucane at intel.com>
> wrote:
> >> 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.
> Well, not exactly: the looking 'prettier' is somehow an after effect.
> Apparently white space improves readability (speed and comprehension),
> both real and perceived [1]. That might be one of the reasons why we find
> them prettier :)
> [1]
> http://usabilitynews.org/reading-online-text-a-comparison-of-four-white-spa
> ce-layouts/

That particular article relates to blocks of text rather than tabular data so I'm not sure it's relevant. I'm happy with the additional whitespace requirements though: just pointing out that it might be contentious :)

> >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.
> This is interesting: I wonder if other folks find the same thing. I would
> have no problem with the font change.

I wonder how'd you'd determine this? Some form of A/B testing (or other usability testing method)? Try it yourself though: I think it's a marked improvement.

> Cheers
> Belén
> >
> >Acked-by: Stephen Finucane <stephen.finucane at intel.com>

More information about the Patchwork mailing list