Bug 14202 - Table caption renders too narrow sometimes
Summary: Table caption renders too narrow sometimes
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 523.x (Safari 3)
Hardware: Mac OS X 10.4
: P2 Normal
Assignee: mitz
URL: http://www.myfoxutah.com/myfox/pages/...
Keywords: HasReduction
Depends on:
Blocks:
 
Reported: 2007-06-17 13:42 PDT by David Kilzer (:ddkilzer)
Modified: 2007-06-25 03:32 PDT (History)
1 user (show)

See Also:


Attachments
Screenshot (411.29 KB, image/png)
2007-06-17 13:43 PDT, David Kilzer (:ddkilzer)
no flags Details
Test case (web page, complete) (246.31 KB, application/zip)
2007-06-17 13:44 PDT, David Kilzer (:ddkilzer)
no flags Details
Reduction (405 bytes, text/html)
2007-06-18 16:10 PDT, mitz
no flags Details
Mark the caption for relayout when the table width changes (287.46 KB, patch)
2007-06-19 01:41 PDT, mitz
darin: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description David Kilzer (:ddkilzer) 2007-06-17 13:42:14 PDT
* SUMMARY
Loading the URL on this bug sometimes results in a caption that is too tall.

* STEPS TO REPRODUCE
1. Launch Safari/WebKit.
2. Open attached test case.
3. If caption under image is not a skinny column, reload the page until it is rendered that way.

* EXPECTED RESULTS
The caption under the image should fill the width of the image and only occupy 4-5 lines of text.

* ACTUAL RESULTS
The caption under the image is about 1 inch wide and occupies numerous lines of text.

* REGRESSION
Unknown since I can't test with archived Safari 2.0.4 (419.3) with the 3.0 beta installed.

* NOTES
See attached screenshot as well.
Comment 1 David Kilzer (:ddkilzer) 2007-06-17 13:43:00 PDT
Created attachment 15091 [details]
Screenshot
Comment 2 David Kilzer (:ddkilzer) 2007-06-17 13:44:49 PDT
Created attachment 15092 [details]
Test case (web page, complete)

Test case for the bug.  This page was saved from Firefox 2.0.0.4 as "Web page, complete".

Unzip the file and open table-caption-issue-ff.html.
Comment 3 David Kilzer (:ddkilzer) 2007-06-17 13:55:11 PDT
Note that there is also a JavaScript error on the page, but I don't think this impacts the layout:

SyntaxError: Invalid regular expression: internal error: code overflow
http://foxtv.us.intellitxt.com/v3/func_033.js?v=200706050032_patch1
Line: 2

Comment 4 mitz 2007-06-18 16:10:09 PDT
Created attachment 15110 [details]
Reduction
Comment 5 mitz 2007-06-19 01:41:48 PDT
Created attachment 15115 [details]
Mark the caption for relayout when the table width changes

This improves the results on several Mozilla tests, but caption width still seems to be off by a few pixels when the table has a border. That's a separate bug, however.
Comment 6 Darin Adler 2007-06-21 23:44:11 PDT
Comment on attachment 15115 [details]
Mark the caption for relayout when the table width changes

r=me
Comment 7 Mark Rowe (bdash) 2007-06-25 03:32:37 PDT
Landed in r23762.