Skip to content

Issue 111 goto last edit#114

Closed
abrookins wants to merge 5 commits into
sublimehq:masterfrom
abrookins:issue_111_goto_last_edit
Closed

Issue 111 goto last edit#114
abrookins wants to merge 5 commits into
sublimehq:masterfrom
abrookins:issue_111_goto_last_edit

Conversation

@abrookins

Copy link
Copy Markdown

Jump to the last edit when a user hits '.

  • ViRecordLastEdit saves the position of edits the user makes, in settings
    • ViGotoLastEdit jumps to the position of the last edit in a view and stores the original cursor position
    • If the user is at the position of the last edit and hits '., ViGotoLastEdit jumps back to the original position

Now with a proper keybinding.

 - ViRecordLastEdit saves the position of edits the user makes, in
   settings

 - ViGotoLastEdit jumps to the position of the last edit in a view and
   stores the original cursor position

 - If the user is at the position of the last edit and hits `'.`, ViGotoLastEdit jumps
   back to the original position
@misfo

misfo commented Apr 5, 2012

Copy link
Copy Markdown
Collaborator

The rest of this looks good, I think. I'll give it a try soon.

My other inline comment isn't showing up here. Weird Github bug?!?! Did you get an email/notification for the previous comment?

@abrookins

Copy link
Copy Markdown
Author

Thanks. I did get your comment - it was on my first pull request. I made a second one after adding the key binding because I wasn't quite sure how to update a pull request with more changes.

I'll take a shot at using add_regions the way you've done on #97, which seems better.

@abrookins

Copy link
Copy Markdown
Author

Updated to use get_regions() and add_regions().

@misfo

misfo commented Apr 6, 2012

Copy link
Copy Markdown
Collaborator

Works well for me 👍

I'll keep using this locally to see if the EventListener causes any performance issues (but I don't see why it would).

@krzkrzkrz

Copy link
Copy Markdown

Any news on this?

@abrookins abrookins closed this Feb 26, 2022
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.

3 participants