[Sugar-devel] [PATCH] Allow view-source to launch external activities

Tomeu Vizoso tomeu at sugarlabs.org
Tue Dec 16 08:18:25 EST 2008


On Sun, Dec 7, 2008 at 5:16 PM, Jameson Quinn <jameson.quinn at gmail.com> wrote:
> This patch has been smoke-tested, but I would like another day to test it
> properly. Unfortunately, since that day is Wednesday, I am sending a
> preliminary version now.

Sorry about the late request. Some questions:

- DocumentData: what about using a dictionary instead of that
versioned scheme? It's quite common in dbus interfaces, in order to
easily cope with format changes.

- Have you checked the UI modifications with Eben? Maybe he could
apply your patch to his jhbuild and give feedback. Or perhaps you
would like to blog about this modification and paste a screenshot so
more people can give opinions?

Sounds pretty good to me, though I will raise some small nitpicks in a
further patch ;)

Thanks,

Tomeu


More information about the Sugar-devel mailing list