Merge lp:~tualatrix/acire/double-click-to-start into lp:acire

[Ding Zhou]

Hi, I've made a change to add support for double click on the snippet to execute it.

I hope you like it and merge it into the main branch.

[Jono Bacon (jonobacon)]

Thanks for the merge request! I am actually in the process of hacking Acire to work in a more scalable way internally, so your code would not apply to these changes, but while I am making these changes I will apply a double-click to run feature. Thanks!

review: Disapprove

Unmerged revisions

39. By Ding Zhou on 2010-03-10

Add support for double click to run the snippets

=== modified file 'bin/acire'
--- bin/acire	2010-03-03 00:20:16 +0000
+++ bin/acire	2010-03-10 09:10:29 +0000
@@ -197,6 +197,10 @@
         cat = self.categories_combo.get_active_text()
         self.update_snippets_treeview(cat)
 
+    def snippet_double_clicked(self, widget, path, column):
+        """Run an currently selected snippet when double click it"""
+
+        self.run_snippet(self, widget)
                             
     def snippet_selected(self, widget, data=None):
         """Snippet is selected from the treeview"""
@@ -205,7 +209,10 @@
         (model, it) = selection.get_selected()
 
         # set the current snippet variables
-        self.current_name = self.snippets_model.get_value(it, 1)
+        current_name = self.snippets_model.get_value(it, 1)
+        if self.current_name == current_name:
+            return
+        self.current_name = current_name
         self.current_filename = self.snippets_model.get_value(it, 2)
         self.current_description = self.snippets_model.get_value(it, 3)
 

=== modified file 'data/ui/AcireWindow.ui'
--- data/ui/AcireWindow.ui	2010-03-02 22:11:07 +0000
+++ data/ui/AcireWindow.ui	2010-03-10 09:10:29 +0000
@@ -108,6 +108,7 @@
                         <property name="search_column">1</property>
                         <property name="tooltip_column">3</property>
                         <signal name="cursor_changed" handler="snippet_selected"/>
+			<signal name="row-activated" handler="snippet_double_clicked"/>
                       </object>
                     </child>
                   </object>