Summary: | Animated keyboard scrolling is extremely chaotic | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Tim Horton <thorton> | ||||||||
Component: | New Bugs | Assignee: | Tim Horton <thorton> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | bdakin, commit-queue, dino, simon.fraser, webkit-bug-importer, wenson_hsieh | ||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||
Version: | WebKit Nightly Build | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
Attachments: |
|
Description
Tim Horton
2019-03-22 16:18:15 PDT
Created attachment 365773 [details]
Patch
Comment on attachment 365773 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=365773&action=review > Source/WebKit/UIProcess/ios/WKKeyboardScrollingAnimator.mm:360 > + WTFLogAlways("THORTON STOP"); Obviously without this 🙃 Comment on attachment 365773 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=365773&action=review > Source/WebKit/UIProcess/ios/WKKeyboardScrollingAnimator.h:65 > +- (BOOL)hasScrollingKeyPressed; I don't like this term. It's no clear if it means "scrolling key is currently depressed", or "scrolling key was pressed sometime in the past". (In reply to Simon Fraser (smfr) from comment #4) > Comment on attachment 365773 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=365773&action=review > > > Source/WebKit/UIProcess/ios/WKKeyboardScrollingAnimator.h:65 > > +- (BOOL)hasScrollingKeyPressed; > > I don't like this term. It's no clear if it means "scrolling key is > currently depressed", or "scrolling key was pressed sometime in the past". As you might be able to tell from the renaming in this patch, I'm having the same fight with myself :D I'll try a third name. Created attachment 365889 [details]
Patch
Comment on attachment 365889 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=365889&action=review > Source/WebKit/UIProcess/ios/WKKeyboardScrollingAnimator.h:65 > +- (BOOL)hasScrollingKeyDown; scrollTriggeringKeyIsPressed / scrollTriggeringKeyIsDown? Created attachment 365890 [details]
Patch
Comment on attachment 365890 [details] Patch Clearing flags on attachment: 365890 Committed r243454: <https://trac.webkit.org/changeset/243454> All reviewed patches have been landed. Closing bug. |