[sugar] [PATCH] Journal able to use "open with" for activity bundles

Tomeu Vizoso tomeu
Wed May 28 14:16:28 EDT 2008


Hi,

+        if self._jobject.is_activity_bundle():
+            menu_item = MenuItem(_('Start'))
+            menu_item.connect('activate', self._resume_menu_item_activate_cb,
+                              None)
+            palette.menu.append(menu_item)
+            menu_item.show()

Why are we adding a Start menu item? How is it different from clicking
on the button?

-            self._jobject.resume(service_name)
+            if not service_name:
+                self._jobject.resume()
+            else:
+                self._jobject.resume(service_name)

Why is this needed? self._jobject.resume(service_name) when
service_name == None should be the same as self._jobject.resume().

The rest seems good to me.

Thanks,

Tomeu



More information about the Sugar-devel mailing list