Skip to content
New issue

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

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

Already on GitHub? # to your account

improvement for vim navigation #870

Closed
qcfu-bu opened this issue Aug 13, 2024 · 5 comments · Fixed by #875
Closed

improvement for vim navigation #870

qcfu-bu opened this issue Aug 13, 2024 · 5 comments · Fixed by #875
Labels
enhancement New feature or request

Comments

@qcfu-bu
Copy link

qcfu-bu commented Aug 13, 2024

Using vscoq's interpret to point command, the current line of proof that the cursor is on is not interpreted. This causes a bit of inconvenience when using with the vscode vim extension. Basically using vim, it is impossible to place the cursor in such a way that the interpret to point command interprets the current line. This is inconsistent with the behavior of proof general in emacs where the interpret to point includes the line of proof that the cursor is currently on.

@gares
Copy link
Member

gares commented Aug 13, 2024

IIRC PG has an option to change the meaning of "here". But I see your point.
I don't know if the best action is to add such an option, or to add another command, i.e. "interpret to next point".

@rtetley rtetley added the enhancement New feature or request label Aug 14, 2024
@rtetley
Copy link
Collaborator

rtetley commented Aug 14, 2024

I honestly don't have an opinion. Is it important to keep both options ?

@TheoWinterhalter
Copy link

One thing I'm wondering is what would happen if your cursor is in between to sentences, which one would be interpreted as "the point"?

Using a pipe for the cursor:

tac1. | tac2.

Here I would expect "Interpret to point" to only process up to and including tac1 but I would be very surprised if it were to include tac2 as well.

ta|c1. tac2.

There I would be ok with it being the same, ie interpreting tac1 too.

@rtetley
Copy link
Collaborator

rtetley commented Aug 14, 2024

That's a good point. Unfortunately I think currently those spaces would be interpreted as being part of the tac2 command. I have to check particulars.

@qcfu-bu
Copy link
Author

qcfu-bu commented Aug 19, 2024

One thing I'm wondering is what would happen if your cursor is in between to sentences, which one would be interpreted as "the point"?

Using a pipe for the cursor:

tac1. | tac2.

Here I would expect "Interpret to point" to only process up to and including tac1 but I would be very surprised if it were to include tac2 as well.

ta|c1. tac2.

There I would be ok with it being the same, ie interpreting tac1 too.

When I try interpret to point in emacs, it will only include tac1. and not tac2.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
enhancement New feature or request
Projects
None yet
4 participants