Patch #1233 2006-07-23 20:53

tdragon

Double-click or Enter runs wizard/template
Download
1233-Double_click_o.patch (2.1 KB)
Category
Application::Refinement
Status
Closed
Close date
2006-07-26 18:25
Assigned to
 
Index: src/sdk/newfromtemplatedlg.cpp
===================================================================
--- src/sdk/newfromtemplatedlg.cpp    (revision 2778)
+++ src/sdk/newfromtemplatedlg.cpp    (working copy)
@@ -63,18 +63,22 @@
 
     // projects
     EVT_LIST_ITEM_RIGHT_CLICK(XRCID("listProjects"), NewFromTemplateDlg::OnListRightClick)
+    EVT_LIST_ITEM_ACTIVATED(XRCID("listProjects"), NewFromTemplateDlg::OnListActivate)
     EVT_CHOICE(XRCID("cmbProjectCategories"), NewFromTemplateDlg::OnCategoryChanged)
 
     // targets
     EVT_LIST_ITEM_RIGHT_CLICK(XRCID("listTargets"), NewFromTemplateDlg::OnListRightClick)
+    EVT_LIST_ITEM_ACTIVATED(XRCID("listTargets"), NewFromTemplateDlg::OnListActivate)
     EVT_CHOICE(XRCID("cmbTargetCategories"), NewFromTemplateDlg::OnCategoryChanged)
 
     // files
     EVT_LIST_ITEM_RIGHT_CLICK(XRCID("listFiles"), NewFromTemplateDlg::OnListRightClick)
+    EVT_LIST_ITEM_ACTIVATED(XRCID("listFiles"), NewFromTemplateDlg::OnListActivate)
     EVT_CHOICE(XRCID("cmbFileCategories"), NewFromTemplateDlg::OnCategoryChanged)
 
     // workspaces
     EVT_LIST_ITEM_RIGHT_CLICK(XRCID("listCustoms"), NewFromTemplateDlg::OnListRightClick)
+    EVT_LIST_ITEM_ACTIVATED(XRCID("listCustoms"), NewFromTemplateDlg::OnListActivate)
     EVT_CHOICE(XRCID("cmbCustomCategories"), NewFromTemplateDlg::OnCategoryChanged)
 
     // context menu for wizard scripts
@@ -347,6 +351,11 @@
     delete menu;
 }
 
+void NewFromTemplateDlg::OnListActivate(wxListEvent& event)
+{
+    this->EndModal(wxID_OK);
+}
+
 void NewFromTemplateDlg::OnCategoryChanged(wxCommandEvent& event)
 {
     BuildListFor(GetVisibleOutputType(), GetVisibleListCtrl(), GetVisibleCategory());
Index: src/sdk/newfromtemplatedlg.h
===================================================================
--- src/sdk/newfromtemplatedlg.h    (revision 2778)
+++ src/sdk/newfromtemplatedlg.h    (working copy)
@@ -33,6 +33,7 @@
         void ClearList();
         void ClearListFor(wxListCtrl* list);
         void OnListRightClick(wxListEvent& event);
+        void OnListActivate(wxListEvent& event);
         void OnCategoryChanged(wxCommandEvent& event);
         void OnEditScript(wxCommandEvent& event);
         void OnEditGlobalScript(wxCommandEvent& event);
tdragon 2006-07-23 21:10

Seems like nearly every time I go to run one of the File wizards, I forget and try to double-click it. :P

The patch is simple enough: Add an OnActivate callback that calls EndModal, and let the EVT_LIST_ITEM_ACTIVATED event take care of the rest.

The effect is marginally different from clicking Go in that the internal handler for the Go button also calls Validate and TransferDataFromWindow, but since this dialog doesn't use any of that, EndModal is enough. A more permanently correct solution would copy the internal OnOK handler.

killerbot 2006-07-26 18:25

applied rev 2788

thanks for the help !