Skip to content

Common menus - #914

Merged
Matafou merged 4 commits into
ProofGeneral:masterfrom
Matafou:common-menus
Nov 6, 2026
Merged

Matafou merged 4 commits into
ProofGeneral:masterfrom
Matafou:common-menus

Conversation

@Matafou

@Matafou Matafou commented Nov 6, 2026

Copy link
Copy Markdown
Contributor

Coq: the menus of the queries, options, abbrevs and project file shared by all backends

These menus become constants, like coq-compile-menu, coq-hyps-menu and coq-insert-menu, so that other backends can use them:

  • coq-queries.el:
    • coq-queries-menu-entries: Print, Check, About, Other, Store Response and Store Goal. It is a list of entries, not a submenu.
    • coq-other-queries-menu.
    • coq-options-menu.
  • coq-templates.el: coq-abbrevs-menu.
  • coq-project.el: coq-project-menu-entries.

"Set Printing Width" now goes through the common command coq-ask-printing-width, which does the same as before in PG. Otherwise the PG menu is unchanged (checked by comparing it with master's).

  • Fix: "Set Printing Width" (C-c C-a C-w, menu OPTIONS) now sets the width even when coq-auto-adapt-printing-width is nil; before, it silently did nothing.

Test: every command of the shared menus is defined without Proof General. Also a CHANGES entry.

Matafou and others added 4 commits November 6, 2026 16:26
…onstants.

Like coq-compile-menu, coq-hyps-menu and coq-insert-menu: the entries
written in coq-abbrev.el move next to their commands, so that other
backends can use them.  coq-queries.el: coq-queries-menu-entries (the
main queries and Store Goal/Response, at the top level of the PG
menu), coq-other-queries-menu and coq-options-menu.  coq-templates.el:
coq-abbrevs-menu.  coq-project.el: coq-project-menu-entries.  The PG
menu is unchanged.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The command common to all backends, which calls
coq-ask-printing-width-function: for Proof General, the same
coq-ask-adapt-printing-width-and-show as before.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
coq-ask-adapt-printing-width-and-show (C-c C-a C-w, menu OPTIONS -> Set
Printing Width) did nothing when coq-auto-adapt-printing-width was nil:
coq-adapt-printing-width tested it.  New optional argument FORCE of
coq-adapt-printing-width, given by the command asking the width.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@Matafou
Matafou merged commit c98fc14 into ProofGeneral:master Nov 6, 2026
140 checks passed
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.

1 participant