Helge Hafting wrote:

> While working on a table, I noticed that I could not set
> or reset table borders using the table dialog. Making
> sure I clicked on the cell containing the cursor did not helt.
> The "set" and "clear" buttons were grayed out.

Thanks for spotting this, it is caused by my getStatus patch. The attached
patch (which I am applying right now) fixes that.
But this behaviour shows also that the tabular dialog is broken (ok, we know
that already ;-) )
It should have cancel and ok buttons (currently the border buttons are used
as these by the button controller).


Georg
Index: src/insets/ChangeLog
===================================================================
RCS file: /usr/local/lyx/cvsroot/lyx-devel/src/insets/ChangeLog,v
retrieving revision 1.1128
diff -u -p -r1.1128 ChangeLog
--- src/insets/ChangeLog	14 Apr 2005 10:19:38 -0000	1.1128
+++ src/insets/ChangeLog	15 Apr 2005 08:06:14 -0000
@@ -1,3 +1,7 @@
+2005-04-15  Georg Baum  <[EMAIL PROTECTED]>
+
+	* insettabular.C (getStatus): handle LFUN_INSET_MODIFY
+
 2005-04-11  Jean-Marc Lasgouttes  <[EMAIL PROTECTED]>
 
 	* insetfloat.C (InsetFloat): 
Index: src/insets/insettabular.C
===================================================================
RCS file: /usr/local/lyx/cvsroot/lyx-devel/src/insets/insettabular.C,v
retrieving revision 1.461
diff -u -p -r1.461 insettabular.C
--- src/insets/insettabular.C	13 Apr 2005 09:43:58 -0000	1.461
+++ src/insets/insettabular.C	15 Apr 2005 08:06:16 -0000
@@ -987,6 +994,13 @@ bool InsetTabular::getStatus(LCursor & c
 		}
 	}
 
+	case LFUN_INSET_MODIFY:
+		if (translate(cmd.getArg(0)) == TABULAR_CODE) {
+			status.enabled(true);
+			return true;
+		}
+		// Fall through
+
 	default:
 		// we try to handle this event in the insets dispatch function.
 		return cell(cur.idx())->getStatus(cur, cmd, status);

Reply via email to