[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [O] [Bug] M-S Ret keybinding translated to M-Ret
From: |
Matt Lundin |
Subject: |
Re: [O] [Bug] M-S Ret keybinding translated to M-Ret |
Date: |
Thu, 08 Feb 2018 17:51:32 -0600 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.3 (gnu/linux) |
Allen Li <address@hidden> writes:
> On Wed, Feb 7, 2018 at 1:47 PM Matt Lundin <address@hidden> wrote:
>
>> Matt Lundin <address@hidden> writes:
>
>> > I discovered that when trying to insert a new item with a checkbox using
>> > M-S Return (org-insert-todo-heading), the keybinding is "translated" to
>> > M-Return (org-meta-return). As a result, all I get is simple list item
>> > without a checkbox.
>> >
>
>> The attached patch fixes this issue.
>
> I'm pretty sure your patch would break those binding in terminal Emacs
> completely, since <return>
> cannot be typed in the terminal and Emacs does not remap these
> bindings.
Thanks for pointing this out. I hope someone who knows emacs keybindings
better can take the lead here. I was just following the example of the
following commit, which uses "<return>".
,----
| commit c3fffcd8d21f248f946e2d39d4bde3ce80dcf6d4
| Author: Nicolas Goaziou <address@hidden>
| Date: Mon Jan 15 22:59:40 2018 +0100
|
| Fix C-<return> binding
`----
> Furthermore, I cannot reproduce your issue on Org mode version 9.1.6
> (9.1.6-33-g5b4a71-elpaplus)
That is because the version in elpa not have the problematic commit yet.
The elpa version you are using still uses old way of defining these
keys:
(org-defkey org-mode-map [(meta shift return)] 'org-insert-todo-heading)
The problematic commit (6965098a5357b09575a1119074545eb708cf5944)
changed that line to the following (along with several related changes):
(org-defkey org-mode-map (kbd "M-S-RET") #'org-insert-todo-heading)
Could you evaluate this latter line and see if you can reproduce the
problem.
Thanks,
Matt