Ticket #4253: activate-item-toolkit.patch

src/sugar3/graphics/palette.py

@@ -22 +22 @@
 """
 STABLE.
 """
-
 from gi.repository import Gtk
 from gi.repository import GObject
 from gi.repository import Pango
@@ -33 +32 @@
 from sugar3.graphics.icon import Icon
 from sugar3.graphics.palettewindow import PaletteWindow, \
     _PaletteWindowWidget, _PaletteMenuWidget
+from sugar3.graphics.palettemenu import PaletteMenuBox
 
 # DEPRECATED
 # Import these for backwards compatibility
@@ -342 +342 @@
             self._content.remove(self._content.get_children()[0])
 
         if widget is not None:
+            if isinstance(widget, PaletteMenuBox):
+                widget.connect('item-activate', self.__item_activate_cb)
             self._content.add(widget)
             self._content.show()
         else:
@@ -414 +416 @@
 
     menu = GObject.property(type=object, getter=get_menu)
 
+    def __item_activate_cb(self, widget):
+        self.popdown(immediate=True)
+
 
 class PaletteActionBar(Gtk.HButtonBox):
 

src/sugar3/graphics/palettemenu.py

@@ -24 +24 @@
 
 
 class PaletteMenuBox(Gtk.VBox):
+
+    __gsignals__ = {
+        'item-activate': (GObject.SignalFlags.RUN_FIRST, None, [])
+    }
+
     def __init__(self):
         Gtk.VBox.__init__(self)
 
@@ -37 +42 @@
             item = self._wrap_widget(item_or_widget, horizontal_padding,
                                      vertical_padding)
 
+        if isinstance(item, PaletteMenuItem):
+            item.connect('activate', self.__item_activated_cb)
+
         self.pack_start(item, False, False, 0)
 
     def _wrap_widget(self, widget, horizontal_padding, vertical_padding):
@@ -56 +64 @@
         hbox.pack_start(widget, True, True, horizontal_padding)
         return vbox
 
+    def __item_activated_cb(self, item):
+        self.emit('item-activate')
+
 
 class PaletteMenuItemSeparator(Gtk.EventBox):
     """Contains a HSeparator and has the proper height for the menu."""