Subject: Re: bug#76235: [PATCH] Improve prompt for 'project-switch-project'

From: Juri Linkov <[email-masked]>
Date: Tue, 18 Feb 2025 19:13:36 +0200

>>>> Maybe better to make the prompt configurable
>>>> (not defcustom, but only a variable,
>>>> so it's easy to apply properties with face).
>>>> Then I'd like to change it to just "[%s]" to save space.
>>>> A longer explanatory text is more suitable for everyone
>>>> who use the command infrequently.
>>> Could we reuse that variable in both commands?
>> Sharing the same variable would be nice.
>
> Like this?

Whatever, then I'd change it anyway to something with
square brackets since with brackets it looked better for e.g.
'C-x 4 p p' with [other-window] prefix in brackets too:

  [execute in ...]:[other-window]-

(And 'completing-read-multiple' now uses brackets too.)

> +(defvar project-command-prompt (concat
> +                                "Command in `"
> +                                (propertize "%s" 'face 'font-lock-string-face)
> +                                "': ")
> +  "Prompt format to use for choosing a command by key or key sequence.")
> @@ -2032,7 +2038,7 @@ project-any-command
> -         (prompt-format (or prompt-format "[execute in %s]:"))
> +         (prompt-format (or prompt-format project-command-prompt))
> @@ -2247,10 +2253,8 @@ project--switch-project-command
> -                                             (format-message "Command in `%s': "
> -                                                             (propertize
> -                                                              dir 'face
> -                                                              'font-lock-string-face))
> +                                             (format-message project-command-prompt
> +                                                             dir)