[sugar] [PATCH] Journal able to use "open with" for activity bundles
Tomeu Vizoso
tomeu at tomeuvizoso.net
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
mailing list