[Piglit] [PATCH v2 1/8] html: Use CSS for indenting table rows rather than colspan.

Kenneth Graunke kenneth at whitecape.org
Wed May 8 12:02:01 PDT 2013


On 04/30/2013 10:34 AM, Dylan Baker wrote:
> From: Kenneth Graunke <kenneth at whitecape.org>
>
> Previously, we indented groups of tests by adding 10 columns
> of whitespace to the table and adding colspan.  This is pretty
> ugly; we really should use CSS for things like this.
>
> The new CSS-based approach should work with deeper nesting levels,
> and simplifies the tables.
>
> Signed-off-by: Kenneth Graunke <kenneth at whitecape.org>
> Reviewed-by: Dylan Baker <baker.dylan.c at gmail.com>

I just pushed this series (at last!) with one additional fix: changing 
width: 100% to min-width: 100% makes the table-layout: fixed stuff work 
even when there are way too many columns.

Thanks for resurrecting this series, cleaning it up, and getting it 
through the process!


More information about the Piglit mailing list