[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