Bug 81824

Summary:

Our sections' extra logical height distribution algorithm is broken

Product:

WebKit

Reporter:

Julien Chaffraix <jchaffraix>

Component:

Tables

Assignee:

Nobody <webkit-unassigned>

Status:

NEW

Severity:

Normal

CC:

ahmad.saleem792, ap, bfulgham, robert, simon.fraser, webkit-bug-importer, zalan

Priority:

Keywords:

InRadar

Version:

528+ (Nightly build)

Hardware:

All

OS:

All

Attachments:

Description	Flags
One example of our broken behavior	none
Idea patch (if someone wants to look into that): Unfortunately doesn't handle border-spacing properly	none
Zipped test cases for the algorithm, missing coverage for extra logical height distribution at the row level	none

Julien Chaffraix

Reported 2012-03-21 12:22:10 PDT

Currently we only allocate the extra logical height to one section. This means that we don't follow at all web-authors' specified height on the sections. This also doesn't match any other browsers which implement a several pass algorithm to handle the different type of height on the sections and on the rows.

Attachments
One example of our broken behavior (604 bytes, text/html) 2012-03-21 12:24 PDT, Julien Chaffraix	no flags	Details
Idea patch (if someone wants to look into that): Unfortunately doesn't handle border-spacing properly (11.17 KB, patch) 2012-06-15 17:59 PDT, Julien Chaffraix	no flags	Details Formatted Diff Diff
Zipped test cases for the algorithm, missing coverage for extra logical height distribution at the row level (8.39 KB, application/zip) 2012-09-10 14:44 PDT, Julien Chaffraix	no flags	Details
View All Add attachment proposed patch, testcase, etc.

Julien Chaffraix

Comment 1 2012-03-21 12:24:25 PDT

Created attachment 133095 [details] One example of our broken behavior

Julien Chaffraix

Comment 2 2012-06-15 17:59:43 PDT

Created attachment 147928 [details] Idea patch (if someone wants to look into that): Unfortunately doesn't handle border-spacing properly

Julien Chaffraix

Comment 3 2012-09-10 14:44:50 PDT

Created attachment 163212 [details] Zipped test cases for the algorithm, missing coverage for extra logical height distribution at the row level

Ahmad Saleem

Comment 4 2022-09-10 07:02:29 PDT

All browsers differ in the attached test case: *** Safari Technology Preview 153 *** Just a single container with top and bottom thick (kind of overlapped) border *** Chrome Canary 107 *** Two containers with divided from center and top bottom is thick like overlapped and center as well. *** Firefox Nightly 106 *** Three containers and middle two are thick lines _____ Just wanted to share updated status. Thanks!

Radar WebKit Bug Importer

Comment 5 2022-09-10 10:26:25 PDT

<rdar://problem/99781034>

Note You need to log in before you can comment on or make changes to this bug.