Bug 34758 – TD width in precentage doesn't work.

• Component: Tables
• Status: RESOLVED FIXED

[Dimitriy]

Created attachment 48422 [details]
Safari and google chrome

I tried to set width in TD in precents (in HTML and in CSS), but it doesn't works(In newest Safari and google chrome), or works unnormal. In other browsers it works as i need.

There it is
HTML:
<table width="80%" height="100%" align="center" class="main">
<tr><td colspan="3" class="header" height="150"><table width="1000" height="100%"><tr align="center"><td align="center">&nbsp;</td></tr></table></td></tr>
	<tr valign="top"><td class="block1">s<? echo $menu; ?></
			<td class="block2"><? echo $zareg;echo $content; ?></td>
					<td class="block3"><table width="100%" cellspacing="0" cellpadding="0"><tr align="center"><td><div class="frame_top"><b><? echo $user; ?></b></div></td></tr><tr align="center"><td><div class="frame_main"><? echo $umenu; ?></td></tr></table></td></tr></table>

class="block1" - left block
class="block2" - center block
class="block3" - right block
class="header" - header works correctly, but it could be one of reasons of the trouble.

CSS:
.block1 {
	border: 1px solid black;
	width:15%;
}
.block2 {
	border: 1px solid black;
	width:50%;
}
.block3 {
	border: 1px solid black;
	width:15%;
}
.header {
	background-image: url('header2.jpg'); 
	border: 1px solid black;
	width:80%;
	background-repeat: no-repeat;
}


I attached screen how it looks in Safari and Google Chrome, and here http://img692.imageshack.us/img692/2519/othere.jpg how it looks in other browsers.

[Julien Chaffraix]

I think it may be another instance of nested tables with percent width / height not get a proper layout. See also 11645.

[Julien Chaffraix]

Created attachment 115882 [details]
Reduced test case, may be reduced more but it shows the problem

[Julien Chaffraix]

Created attachment 115968 [details]
WIP: Fixes the bug, missing some expectations update or rebaseline and the test case.

[Julien Chaffraix]

Created attachment 116782 [details]
WIP2: Now skipping properly and with the test case, missing baseline.

[Julien Chaffraix]

Comment on attachment 116782 [details]
WIP2: Now skipping properly and with the test case, missing baseline.

It was not meant for review.

[WebKit Review Bot]

Comment on attachment 116782 [details]
WIP2: Now skipping properly and with the test case, missing baseline.

Attachment 116782 [details] did not pass chromium-ews (chromium-xvfb):
Output: http://queues.webkit.org/results/10641536

New failing tests:
tables/mozilla/bugs/bug2947.html
tables/mozilla/bugs/bug14929.html
fast/table/027-vertical.html
tables/mozilla_expected_failures/core/conflicts.html
fast/table/027.html

[Julien Chaffraix]

Created attachment 116796 [details]
Proposed change 1: code change + test case modified for DRT

[WebKit Review Bot]

Comment on attachment 116796 [details]
Proposed change 1: code change + test case modified for DRT

Attachment 116796 [details] did not pass chromium-ews (chromium-xvfb):
Output: http://queues.webkit.org/results/10679315

New failing tests:
tables/mozilla/bugs/bug2947.html
tables/mozilla/bugs/bug14929.html
tables/mozilla_expected_failures/core/conflicts.html

[Julien Chaffraix]

Created attachment 116836 [details]
Proposed change 2: Update Chromium Linux baseline (did not do that properly).

[Dave Hyatt]

Comment on attachment 116836 [details]
Proposed change 2: Update Chromium Linux baseline (did not do that properly).

r=me

[Julien Chaffraix]

Created attachment 117926 [details]
Patch for landing

[WebKit Review Bot]

Comment on attachment 117926 [details]
Patch for landing

Clearing flags on attachment: 117926

Committed r102040: <http://trac.webkit.org/changeset/102040>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.