Bug 156845

Summary:

[EFL] Enable overflow scrolling of ScrollableArea in Touch Mode

Product:

WebKit

Reporter:

Joonghun Park <jh718.park>

Component:

WebKit EFL

Assignee:

Joonghun Park <jh718.park>

Status:

RESOLVED WONTFIX

Severity:

Normal

CC:

gyuyoung.kim, lucas.de.marchi, mcatanzaro

Priority:

Version:

WebKit Nightly Build

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
Patch	none
Add missing parenthese	none
Fix debug build by fixing a typo in ASSERT	none
Add FIXME comment for implementing delegated overflow scroll later	achristensen: review-

Joonghun Park

Reported 2016-04-21 04:21:20 PDT

When using touch mode, enable overflow scroll in inner scrollableArea, e.g. iframe.

Attachments
Patch (22.54 KB, patch) 2016-04-21 04:25 PDT, Joonghun Park	no flags	Details Formatted Diff Diff
Add missing parenthese (22.57 KB, patch) 2016-04-21 08:59 PDT, Joonghun Park	no flags	Details Formatted Diff Diff
Fix debug build by fixing a typo in ASSERT (22.44 KB, patch) 2016-04-21 15:10 PDT, Joonghun Park	no flags	Details Formatted Diff Diff
Add FIXME comment for implementing delegated overflow scroll later (22.78 KB, patch) 2016-04-21 22:32 PDT, Joonghun Park	achristensen: review-	Details Formatted Diff Diff
Show Obsolete (3) View All Add attachment proposed patch, testcase, etc.

Joonghun Park

Comment 1 2016-04-21 04:25:30 PDT

Created attachment 276908 [details] Patch

Joonghun Park

Comment 2 2016-04-21 04:27:17 PDT

FYI, I'm gonna upload scroll snap enabling patch based on this change in the following patch.

Joonghun Park

Comment 3 2016-04-21 06:35:37 PDT

I tested this change's behaviour at before r199738, ANGLE update patch. It seems that I have to rebase this patch.

Joonghun Park

Comment 4 2016-04-21 08:59:39 PDT

Created attachment 276933 [details] Add missing parenthese

Joonghun Park

Comment 5 2016-04-21 15:10:27 PDT

Created attachment 276967 [details] Fix debug build by fixing a typo in ASSERT

Joonghun Park

Comment 6 2016-04-21 22:32:55 PDT

Created attachment 277010 [details] Add FIXME comment for implementing delegated overflow scroll later

Alex Christensen

Comment 7 2017-03-03 08:34:50 PST

Comment on attachment 277010 [details] Add FIXME comment for implementing delegated overflow scroll later EFL port was removed. This patch is no longer needed in trunk.

Michael Catanzaro

Comment 8 2017-03-11 10:42:47 PST

Closing this bug because the EFL port has been removed from trunk. If you feel this bug applies to a different upstream WebKit port and was closed in error, please either update the title and reopen the bug, or leave a comment to request this.

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