Juergen Spitzmueller wrote:
> > It is probably this undispatched that makes the cursor quit. I'd
> > propose to remove it (after testing).
>
> Excellent intuition. Your suspicion holds true.

Now that I removed it, I remember why I put it there just a few weeks ago 
(sic!): in order to fix bug 2526.
Attached is a patch that fixes bug 2556 for tabulars while leaving the cursor 
inside the cell.

OK to apply, Jean-Marc?

Jürgen
Index: src/insets/insettabular.C
===================================================================
--- src/insets/insettabular.C	(Revision 13906)
+++ src/insets/insettabular.C	(Arbeitskopie)
@@ -473,6 +473,9 @@ void InsetTabular::doDispatch(LCursor & 
 			break;
 		}
 
+		if (cmd.button() == mouse_button::button3)
+			bvcur.setCursor(cur);
+
 		break;
 
 	case LFUN_MOUSE_MOTION:
@@ -490,10 +493,8 @@ void InsetTabular::doDispatch(LCursor & 
 
 	case LFUN_MOUSE_RELEASE:
 		//lyxerr << "# InsetTabular::MouseRelease\n" << bvcur << endl;
-		if (cmd.button() == mouse_button::button3) {
-			bvcur.setCursor(cur);
+		if (cmd.button() == mouse_button::button3)
 			InsetTabularMailer(*this).showDialog(&cur.bv());
-		}
 		break;
 
 	case LFUN_CELL_BACKWARD:

Reply via email to