emacs-orgmode
[Top][All Lists]
Advanced

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

Re: [O] [patch] Improved block insertion


From: Rasmus
Subject: Re: [O] [patch] Improved block insertion
Date: Sun, 08 Apr 2018 12:59:18 +0200
User-agent: Emacs Gnus

Hi,

Thanks for the comments!

I have fixed up the patches.  

> I don't think the old key-binding should be kept.

OK.

> Suggestion:
>
>   (key (pcase (read-char-exclusive prompt)
>         ((or ?\s ?\t ?\r) ?\t)
>         (char char)))


Ha, actually I used pcase at first but then changed it to something more
simple.  Are there any performance issues with pcase or can it be used
unconditionally?

>> +    (let ((menu-choice (org--insert-structure-template-mks)))
>> +      (if (equal (nth 0 menu-choice) "\t")
>> +      (read-string "Structure type: ")
>> +    (nth 1 menu-choice)))))
>
> (pcase (org--insert-structure-template-mks)
>   (`("\t" . ,_) (read-string "Structure type: "))
>   (`(,_ ,choice . ,_) choice))

Thanks, that’s nice. 

Any worries about pushing the patches now?

Rasmus

-- 
I hear there's rumors on the, uh, Internets. . .

Attachment: 0001-org-macs-Move-org-mks-from-org-capture-to-org-macs.patch
Description: Text Data

Attachment: 0002-org-macs-Make-tab-space-and-RET-equivalent-in-org-mk.patch
Description: Text Data

Attachment: 0003-org-org-structure-template-alist-uses-string-keys.patch
Description: Text Data

Attachment: 0004-org-tempo-Various-improvements.patch
Description: Text Data

Attachment: 0005-org-Change-structure-insertion.patch
Description: Text Data

Attachment: 0006-org-Change-org-insert-structure-template-to-C-c-C.patch
Description: Text Data


reply via email to

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