Default insert behavior

[matteoagosti]

No description provided.

[peyerluk]

Since this is related to #20 a tip from Philipe. You can convert an issue into a pull request like so: http://opensoul.org/blog/archives/2012/11/09/convert-a-github-issue-into-a-pull-request/

[matteoagosti]

• Thanks Philipe for the hint!
• Agree on getting rid of the id when present
• What do you mean with last child? I tested it out adding a paragraph at the end of the editor's div (so that should be the last child) and the insertion worked.

There is actually an additional issue with the newly created block: calling cursor.isAtTheBeginning or cursor.isAtTheEnd return both false. Probably this is related to the way I set the cursor when creating the new block as I just trigger the focus event instead of placing the cursor. I'll look into that.

[matteoagosti]

The additional issue I mentioned above wasn't related to the way the cursor is placed (it is actually correct when triggering focus), but to a bug in parser.isStartOffset and parser.isEndOffset. When we have a newly created block it won't contain any children, so the offset should be considered the start and the end. Check commit e42c6ed

[peyerluk]

With last child I meant what you already tested for. I wasn't sure what insertBefore() would do if the second param referenceElement is undefined.

should have checked mdn:

If referenceElement is null, newElement is inserted at the end of the list of child nodes.

[matteoagosti]

Yep, I'm using MDN as well trying not to rely on jQuery :P