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

[Tomeu Vizoso]

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