Alfredo Braunstein wrote: > But I think the behaviour would be different: in the cvs code the selection > is rectangular, in yours the selection is like in text:
You're (unfortunately) right. This different behaviour was not intended (and is counter-intuitive). So after all, the correct fix for the UNSET_ALL_LINES stuff is as simple as the attached. Jürgen
Index: insettabular.C =================================================================== RCS file: /usr/local/lyx/cvsroot/lyx-devel/src/insets/insettabular.C,v retrieving revision 1.445 diff -u -r1.445 insettabular.C --- insettabular.C 25 Nov 2004 19:13:05 -0000 1.445 +++ insettabular.C 28 Nov 2004 11:42:29 -0000 @@ -1352,12 +1352,10 @@ case LyXTabular::SET_ALL_LINES: setLines = true; case LyXTabular::UNSET_ALL_LINES: -#if 0 for (int i = sel_row_start; i <= sel_row_end; ++i) for (int j = sel_col_start; j <= sel_col_end; ++j) tabular.setAllLines( tabular.getCellNumber(i,j), setLines); -#endif break; case LyXTabular::SET_LONGTABULAR:
