Bug 248487

Summary:

Remove a float from an element's list even if its style suggests it can't contain floats

Product:

WebKit

Reporter:

Ahmad Saleem <ahmad.saleem792>

Component:

Layout and Rendering

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

bfulgham, simon.fraser, webkit-bug-importer, zalan

Priority:

Keywords:

InRadar

Version:

Safari Technology Preview

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
Visual Glitch Video on JSFiddle Test Case	none

Ahmad Saleem

Reported 2022-11-29 12:45:57 PST

Hi Team, While going through Blink's commit, I came across another potential case to make Layout & Rendering bit more robust. Test Case (Showing "black" scrollbar on left side in Safari till repaint then it shows on left) - https://jsfiddle.net/eso3k7vw/show Blink Commit - https://src.chromium.org/viewvc/blink?view=revision&revision=190902 Webkit GitHub Source - https://github.com/WebKit/WebKit/blob/52128324756a99e79952e4d3431bff7fdceb9bb9/Source/WebCore/rendering/RenderBlockFlow.cpp#LL2816 Just wanted to raise bug before trying to do PR. Thanks!

Attachments
Visual Glitch Video on JSFiddle Test Case (9.00 MB, video/quicktime) 2022-12-01 08:06 PST, Ahmad Saleem	no flags	Details
View All Add attachment proposed patch, testcase, etc.

Ahmad Saleem

Comment 1 2022-12-01 08:06:43 PST

Created attachment 463829 [details] Visual Glitch Video on JSFiddle Test Case @Alan - visual glitch with this test case and why I am trying to merge this because it causes Safari to do painting issue and also show scrollbar on incorrect side.

EWS

Comment 2 2022-12-05 06:41:24 PST

Committed 257370@main (6112c5929ca9): <https://commits.webkit.org/257370@main> Reviewed commits have been landed. Closing PR #6933 and removing active labels.

Radar WebKit Bug Importer

Comment 3 2022-12-05 06:42:22 PST

<rdar://problem/102980611>

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