Bug 247798 - Remove LTR scrollbar padding hack
Summary: Remove LTR scrollbar padding hack
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: Safari Technology Preview
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2022-11-11 04:14 PST by Ahmad Saleem
Modified: 2023-05-08 07:46 PDT (History)
4 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 2022-11-11 04:14:33 PST 
Hi Team,

I came across another Blink commit, which removed some quirk / hack we added for LTR Scrollbar, I don't know whether Webkit can remove it as of right now or not but it is something better to track here on bugzilla for future.

Blink Commit - https://chromium.googlesource.com/chromium/blink/+/0e054f07273d9f5104b4b3926dc438fa794436fe

Webkit GitHub Code - https://github.com/WebKit/WebKit/blob/f0c32585c22cc6c30895d62606a1dc52d1800c41/Source/WebCore/rendering/LegacyLineLayout.cpp#L2156

_____

Just wanted to raise this bug for future tracking and if Line Layout is progressed that we can remove this then just to have bug for this purposes.

Thanks!
Comment 1 Radar WebKit Bug Importer 2022-11-18 04:15:17 PST 
<rdar://problem/102510754>
Comment 2 Ahmad Saleem 2023-05-08 07:17:40 PDT 
@Alan - do we have to do this or IFC will take care of this and will remove this hack?
Comment 3 zalan 2023-05-08 07:46:11 PDT 
We do this in IFC too (mostly to be inline with legacy line layout).