Skip to content

Add support for 'repeat-mode'#867

Open
username231419 wants to merge 2 commits into
ProofGeneral:masterfrom
username231419:master
Open

Add support for 'repeat-mode'#867
username231419 wants to merge 2 commits into
ProofGeneral:masterfrom
username231419:master

Conversation

@username231419

Copy link
Copy Markdown

Emacs has a repeat-mode that you can use to avoid typing in the prefix of a keychord for related commands. Eg. when enabled you can just press o after C-x o to switch to then next window. This adds similar support for at least two commands that I find useful when loading and undoing commands in a Rocq buffer.

* coq.el (coq-guess-or-ask-for-string): Pass a non-nil value for
INHERIT-INPUT when invoking 'read-string'.

@Matafou Matafou left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks like a nice idea.

Comment thread coq/coq.el
(read-string
(if guess (concat s " (default " guess "): ") (concat s ": "))
nil 'proof-minibuffer-history guess)))
nil 'proof-minibuffer-history guess t)))

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this related to the repeat-mode?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You are right, it is not. This is just an unrelated change I had commuted and accidentally pushed in this PR. Should I remove it, because I would have also wanted to submit it as well anyway.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is probably better to do another PR. But we can leave it here. What is the recommended policy for this?

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