Fix for #689 and improve blank line handling for the ip/ap text-objects #696
Conversation
Any feedback on this? |
I think it looks great. Definitely works better now. I notice that hitting undo after 'dap' or 'dip' leaves the cursor at the end of the deleted region. Vim leaves the cursor at the beginning of the deleted region. Might be worth opening a new issue for this though, not sure how deep this problem runs. Ideally you'd combine some of these commits so the changes are more obvious. |
Seems to need rebasing please. |
… ip/ap text-objects
Refactored and rebased. |
please remove |
Sneaky .swp removed :) |
@@ -205,15 +205,36 @@ describe "TextObjects", -> | |||
expect(editorElement.classList.contains('normal-mode')).toBe(true) | |||
|
|||
it "selects inside the current paragraph in visual mode", -> | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this seems to be an unnecessary empty line; with or without it I'm still recommending an immediate merge for this PR 8-)
Great work @MarkusSN; thanks for fixing this! |
Fix for #689 and improve blank line handling for the ip/ap text-objects
Fixes #689. Also makes the handling of blank lines, and lines that only contain whitespace more like in vim, e.g. you can now select the lines between two paragraphs by doing 'vip' .