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

Reply via email to