Bug 15259

Summary: REGRESSION:Text overflows if a empty <td> is followed by a <td align="center">
Product: WebKit Reporter: Anantha Keesara <anantha>
Component: Layout and RenderingAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: gsherloc, mitz
Priority: P1 Keywords: HasReduction, InRadar, Regression
Version: 523.x (Safari 3)   
Hardware: All   
OS: All   
URL: http://www.myspace.com
Attachments:
Description Flags
Text overflows If a empty <td> is followed by a <td align="center">
none
Patch, including change log and a layout test hyatt: review+

Description Anantha Keesara 2007-09-22 21:53:24 PDT
1. Go to http://www.myspace.com
2. Navigate to Music tab from the Search Menu on the Home page.
3. Navigate to Music Videos from the Search Menu on MySpace Music page (http://www.myspace.com/index.cfm?fuseaction=music&MyToken=36ce119d-bb2c-459a-8c24-fe70590f5b06 )

Issue: ‘Browse By Artist’ list is displayed outside the table area.
Behavior on other Browsers : 
IE ,FF, Opera : no issue.
Comment 1 Anantha Keesara 2007-09-22 21:56:24 PDT
Created attachment 16354 [details]
Text overflows If a empty <td> is followed by a <td align="center">
Comment 2 Gavin Sherlock 2007-09-22 22:06:29 PDT
This doesn't happen on released Safari on the Mac, but does happen with a nightly build (I was using revision 25571), so is a regression (I don't have privileges to change the Summary to indicate as such).
Comment 3 mitz 2007-09-23 00:37:36 PDT
This appears to be a bug in preferred width calculation when there is word spacing involved. A simpler test case:
<div style="position: absolute; border:solid ; word-spacing: 10px;"> lorem ipsum dolor</div>
Comment 4 David Kilzer (:ddkilzer) 2007-09-23 09:57:02 PDT
This bug occurs in WebKit Nightly r11976.

Comment 5 David Kilzer (:ddkilzer) 2007-09-23 09:59:01 PDT
<rdar://problem/5499902>
Comment 6 mitz 2007-09-24 08:20:54 PDT
Bug 15270 looks similar. Fixing this bug should fix it too.
Comment 7 mitz 2007-10-20 20:48:39 PDT
Created attachment 16756 [details]
Patch, including change log and a layout test
Comment 8 Dave Hyatt 2007-10-20 22:36:48 PDT
Comment on attachment 16756 [details]
Patch, including change log and a layout test

r=me
Comment 9 mitz 2007-10-21 00:23:56 PDT
Fixed in <http://trac.webkit.org/projects/webkit/changeset/26846>.