Verbose function names

[fphilipe]

I've been glancing over the codebase and there's one thing that I found to be a bit too verbose. In src/cursor.js there are several methods that have the in their names.

• isAtTheEnd
• isAtTheBeginning
• moveAtTheEnd
• moveAtTheBeginning

In my opinion, these would read nicer without the. Also, if I understood the move methods correctly, they move the cursor before or after an element. Wouldn't it make more sense to name them moveTo{End,Beginning} or maybe even move{Before,After}Element?

[matteoagosti]

@fphilipe I totally agree with the rename you did, names were to verbose, however I'm not that convinced about the single move method as its semantic with the two arguments may lead to confusion (what if one pass both parameters?). However I would rename those in moveBefore and moveAfter, what do you think?

[fphilipe]

Damnit, wanted to add a commit to this PR but pushed that commit to master instead (c7b8eb9). The commit renames the move methods, as you suggested, to moveBefore and moveAfter. Sorry about that 😁

[matteoagosti]

No problem :)

[matteoagosti]

By the way, do the merge once you're OK.

[fphilipe]

OK, I'll just cherry-pick this commit since it's only one 😃

[fphilipe]

Landed in ce4aacb