t9md / atom-cursor-history

Cursor position history manager
https://atom.io/packages/cursor-history
MIT License
59 stars 7 forks source link

option to only save one cursor per file #18

Closed farfromrefug closed 8 years ago

farfromrefug commented 8 years ago

I added an option to only save one cursor position (last one) per file. This mostly get a navigation like xcode or such. Something i always wanted!

brunolemos commented 8 years ago

That seems a cool option, but would be more flexible as a new action, like:

cursor-history:next-file
cursor-history:prev-file

So it would keep saving all the cursors but with the possibility to have a shortcut to go to the last cursor of another file.

t9md commented 8 years ago

Thanks for contribution, But I myself don't need this feature, I'll take @brunolemos 's suggestion as compromise. So can't directly merge this PR. Will handle distinct issue #20.

t9md commented 8 years ago

I tried, but providing next-file, prev-file get things very complecated. OK for previous entry, but when go to next It's better to pick latest entry(not oldest entry), but when I try to pick latest entry I have to compact all entries to keep current history index correct.

I don't want to make code complex. So I changed my mind to pick @farfromrefug 's suggestion, but I implement it by myself.

t9md commented 8 years ago

released v0.6.0

farfromrefug commented 8 years ago

Thanks @t9md Glad to see you did it your way ;)