emacs-devel
[Top][All Lists]
Advanced

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

Re: emacs-30 ab7c40ea52a: Fix Imenu in 'emacs-news-view-mode'


From: Robert Pluim
Subject: Re: emacs-30 ab7c40ea52a: Fix Imenu in 'emacs-news-view-mode'
Date: Mon, 22 Jul 2024 14:36:55 +0200

>>>>> On Mon, 22 Jul 2024 14:39:28 +0300, Eli Zaretskii <eliz@gnu.org> said:


    Eli> I tried, but it didn't work.  I guess this keymap stuff is above my
    Eli> paygrade (make-composed-keymap? what's that?)  So I ended up doing
    Eli> what did work.

`make-composed-keymap' just makes a new keymap referring to the
keymap(s) you pass it. I guess my mind just twists the right way 🙂

    Eli> Feel free to make any changes you see fit, but please be sure to test
    Eli> them well, since this is the release branch.  One requirement is that
    Eli> if you go back top emacs-news-mode, those key bindings go away.
    Eli> Another requirement is not to disable outline-minor-mode bindings and
    Eli> menus.

Iʼd be stunned if changing major mode messed with a minor mode. But
Iʼll test it.

Should `emacs-news-view-mode' still have bindings for the
`emacs-news-mode' commands that change the buffer? Theyʼd all fail
because of the read-only buffer, but we could remove them. (hmm, the
menus are wrong for `emacs-news-view-mode' anyway. Iʼll fix that as
well).

Robert
-- 



reply via email to

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