Removing table-layout: fixed property from chrome://system page.

Removing table-layout: fixed property from chrome://system page.

The property causes trouble when button text is too long (like in French)

TEST=Manual (with lanugage set to french).
BUG=chromium-os:18471
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=107157

[arv (Not doing code reviews), 2011-10-21 18:51:14]

Doesn't this break things? Why was table-layout: fixed in the first place?

[tbarzic, 2011-10-21 19:23:33]

On 2011/10/21 18:51:14, arv wrote:
> Doesn't this break things? Why was table-layout: fixed in the first place?

I don't think it should break anything (at least I don't see anything broken).
The only bad thing I noticed is that the first column changes its width when
there is a cell that has really wide lines in the third column, but that doesn't
seem to bad.

[arv (Not doing code reviews), 2011-10-21 19:29:16]

LGTM

I would prefer if you can also get an LGTM from the original author
(maybe that is you?)

erik








On Fri, Oct 21, 2011 at 12:23,  <tbarzic@chromium.org> wrote:
> On 2011/10/21 18:51:14, arv wrote:
>>
>> Doesn't this break things? Why was table-layout: fixed in the first place?
>
> I don't think it should break anything (at least I don't see anything
> broken).
> The only bad thing I noticed is that the first column changes its width when
> there is a cell that has really wide lines in the third column, but that
> doesn't
> seem to bad.
>
>
> http://codereview.chromium.org/8369003/
>

[tbarzic, 2011-10-21 19:38:01]

On 2011/10/21 19:29:16, arv wrote:
> LGTM
> 
> I would prefer if you can also get an LGTM from the original author
> (maybe that is you?)
> 
> erik
> 
> 
> 
> 
> 
> 
> 
> 
> On Fri, Oct 21, 2011 at 12:23,  <mailto:tbarzic@chromium.org> wrote:
> > On 2011/10/21 18:51:14, arv wrote:
> >>
> >> Doesn't this break things? Why was table-layout: fixed in the first place?
> >
> > I don't think it should break anything (at least I don't see anything
> > broken).
> > The only bad thing I noticed is that the first column changes its width when
> > there is a cell that has really wide lines in the third column, but that
> > doesn't
> > seem to bad.
> >
> >
> > http://codereview.chromium.org/8369003/
> >

Yes, I'm the original author :)