guardian / scribe

DEPRECATED: A rich text editor framework for the web platform

Home Page:http://guardian.github.io/scribe/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

extra arguments to Range.setEndAfter()

mindplay-dk opened this issue · comments

Search the codebase for .setEndAfter(), and you'll see Range.setEndAfter being called in a few places with what appears to be an extra 0 argument after the node?

According to Mozilla's DOM reference, that method takes one argument:

https://developer.mozilla.org/en-US/docs/Web/API/Range/setEndAfter

There are calls to Range.setStartAfter passing an extra second argument as well.

I'll check with @OliverJAsh in case of any undocumentated behaviour but it looks like the additional parameters could be removed