help-gnu-emacs
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Any infrastructure to select one of a few options in Emacs core?


From: John Mastro
Subject: Re: Any infrastructure to select one of a few options in Emacs core?
Date: Thu, 19 May 2016 18:16:22 -0700

Marcin Borkowski <mbork@mbork.pl> wrote:
> I went with this code:
>
> --8<---------------cut here---------------start------------->8---
> (defun select-choice (message choices)
>   "Display a buffer with MESSAGE and then CHOICES.
> Each element of CHOICES should be a cons cell, whose car is the
> result and the cdr is the description.  The user may select one
> by pressing a digit."
>   ;; TODO: This function is way too simple and not fool-proof!
>   (if (> (length choices) 9)
>       (error "Too many choices!")
>     (save-window-excursion
>       (pop-to-buffer " *amsrefs doi selection*" t t)
>       (delete-other-windows)            ; TODO: this should be fixed
>       (erase-buffer)
>       (insert message "\n\n")
>       (let ((i 0))
>         (dolist (choice choices)
>           (setq i (1+ i))
>           (insert (format "%d. %s\n\n" i (cdr choice)))))
>       (let ((key (read-key)))
>         (if (<= 49 key 57)
>             (car (nth (- key 49) choices))
>           (error "Wrong key pressed!"))))))
> --8<---------------cut here---------------end--------------->8---
>
> As it is said in the comment, it is far from ideal, but the whole
> program I've been writing for my friend is one big quick hack, so
> I didn't bother too much.

I've sometimes wanted something like this too.

I played around with a few cosmetic changes to your function and ended
up with this:

(defun read-choice (prompt choices)
  (if (> (length choices) 9)
      (error "Too many choices (max 9)")
    (let ((cursor-in-echo-area t)
          (prompt (if (get-text-property 0 'face prompt)
                      prompt
                    (propertize prompt
                                'face
                                'minibuffer-prompt))))
      (save-window-excursion
        (pop-to-buffer " *Read choice*" t t)
        (setq-local cursor-type nil)
        (erase-buffer)
        (let ((i 0))
          (dolist (choice choices)
            (setq i (1+ i))
            (insert (propertize (number-to-string i)
                                'face
                                'font-lock-variable-name-face)
                    (format " %s\n" (cdr choice))))
          (goto-char (point-min))
          (fit-window-to-buffer)
          (let ((key (- (read-key prompt) 48)))
            (if (<= 1 key i)
                (car (nth (1- key) choices))
              (user-error "Invalid key"))))))))

Same functionality but posted anyway in the spirit of sharing :)

Why did you find `delete-other-windows' necessary? (Your TODO mentions
that it should be "fixed", so I assume you would have preferred not to
use it.)

        John



reply via email to

[Prev in Thread] Current Thread [Next in Thread]