Bug 97239 – Baseline alignment with inline <table> is broken when margin-top is applied

• Status: RESOLVED
• Resolution: FIXED
• Component: Layout and Rendering

[Daniel Holbert]

Created attachment 164952 [details]
testcase 1

STEPS TO REPRODUCE:
 1. Load attached testcase. (with inline-block "aaa" and inline-table "bbb")

EXPECTED RESULTS:
 "aaa" and "bbb" are baseline-aligned.

ACTUAL RESULTS:
 They're misaligned.  "aaa" ends up baseline-aligned with where "bbb" *would* be, if it weren't for the margin-top.

So basically, it looks like Webkit's not taking the margin into consideration when doing baseline-alignment here.

Reproduced with latest Chrome build from google repo's "Dev Channel" on Ubuntu.
Version 23.0.1270.0 dev

[Daniel Holbert]

Created attachment 164955 [details]
screenshot (w/ red showing where "aaa" should be)

Here's a screenshot of the rendering, with red arrow & line showing where "aaa" _should_ be.

[Daniel Holbert]

Created attachment 164957 [details]
reference case 1 (with inline-block instead of inline-table)

Here's a reference case, using an inline-block for "bbb" instead of an inline-table. My chrome dev channel build gets this one right. ("aaa" and "bbb" are aligned)

testcase 1 should render just like this reference case.

In Opera 12.02 and Firefox 15.0.1, the testcase & reference case render identically. (aside from the explanatory text)

[Yuki Sekiguchi]

Created attachment 205153 [details]
Patch

[Brady Eidson]

Comment on attachment 205153 [details]
Patch

Assuming that patches for review since 2013 are stale, r-

[Radar WebKit Bug Importer]

<rdar://problem/100910532>

[Sammy Gill]

Pull request: https://github.com/WebKit/WebKit/pull/5153

[EWS]

Committed 255357@main (c0048e83eb23): <https://commits.webkit.org/255357@main>

Reviewed commits have been landed. Closing PR #5153 and removing active labels.