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

Description Rado 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/
Comment 1 Alexey Proskuryakov 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)?
Comment 2 Radar WebKit Bug Importer 2023-08-07 19:51:44 PDT 
<rdar://problem/113535569>
Comment 3 Karl Dubost 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?
Comment 4 Karl Dubost 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 ***
Comment 5 Rado 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
Comment 6 Rado 2023-08-09 12:26:29 PDT 
Fixed in today's macOS 14 public beta. Thanks