Jürgen Spitzmüller wrote:
rgheck wrote:
Which behavior do you mean?

Opening Editable (not HIGHLY_EDITABLE) inset's dialog with C-i.
My puzzlement concerns why Ctrl-I works to collapse (say) ERT that the cursor is inside in 1.5 but not in current
trunk.

Ah, then we are talking about different issues.

In any event, the attached patch fixes the Ctrl-I problem. It also addresses what looks like it would be a similar problem in the mouse handling code. Comments, anyone?

Richard

Index: BufferView.cpp
===================================================================
--- BufferView.cpp	(revision 23912)
+++ BufferView.cpp	(working copy)
@@ -1301,7 +1301,7 @@
 			}
 		}
 		// if it did not work, try the underlying inset.
-		if (!cur.result().dispatched())
+		if (!inset || !cur.result().dispatched())
 			cur.dispatch(tmpcmd);
 
 		if (!cur.result().dispatched())
@@ -1520,7 +1520,7 @@
 
 	// Now dispatch to the temporary cursor. If the real cursor should
 	// be modified, the inset's dispatch has to do so explicitly.
-	if (!cur.result().dispatched())
+	if (!inset || !cur.result().dispatched())
 		cur.dispatch(cmd);
 
 	// Notify left insets

Reply via email to