Le 17/12/2015 01:36, Guillaume Munch a écrit :
Here's a version with tabular-feature instead of table-modify (and no
change to Shortcuts.lyx given that the docs are now frozen).
A few comments on patches (not all of them for you):
* you still handle inset-modify tabular for documents and of binding
files, right? Richard, how difficult would it be to use lf2lfun in
lyx2lyx to translate info insets? We do not really know what version of
lfuns we have in lyx files, do we?
* does the lfun2lfun update require a new version for our ui/bind files?
* did you update LFUNS.lyx using the relevant script (I guess you did,
but want to be sure).
* in the 3rd patch, why don't you return "false" instead of
case LFUN_TABULAR_FEATURE: {
+ if (!isTable()) {
+ status.clear();
+ status.setUnknown(true);
+ return true;
+ }
Note that I only read the poatches, I did not try them out yet.
JMarc
