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
_______________________________________________
Sugar mailing list
Sugar@lists.laptop.org
http://lists.laptop.org/listinfo/sugar
