Adding the sugar object chooser call

sugarlabs / sugar-web

Components for Sugar web activities

Apache License 2.0

13 stars 32 forks source link

Closed surajgillespie closed 11 years ago

dnarvaez commented 11 years ago

A few things:

Maybe this should be showObjectChooser instead of display? I haven't seen the verb display used in the sugar API.
You need to rename the message too.
I think it should be activity.show_object_chooser, not datastore.show_object_chooser. I'd keep the datastore namespace for the low level service, not UI related stuff.

dnarvaez commented 11 years ago

Also please rebase when resubmitting patches (see developer.sugarlabs.org).

manuq commented 11 years ago

I agree with all the above.