[PATCH] Adding LFUN for removing LyX modules
Jean-Marc Lasgouttes
lasgouttes at lyx.org
Sat Jan 8 22:19:44 UTC 2022
Le 22/10/2021 à 23:27, Yuriy Skalko a écrit :
> Thanks for the suggestions, Jean-Marc. Moved the check there. Probably
> it is worth to do the same for layoutModuleCanBeAdded, right? As I see
> now, the check for removing modules in frontend (updateDelPB) is more
> complex. Is that additional check really necessary? With current LyX
> built-in modules situation handled there is impossible.
>
>
>> Cosmetics, but please add * \li Origin here... Pavel
>
> Ok, added.
Hi, Yuriy,
I was about to put this patch in when I noticed that it actually doses
not check whether the module is loaded. This looks as something
interesting to check in getStatus.
Then, it would be great to add relevant error messages with
flag.message("...") when the lfun is disabled. I understand that this is
not done in many places, but we should do that more often.
Could you please at least do the first part? And add an entry about the
new function to lib/RELEASE-NOTE?
Thanks,
JMarc
More information about the lyx-devel
mailing list