Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Turn macOS smooth keyboard scrolling on by default #4453

Merged
merged 1 commit into from Sep 30, 2022

Conversation

rr-codes
Copy link
Contributor

@rr-codes rr-codes commented Sep 18, 2022

cd1dbd2

Turn macOS smooth keyboard scrolling on by default
https://bugs.webkit.org/show_bug.cgi?id=228159
rdar://80912063

Reviewed by Tim Horton and Wenson Hsieh.

No tests added.

Set the default value for EventHandlerDrivenSmoothKeyboardScrollingEnabled to true in WebKit.

* LayoutTests/css3/scroll-snap/scroll-padding-overflow-paging.html:
Updated to reflect non-instananeous scrolling.

* LayoutTests/fast/repaint/resources/fixed-move-after-keyboard-scroll-iframe.html:
Updated to reflect non-instananeous scrolling.

* LayoutTests/fast/scrolling/arrow-key-scroll-in-rtl-document-expected.txt:
* LayoutTests/fast/scrolling/arrow-key-scroll-in-rtl-document.html:
* LayoutTests/platform/mac-wk1/fast/scrolling/arrow-key-scroll-in-rtl-document-expected.txt: Added.
Updated these tests to reflect the new distance scrolled.

* Source/WTF/Scripts/Preferences/WebPreferencesInternal.yaml:
`EventHandlerDrivenSmoothKeyboardScrollingEnabled` is now true on default for WebKit.

* Source/WebCore/page/EventHandler.cpp:
(WebCore::EventHandler::startKeyboardScrollAnimationOnEnclosingScrollableContainer):
Added check if renderer is null to fix crash in LayoutTests/fast/forms/select/select-change-type-on-focus.

* Tools/TestWebKitAPI/Tests/WebKit/SpacebarScrolling.cpp:
(TestWebKitAPI::didRunJavascript):
(TestWebKitAPI::TEST):
Updated to reflect non-instananeous scrolling.

Canonical link: https://commits.webkit.org/255031@main

2e5a42e

Misc iOS, tvOS & watchOS macOS Linux Windows
βœ… πŸ§ͺ style βœ… πŸ›  ios βœ… πŸ›  mac βœ… πŸ›  wpe ❌ πŸ›  πŸ§ͺ win
βœ… πŸ§ͺ bindings βœ… πŸ›  ios-sim βœ… πŸ›  mac-debug βœ… πŸ›  gtk βœ… πŸ›  wincairo
βœ… πŸ§ͺ webkitperl   πŸ§ͺ ios-wk2 βœ… πŸ›  mac-AS-debug βœ… πŸ§ͺ gtk-wk2
  πŸ§ͺ api-ios   πŸ§ͺ api-mac βœ… πŸ§ͺ api-gtk
βœ… πŸ›  πŸ§ͺ jsc βœ… πŸ›  tv βœ… πŸ§ͺ mac-wk1 βœ… πŸ›  jsc-armv7
βœ… πŸ›  tv-sim βœ… πŸ§ͺ mac-wk2 βœ… πŸ§ͺ jsc-armv7-tests
βœ… πŸ›  πŸ§ͺ merge βœ… πŸ›  watch   πŸ§ͺ mac-AS-debug-wk2 βœ… πŸ›  jsc-mips
βœ… πŸ›  watch-sim βœ… πŸ§ͺ mac-wk2-stress βœ… πŸ§ͺ jsc-mips-tests

@rr-codes rr-codes self-assigned this Sep 18, 2022
@rr-codes rr-codes added Scrolling Bugs related to main thread and off-main thread scrolling WebKit Local Build labels Sep 18, 2022
@rr-codes rr-codes marked this pull request as draft September 18, 2022 02:44
@webkit-ews-buildbot webkit-ews-buildbot added the merging-blocked Applied to prevent a change from being merged label Sep 18, 2022
@rr-codes rr-codes removed the merging-blocked Applied to prevent a change from being merged label Sep 18, 2022
@webkit-ews-buildbot webkit-ews-buildbot added the merging-blocked Applied to prevent a change from being merged label Sep 18, 2022
@rr-codes rr-codes removed the merging-blocked Applied to prevent a change from being merged label Sep 20, 2022
@webkit-ews-buildbot webkit-ews-buildbot added the merging-blocked Applied to prevent a change from being merged label Sep 20, 2022
@rr-codes rr-codes removed the merging-blocked Applied to prevent a change from being merged label Sep 20, 2022
@webkit-ews-buildbot webkit-ews-buildbot added the merging-blocked Applied to prevent a change from being merged label Sep 20, 2022
@rr-codes rr-codes removed the merging-blocked Applied to prevent a change from being merged label Sep 20, 2022
@webkit-ews-buildbot webkit-ews-buildbot added the merging-blocked Applied to prevent a change from being merged label Sep 21, 2022
@rr-codes rr-codes removed the merging-blocked Applied to prevent a change from being merged label Sep 21, 2022
@webkit-ews-buildbot webkit-ews-buildbot added the merging-blocked Applied to prevent a change from being merged label Sep 22, 2022
@rr-codes rr-codes removed the merging-blocked Applied to prevent a change from being merged label Sep 22, 2022
@rr-codes rr-codes marked this pull request as ready for review September 29, 2022 03:08
@@ -275,7 +275,7 @@ EventHandlerDrivenSmoothKeyboardScrollingEnabled:
WebKitLegacy:
default: false
WebKit:
default: false
default: true
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

dances

@webkit-ews-buildbot webkit-ews-buildbot added the merging-blocked Applied to prevent a change from being merged label Sep 30, 2022
@rr-codes rr-codes added merge-queue Applied to send a pull request to merge-queue and removed merging-blocked Applied to prevent a change from being merged labels Sep 30, 2022
https://bugs.webkit.org/show_bug.cgi?id=228159
rdar://80912063

Reviewed by Tim Horton and Wenson Hsieh.

No tests added.

Set the default value for EventHandlerDrivenSmoothKeyboardScrollingEnabled to true in WebKit.

* LayoutTests/css3/scroll-snap/scroll-padding-overflow-paging.html:
Updated to reflect non-instananeous scrolling.

* LayoutTests/fast/repaint/resources/fixed-move-after-keyboard-scroll-iframe.html:
Updated to reflect non-instananeous scrolling.

* LayoutTests/fast/scrolling/arrow-key-scroll-in-rtl-document-expected.txt:
* LayoutTests/fast/scrolling/arrow-key-scroll-in-rtl-document.html:
* LayoutTests/platform/mac-wk1/fast/scrolling/arrow-key-scroll-in-rtl-document-expected.txt: Added.
Updated these tests to reflect the new distance scrolled.

* Source/WTF/Scripts/Preferences/WebPreferencesInternal.yaml:
`EventHandlerDrivenSmoothKeyboardScrollingEnabled` is now true on default for WebKit.

* Source/WebCore/page/EventHandler.cpp:
(WebCore::EventHandler::startKeyboardScrollAnimationOnEnclosingScrollableContainer):
Added check if renderer is null to fix crash in LayoutTests/fast/forms/select/select-change-type-on-focus.

* Tools/TestWebKitAPI/Tests/WebKit/SpacebarScrolling.cpp:
(TestWebKitAPI::didRunJavascript):
(TestWebKitAPI::TEST):
Updated to reflect non-instananeous scrolling.

Canonical link: https://commits.webkit.org/255031@main
@webkit-commit-queue
Copy link
Collaborator

Committed 255031@main (cd1dbd2): https://commits.webkit.org/255031@main

Reviewed commits have been landed. Closing PR #4453 and removing active labels.

@webkit-commit-queue webkit-commit-queue merged commit cd1dbd2 into WebKit:main Sep 30, 2022
@webkit-commit-queue webkit-commit-queue removed the merge-queue Applied to send a pull request to merge-queue label Sep 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Scrolling Bugs related to main thread and off-main thread scrolling
Projects
None yet
6 participants