Re: [lmi] Allow opening cell editor with double-click

[Vaclav Slavik]

On 2012-02-24 13:49, Greg Chicares wrote:
> I applied that changeset (revision 5407, 20120224T1223Z):
>   http://svn.savannah.nongnu.org/viewvc?view=rev&root=lmi&revision=5407
> but Alt-Spacebar still doesn't seem to pull down the system menu.

I'm sorry, I mis-tested the patch (I couldn't reproduce it in
samples/dataview now, so either I tested in that only or made some other
mistake).

Please apply this patch to wx instead:

--- src/generic/datavgen.cpp.293        2012-02-25 13:49:47.000000000 +0100
+++ src/generic/datavgen.cpp    2012-02-25 13:49:51.000000000 +0100
@@ -3558,6 +3558,12 @@
     switch ( event.GetKeyCode() )
     {
         case WXK_RETURN:
+            if ( event.HasModifiers() )
+            {
+                event.Skip();
+                break;
+            }
+            else
             {
                 // Enter activates the item, i.e. sends 
wxEVT_COMMAND_DATAVIEW_ITEM_ACTIVATED to
                 // it. Only if that event is not handled do we activate column 
renderer (which
@@ -3577,6 +3583,12 @@
             }
 
         case WXK_SPACE:
+            if ( event.HasModifiers() )
+            {
+                event.Skip();
+                break;
+            }
+            else
             {
                 // Space toggles activatable items or -- if not activatable --
                 // starts inline editing (this is normally done using F2 on
@@ -3606,6 +3618,12 @@
             }
 
         case WXK_F2:
+            if ( event.HasModifiers() )
+            {
+                event.Skip();
+                break;
+            }
+            else
             {
                 if( !m_selection.empty() )
                 {

(version to apply on top of the bad patch: 
http://trac.wxwidgets.org/changeset/70680)

Vaclav