Bug 4977 – <form> not being displayed at all (in a <div> with display: table)

• Status: RESOLVED
• Resolution: FIXED
• Component: CSS

[David Kerry]

For some reason, the form at http://camportal.co.uk/v0.4/?page=add does not show in Safari, however 
the rest of the page past the form renders fine. Also the form shows in other browsers/rendering engines.

[David Kerry]

Created attachment 3891 [details]
Example Screen Capture

A screen capture showing the missing form in Safari 2.0.1

[Stuart Morgan]

Confirmed in WebKit 412.2.2.  The problem goes away if the "display: table" is
removed from the enclosing div--it seems odd that it's even there though, since
none of the normal content inside it is set to be displayed as any kind of
sub-table element.

[Eric Seidel (no email)]

There were fixes relating to positioned elements inside tables recently.  It's possible this was fixed.  It's 
hard to know though without a test case (the server is down).  Please attach a reduced test case, or close.

[David Kerry]

Created attachment 5317 [details]
Stripped code to give working example of bug

Still presents a problem in Safari 2.0.2 (416.13), and latest webkit downloaded
today.

[Joost de Valk (AlthA)]

Created attachment 5329 [details]
Testcase

The problem lies in the display:table style.

[Joost de Valk (AlthA)]

The combination of <form> in a div styled with "display: table;" seems to be causing this.

[Joost de Valk (AlthA)]

Created attachment 5330 [details]
Testcase

added a comment to make more comprehensible.

[Karl Schramm]

This seems fixed in 417.8

[David Kilzer (:ddkilzer)]

This is not fixed in Safari 2.0.3 (417.8) on 10.4.5 for me, but it is fixed in ToT WebKit as of r12953.

Note that when running WebKit+Safari, Safari's production version still shows up (and the Dashboard claims Safari is running) which may cause you to think it's been fixed in a production release.

[David Kilzer (:ddkilzer)]

This bug was fixed between the r12724 and r12747 nightly builds:

Broken: WebKit-SVN-r12724.dmg
Fixed:  WebKit-SVN-r12747.dmg

Looking at the log messages, it's likely that r12746 fixed this issue.

Closing bug as RESOLVED/FIXED.

[David Kilzer (:ddkilzer)]

Reopening; should add a layout test to prevent other regressions.

[David Kilzer (:ddkilzer)]

Created attachment 6723 [details]
Patch v1

This bug has already been fixed by a previous commit (see Comment #10), but I thought it should have a layout test to prevent future regressions.  Inspired by Darin's layout test for Bug 5731, I created a similar layout test for this bug.

The only two display styles not shown are "table-column-group" and "table-column".  I'm not sure if this means there is a rendering bug that still needs to be fixed or not, though.

[Geoffrey Garen]

Landed.