In the aforementioned function, I read:
bool const lfun_ok = lyx::getStatus(FuncRequest(code,
argument)).enabled();
bool const read_only = buffer().isReadonly();okPB->setEnabled(!synchronized_view && widget_ok && !read_only && valid_argument);
Question: why is it necessary to check for read only document, whereas getStatus does it already? What about using lfun_ok instead?
JMarc
