Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: bb9af3d426703c0dab09e5de39fe435eb6876f7f
      
https://github.com/WebKit/WebKit/commit/bb9af3d426703c0dab09e5de39fe435eb6876f7f
  Author: Nikolaos Mouchtaris <nmouchta...@apple.com>
  Date:   2024-01-03 (Wed, 03 Jan 2024)

  Changed paths:
    A 
LayoutTests/imported/w3c/web-platform-tests/css/css-scroll-anchoring/focused-element-nested-anchor-expected.txt
    A 
LayoutTests/imported/w3c/web-platform-tests/css/css-scroll-anchoring/focused-element-nested-anchor.html
    M Source/WebCore/page/scrolling/ScrollAnchoringController.cpp
    M Source/WebCore/page/scrolling/ScrollAnchoringController.h
    M Source/WebCore/rendering/RenderElement.cpp
    M Source/WebCore/rendering/RenderObject.cpp
    M Source/WebCore/rendering/RenderObject.h

  Log Message:
  -----------
  [scroll-anchoring] Weird scrolling effect when scrolling YouTube transcript 
panel
https://bugs.webkit.org/show_bug.cgi?id=267005
rdar://120378263

Reviewed by Simon Fraser.

After https://commits.webkit.org/272277@main we descend subscrollers that are 
not maintaining a
scroll anchor. We need to replicate the logic used in 
ScrollAnchoringController::examineAnchorCandidate
for ScrollAnchoringController::didFindPriorityCandidate, to ensure we don't end 
up choosing an anchor
element that is a descendant of a scrollable area that is maintaining a scroll 
anchor.

* 
LayoutTests/imported/w3c/web-platform-tests/css/css-scroll-anchoring/focused-element-nested-anchor-expected.txt:
 Added.
* 
LayoutTests/imported/w3c/web-platform-tests/css/css-scroll-anchoring/focused-element-nested-anchor.html:
 Added.
* Source/WebCore/page/scrolling/ScrollAnchoringController.cpp:
(WebCore::scrollAnchoringControllerForElement):
(WebCore::ScrollAnchoringController::didFindPriorityCandidate):
(WebCore::canDescendIntoElement):
(WebCore::ScrollAnchoringController::isExcludedSubtree):
(WebCore::ScrollAnchoringController::examineAnchorCandidate):
(WebCore::ScrollAnchoringController::chooseAnchorElement):
(WebCore::ScrollAnchoringController::adjustScrollPositionForAnchoring):
* Source/WebCore/page/scrolling/ScrollAnchoringController.h:

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


_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to