Bug 255844 - Add padding:1px to <td> and <th> outside tables, per spec
Summary: Add padding:1px to <td> and <th> outside tables, per spec
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: CSS (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: BrowserCompat, InRadar, WPTImpact
Depends on:
Blocks:
 
Reported: 2023-04-23 11:57 PDT by Ahmad Saleem
Modified: 2023-04-30 11:58 PDT (History)
5 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Ahmad Saleem 2023-04-23 11:57:09 PDT 
Hi Team,

While going through Blink's commit, I came across following UA Stylesheet update commit:

Commit - https://chromium.googlesource.com/chromium/src/+/c5ecd6d100aeefb1b77c310beb11e458fc2d139c

Failing WPT Test Case - http://wpt.live/css/css-tables/html-to-css-mapping-1.html

Web-Spec: https://html.spec.whatwg.org/#tables-2

Our UA Stylesheet does not have following:

td, th { padding: 1px; }

^ Adding this manually in the local branch fixed the failing sub-test in WPT testcase but I don't know whether it would've any unintended effect.

Blink added this with different rule in their stylesheet because they had some styling enforced via HTMLTableCellElement::AdditionalPresentationAttributeStyle() function.

I didn't find any special styling within WebKit in HTMLTableCellElement though.

Appreciate if someone can confirm, if my understanding is right and I am happy to do PR.

Thanks!
Comment 1 Radar WebKit Bug Importer 2023-04-30 11:58:18 PDT 
<rdar://problem/108716695>