[FFmpeg-devel] [PATCH 4/4] web/style.less: Separate out .table-bordered from .table

[Clément Bœsch]

On Sat, Nov 15, 2014 at 03:49:56PM -0800, Timothy Gu wrote:
> On Sat, Nov 15, 2014 at 3:45 PM, Michael Niedermayer <michaelni at gmx.at> wrote:
> > On Fri, Nov 14, 2014 at 05:12:47PM -0800, Timothy Gu wrote:
> >> On Thursday, November 6, 2014, Timothy Gu <timothygu99 at gmail.com> wrote:
> >> >
> >> > The only instance of .table in the web source is the download page for the
> >> > Git repos. I did not change that page because an unbordered table there is
> >> > prettier IMO.
> >> >
> >> > I retained the .table-bordered style in case anyone want to use it in the
> >> > future.
> >> >
> >> > Signed-off-by: Timothy Gu <timothygu99 at gmail.com>
> >> > ---
> >> >  src/less/style.less | 61 ++++++++++++++++++++++++++++++++++++++++++++++-------
> >> >  1 file changed, 53 insertions(+), 8 deletions(-)
> >>
> >> Ping.
> >>
> >
> >> Before-after comparison: http://imgur.com/eXSXFxe&tjnkmXb
> >
> > which way do people prefer ?
> > please comment
> > ill apply it if more people like the look after the patch then before
> > its not clear to me which way is/looks better
> 
> The main reason for this patch is because in fateserver-node I
> redesigned the build result page to utilize this borderless design,
> but that requires a change to the style.less file. When I wrote the
> patch I initially wanted to retain the old style for all the pages
> that uses .table, but it seems like the only page that uses it does
> not look worse without borders.
> 

It doesn't look exactly borderless: http://i.imgur.com/ho6crAM.png (see
the black line). Also, picking the same color for odd row & background is
disturbing without background.

> If the bordered design is what people prefer I can send a new patch
> that splits the styles **and** retains the old style.
> 
> Timothy

-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 473 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20141116/d0a60512/attachment.asc>