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
--
lyx-devel mailing list
lyx-devel@lists.lyx.org
http://lists.lyx.org/mailman/listinfo/lyx-devel

Reply via email to