Ticket #1268: sugar-1268.patch

src/sugar/graphics/palette.py

@@ -195 +195 @@
 
         self._secondary_anim.start()
 
+    def on_popdown(self):
+        # to suppress glitches while later re-opening
+        self.set_palette_state(self.PRIMARY)
+
     def on_enter(self, event):
         PaletteWindow.on_enter(self, event)
         self._secondary_anim.start()

src/sugar/graphics/palettewindow.py

@@ -333 +333 @@
         if not immediate:
             self._popdown_anim.start()
         else:
-            self._popdown_anim.stop()
+            self.on_popdown()
+            self.size_request()
             self.hide()
 
+    def on_popdown(self):
+        self._popdown_anim.stop()
+
     def on_invoker_enter(self):
         self._mouse_detector.start()
 
@@ -414 +418 @@
 
     def next_frame(self, current):
         if current == 1.0:
-            self._palette.show()
-            # we have to invoke update_position() twice
-            # since WM could ignore first move() request
-            self._palette.update_position()
+            self._palette.popup(immediate=True)
 
 
 class _PopdownAnimation(animator.Animation):
@@ -428 +429 @@
 
     def next_frame(self, current):
         if current == 1.0:
-            self._palette.hide()
+            self._palette.popdown(immediate=True)
 
 
 class Invoker(gobject.GObject):