Summary: | [GTK] Touchscreen navigation swipe doesn't work when the page scrolls horizontally | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Alice Mikhaylenko <alicem> | ||||||||
Component: | WebKitGTK | Assignee: | Nobody <webkit-unassigned> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | berto, bugs-noreply, cgarcia, ews-watchlist, gustavo, mcatanzaro | ||||||||
Priority: | P2 | ||||||||||
Version: | WebKit Nightly Build | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
Attachments: |
|
Description
Alice Mikhaylenko
2021-07-05 14:44:06 PDT
Created attachment 432917 [details] Patch Probably doesn't apply without https://bugs.webkit.org/show_bug.cgi?id=226745 Hm, there's a crash when tapping after a scroll. It's the fact that we put the event back if WebCore hasn't handled them and then it gets processed again even though it shouldn't be. Makes me wonder how or why it works for scroll events. Created attachment 433023 [details]
Patch
Fixed. For now, just don't propagate touch events, there's no real point in doing that anyway.
Created attachment 433148 [details]
Patch
Oh wow, so the style script can determine if a name is redundant or not?
Thanks for the patch. If this patch contains new public API please make sure it follows the guidelines for new WebKit2 GTK+ API. See https://trac.webkit.org/wiki/WebKitGTK/AddingNewWebKit2API (In reply to Alexander Mikhaylenko from comment #5) > Created attachment 433148 [details] > Patch > > Oh wow, so the style script can determine if a name is redundant or not? Yeah that's a little weird, but the rule is to name the parameter only in the source file, and leave it unnamed in the header file to save a bit of space (if the name is going to be something obvious like e.g. "Point point"). Yeah, I know the rule, but didn't think the script could catch that. Comment on attachment 433148 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=433148&action=review > Source/WebKit/UIProcess/API/gtk/PageClientImpl.cpp:444 > + // Wheel events can have either scroll events or touch events attached to them. > + // We only want to propagate scroll events; touch events are controlled via their > + // event sequences and if we're scrolling with touch events, that sequence is > + // already claimed and there's no point in propagating it. Thanks for adding a comment to explain the weird behavior. Committed r279779 (239546@main): <https://commits.webkit.org/239546@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 433148 [details]. |