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

UX Improvements: Keep Cursor Position & Split Size #130

Open
MarioRicalde opened this issue Aug 9, 2023 · 3 comments
Open

UX Improvements: Keep Cursor Position & Split Size #130

MarioRicalde opened this issue Aug 9, 2023 · 3 comments

Comments

@MarioRicalde
Copy link

When using :Reject the quickfix split will reload causing the following issues:

  • Cursor position is lost, making the user loose context.
  • The split resizes to it's default size, making the user loose even more context.

A way to partially work around this is to setup a mark m a, and then use ' a to go back to a somewhat relevant position.

If there is no way to work around re-rendering the list, perhaps it's possible to separate the marking process from the actually "removing" process.

Related issue: #120

@MarioRicalde
Copy link
Author

The marks workaround seems to be pretty reliable for the time being, and works "good enough" for the flow where you are Selecting Lines and :Rejecting them.

@romainl
Copy link
Owner

romainl commented Aug 9, 2023

Using marks sounds good but I wouldn't want to lo loose the user's marks. This means that the mark must be saved with something like getpos() and restored with something like setpos().

@MarioRicalde
Copy link
Author

Any pointers as to where could someone add those changes in the plugin, I may be able to allocate some time to look into this one.

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

No branches or pull requests

2 participants