[Webkit-unassigned] [Bug 274367] New: `textarea` should not reset the caret position when it is focused by sequential focus navigation

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=274367

            Bug ID: 274367
           Summary: `textarea` should not reset the caret position when it
                    is focused by sequential focus navigation
           Product: WebKit
           Version: Safari Technology Preview
          Hardware: Unspecified
                OS: Unspecified
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: Forms
          Assignee: webkit-unassigned at lists.webkit.org
          Reporter: ahmad.saleem792 at gmail.com
                CC: cdumez at apple.com, lwarlow at igalia.com,
                    wenson_hsieh at apple.com

Hi Team,

While going through Blink's commit, I came across another failing test case (already in tree with failed expectations).

Blink Commit: https://chromium.googlesource.com/chromium/src.git/+/281b9124b31f6aa3c8030f06d25bef969c3964d5

WebKit Source: https://github.com/WebKit/WebKit/blob/dbcbbe87dd98b259cf8815a53c9bc7276892de6f/Source/WebCore/html/HTMLTextAreaElement.cpp#L227

Changing to following make us align and pass failing tests:

void HTMLTextAreaElement::updateFocusAppearance(SelectionRestorationMode restorationMode, SelectionRevealMode revealMode)
{
    if ((restorationMode == SelectionRestorationMode::RestoreOrSelectAll || restorationMode == SelectionRestorationMode::SelectAll) && hasCachedSelection())
        restoreCachedSelection(revealMode, Element::defaultFocusTextStateChangeIntent());
}

___

Just wanted to raise so we can track and fix these issues.

Thanks!

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20240519/43d48fd4/attachment.htm>