[Webkit-unassigned] [Bug 153361] AX: new lines in content editable elements don't notify accessibility

[bugzilla-daemon at webkit.org]

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

--- Comment #37 from Doug Russell <d_russell at apple.com> ---

> > We should really use the version of indexForVisiblePosition the rest of
> > editing code uses.
> > The one that takes VisiblePosition as the first argument and the scope as
> > the second argument.
> > The variant you're using here won't work in many cases.
> 
> Good to know, this was some of the code I was hoping to get feedback on.
> 
> This version? int indexForVisiblePosition(const VisiblePosition&
> visiblePosition, RefPtr<ContainerNode>& scope)
> 
> How do I determine what to pass to scope?
> 

I was misreading the method signature. Good on this.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.webkit.org/pipermail/webkit-unassigned/attachments/20160302/7dce9baa/attachment-0001.html>