Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed extra line when duplicating region #80

Merged
merged 1 commit into from
Aug 10, 2021
Merged

Conversation

Yevgnen
Copy link
Contributor

@Yevgnen Yevgnen commented Mar 20, 2021

Replace this placeholder text with a summary of the changes in your PR.
The more detailed you are, the better.

When duplicating region, I think there is an extra newline is include when the cursor is at bol.

This

Screen Shot 2021-03-20 at 9 23 37 PM

will result in

Screen Shot 2021-03-20 at 9 23 45 PM

This PR fixed this.

(I'm not sure whether this is indeed an issue. But following many other library or emacs built in functions, IIRC when the cursor is at bol, they generally don't count that line. I have to 'fix' that line every time I duplicate a region.)


Before submitting a PR make sure the following things have been done (and denote this
by checking the relevant checkboxes):

  • The commits are consistent with our contribution guidelines
  • The new code is not generating bytecode or M-x checkdoc warnings
  • You've updated the changelog (if adding/changing user-visible functionality)
  • You've updated the readme (if adding/changing user-visible functionality)

Thanks!

@failable
Copy link

failable commented Aug 1, 2021

bump ;)

@bbatsov bbatsov merged commit c3b6138 into bbatsov:master Aug 10, 2021
@bbatsov
Copy link
Owner

bbatsov commented Aug 10, 2021

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants