To match UIKit the caret should be placed before the text placeholder instead of after it.
<rdar://problem/65295523>
Created attachment 404278 [details] For the bots
Created attachment 404300 [details] Patch
Comment on attachment 404300 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=404300&action=review > Source/WebCore/editing/Editor.cpp:3326 > + m_document.selection().setSelection(VisibleSelection { positionInParentBeforeNode(placeholder.ptr()), SEL_DEFAULT_AFFINITY }, FrameSelection::defaultSetSelectionOptions(UserTriggered)); This is not correct. positionInParentBeforeNode() will assert if the placeholder is not in the document. Will patch this up to check that...
Thanks Wenson!
Created attachment 404380 [details] To Land
Created attachment 404382 [details] To Land
Comment on attachment 404382 [details] To Land Clearing flags on attachment: 404382 Committed r264418: <https://trac.webkit.org/changeset/264418>
All reviewed patches have been landed. Closing bug.