Bug 259878

Summary: REGRESSION (Safari 17 beta): Padding is ignored
Product: WebKit Reporter: Rado <rado>
Component: CSSAssignee: Nobody <webkit-unassigned>
Status: RESOLVED DUPLICATE    
Severity: Major CC: karlcow, simon.fraser, webkit-bug-importer, zalan
Priority: P2 Keywords: InRadar
Version: Safari 17   
Hardware: Mac (Apple Silicon)   
OS: Other   
URL: https://radogado.github.io/n-select/
Attachments:
Description Flags
Padding ignored
none
rendering in safari and stp none

Rado
Reported 2023-08-06 23:08:25 PDT
Created attachment 467210 [details] Padding ignored Safari Version 17.0 (19616.1.24.11.3) doesn't respect the padding-top and padding-bottom properties. It doesn't happen on latest TP (Release 175 (Safari 17.0, WebKit 19617.1.1.2)) and 16.x. Please check the screenshot and URL: https://radogado.github.io/n-select/
Attachments
Padding ignored (439.63 KB, image/jpeg)
2023-08-06 23:08 PDT, Rado 		no flags
Details
rendering in safari and stp (731.40 KB, image/png)
2023-08-07 21:08 PDT, Karl Dubost 		no flags
Details
View All Add attachment proposed patch, testcase, etc.
Alexey Proskuryakov
Comment 1 2023-08-07 19:51:33 PDT
Thank you for the report! Could you please let us know what the impact on end users is (i.e., if there are popular websites affected)?
Radar WebKit Bug Importer
Comment 2 2023-08-07 19:51:44 PDT
<rdar://problem/113535569>
Karl Dubost
Comment 3 2023-08-07 21:08:40 PDT
Created attachment 467218 [details] rendering in safari and stp I'm not able able to reproduce in most recent versions of Safari. Tested on macOS 14.0 --- Safari 17.0 19616.1.26 Safari Technology Preview 175 19617.1.1.2 If there was a regression it seems it has been solved since?
Karl Dubost
Comment 4 2023-08-07 21:46:14 PDT
https://commits.webkit.org/compare/265985@main...265983@main It has been fixed in Bug 259112 with https://github.com/WebKit/WebKit/commit/58d11893af8926dc0092a858dab48021353d7535 *** This bug has been marked as a duplicate of bug 259112 ***
Rado
Comment 5 2023-08-08 01:30:04 PDT
My company site would be affected, but as long as it's fixed in the shipping version of Safari 17, it's fine. Thanks
Rado
Comment 6 2023-08-09 12:26:29 PDT
Fixed in today's macOS 14 public beta. Thanks
Note You need to log in before you can comment on or make changes to this bug.