On Wed, 2005-04-20 at 23:12, Martin Vermeer wrote:
> On Wed, Apr 20, 2005 at 05:47:57PM +0300, Martin Vermeer wrote:

...

> A radical, alternative idea would therefore be to just _merge_ the two
> menus, to the extent that they have entries in common; let the menu
> structure reflect the internal commonality. Perhaps a single submenu
> "Tabular features" ?
> 
> Comments?
> 
> - Martin

Try this one on for size ;-)

- Martin

BTW I'll be travelling. Back on Monday.

Index: src/mathed/math_hullinset.C
===================================================================
RCS file: /usr/local/lyx/cvsroot/lyx-devel/src/mathed/math_hullinset.C,v
retrieving revision 1.163
diff -u -p -r1.163 math_hullinset.C
--- src/mathed/math_hullinset.C	24 Mar 2005 16:26:15 -0000	1.163
+++ src/mathed/math_hullinset.C	21 Apr 2005 06:12:02 -0000
@@ -1139,6 +1139,12 @@ bool MathHullInset::getStatus(LCursor & 
 			flag.enabled(false);
 			return true;
 		}
+		if (s == "valign-top" || s == "valign-middle" 
+		 || s == "valign-bottom" || s == "align-left"
+		 || s == "align-center" || s == "align-right") {
+			flag.enabled(false);
+			return true;
+		}
 		return MathGridInset::getStatus(cur, cmd, flag);
 	}
 	default:
Index: src/mathed/math_nestinset.C
===================================================================
RCS file: /usr/local/lyx/cvsroot/lyx-devel/src/mathed/math_nestinset.C,v
retrieving revision 1.152
diff -u -p -r1.152 math_nestinset.C
--- src/mathed/math_nestinset.C	11 Apr 2005 18:48:34 -0000	1.152
+++ src/mathed/math_nestinset.C	21 Apr 2005 06:12:02 -0000
@@ -866,6 +866,9 @@ bool MathNestInset::getStatus(LCursor & 
 	//string tc = "mathnormal";
 	bool ret = true;
 	switch (cmd.action) {
+	case LFUN_TABULAR_FEATURE:
+		flag.enabled(false);
+		break;
 #if 0
 	case LFUN_TABULAR_FEATURE:
 		// FIXME: check temporarily disabled
Index: lib/ui/stdmenus.ui
===================================================================
RCS file: /usr/local/lyx/cvsroot/lyx-devel/lib/ui/stdmenus.ui,v
retrieving revision 1.43
diff -u -p -r1.43 stdmenus.ui
--- lib/ui/stdmenus.ui	18 Apr 2005 18:09:57 -0000	1.43
+++ lib/ui/stdmenus.ui	21 Apr 2005 06:12:02 -0000
@@ -90,6 +90,7 @@ Menuset
 # Mathed b0rkage means these don't work properly
 		OptSubmenu "Table|T" "edit_tabular"
 		OptSubmenu "Math|M" "edit_math"
+		OptSubmenu "Rows & Cols|C" "tabular_features"
 		Separator
 		OptItem "Increase List Depth|I" "depth-increment"
 		OptItem "Decrease List Depth|D" "depth-decrement"
@@ -117,12 +118,6 @@ Menuset
 
 # not much we can do to help here
 	Menu "edit_tabular"
-		Item "Add Row|A" "tabular-feature append-row"
-		Item "Delete Row|w" "tabular-feature delete-row"
-		Separator
-		Item "Add Column|u" "tabular-feature append-column"
-		Item "Delete Column|D" "tabular-feature delete-column"
-		Separator
 		Item "Multicolumn|M" "tabular-feature multicolumn"
 		Separator
 		Item "Top Line|T" "tabular-feature toggle-line-top"
@@ -130,6 +125,15 @@ Menuset
 		Item "Left Line|L" "tabular-feature toggle-line-left"
 		Item "Right Line|R" "tabular-feature toggle-line-right"
 		Separator
+	End
+
+	Menu "tabular_features"
+		Item "Add Row|R" "tabular-feature append-row"
+		Item "Delete Row|D" "tabular-feature delete-row"
+		Separator
+		Item "Add Column|C" "tabular-feature append-column"
+		Item "Delete Column|e" "tabular-feature delete-column"
+		Separator
 		Submenu "Alignment|i" "edit_align"
 	End
 
@@ -152,10 +156,6 @@ Menuset
 		Separator
 		Submenu "Use Computer Algebra System|S" "edit_math_extern"
 		Separator
-		Submenu "Alignment|A" "edit_align"
-		Separator
-		Item "Add Row|R" "tabular-feature append-row"
-		Item "Delete Row|D" "tabular-feature delete-row"
 		Item "Copy Row" "tabular-feature copy-row"
 		Item "Swap Rows" "tabular-feature swap-row"
 		Item "Add Line Above" "tabular-feature add-hline-above"
@@ -163,8 +163,6 @@ Menuset
 		Item "Delete Line Above" "tabular-feature delete-hline-above"
 		Item "Delete Line Below" "tabular-feature delete-hline-below"
 		Separator
-		Item "Add Column|C" "tabular-feature append-column"
-		Item "Delete Column|e" "tabular-feature delete-column"
 		Item "Copy Column" "tabular-feature copy-column"
 		Item "Swap Columns" "tabular-feature swap-column"
 		Item "Add Line to Left" "tabular-feature add-vline-left"

Attachment: signature.asc
Description: This is a digitally signed message part

Reply via email to