238093 2022-03-18 13:46:22 -0700 preventScroll does not work with dynamically created elements 2024-05-03 05:13:22 -0700 1 1 1 Unclassified WebKit Forms Safari 15 Mac (Intel) macOS 11 NEW https://bugs.webkit.org/show_bug.cgi?id=267051 InRadar P2 Normal --- 1 Elliot.nahman webkit-unassigned augus.dupin caleb cdumez rik simon.fraser webkit-bug-importer wenson_hsieh oldest_to_newest 1852818 0 Elliot.nahman 2022-03-18 13:46:22 -0700 Safari 15 appears to have implemented the preventScroll option for Element.focus(). See the resolved webkit bug: https://bugs.webkit.org/show_bug.cgi?id=178583. While this feature works fine for elements already existing in the DOM, it does not work for dynamically added elements. Please see: https://codepen.io/nonmetalhail/pen/KKZzagd 1852820 1 simon.fraser 2022-03-18 13:52:27 -0700 Thanks for the report. I can reproduce. 1852821 2 webkit-bug-importer 2022-03-18 13:52:39 -0700 <rdar://problem/90502768> 1852830 3 simon.fraser 2022-03-18 14:12:36 -0700 In the bad case we're scrolling via FrameSelection::updateAppearanceAfterLayoutOrStyleChange(). 1852833 4 simon.fraser 2022-03-18 14:17:42 -0700 Layout calls FrameSelection::setNeedsSelectionUpdate() which resets m_selectionRevealMode to SelectionRevealMode::Reveal here: if (innerTextSizeChanged && frame().selection().isFocusedAndActive() && document().focusedElement() == &inputElement()) { // The caps lock indicator was hidden or shown. If it is now visible then it may be occluding // the current selection (say, the caret was after the last character in the text field). // Schedule an update and reveal of the current selection. frame().selection().setNeedsSelectionUpdate(FrameSelection::RevealSelectionAfterUpdate::Forced); } This is trying to ensure that text within the input is scrolled into view, but has the side effect of also triggering page scrolling.