[sugar] [PATCH] Journal able to use "open with" for activity bundles

[Jameson "Chema" Quinn]

This change fixes a bug: journal is unable to do anything but "start" a
bundle, even if you have an activity (Develop) which can handle the activity
bundle mime type.

The immediate question is: should "start" be available only by pressing the
button, or should it also be an option in the pulldown? I think that the
answer is an obvious yes.

The longer term question is: should "start" always be the default for
bundles, or should it be possible to have a bundle whose default action is
"Develop"? However, discussion on this larger question should not block
approving the immediate patch.

On Tue, Jun 10, 2008 at 10:02 AM, Eben Eliason <eben.eliason at gmail.com>
wrote:

> I'm confused, actually.  We already have "Start" as the first option
> in the palette for "raw" activities, and as the action listed on the
> button in the detail view.  What is the intent for the change being
> discussed here?
>
> - Eben
>
>
> On Tue, Jun 10, 2008 at 5:57 AM, Tomeu Vizoso <tomeu at tomeuvizoso.net>
> wrote:
> > (sorry about the big delay)
> >
> > On Wed, May 28, 2008 at 9:44 PM, Jameson Chema Quinn
> > <jquinn at cs.oberlin.edu> wrote:
> >> On Wed, May 28, 2008 at 12:16 PM, Tomeu Vizoso <tomeu at tomeuvizoso.net>
> >> wrote:
> >>>
> >>> 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?
> >>
> >> We add a start menu item in case there is a choice to run or open with;
> if
> >> there are choices, they should all be listed, even if one is available
> in a
> >> simpler way (by just clicking). Also note that I plan, in the future, to
> >> make the default behavior (clicking on the button) configurable using
> >> metadata, so that you can have some bundles which are "for editing".
> >
> > Eben, can you comment on this UI change?
> >
> > Thanks,
> >
> > Tomeu
> >
> _______________________________________________
> Sugar mailing list
> Sugar at lists.laptop.org
> http://lists.laptop.org/listinfo/sugar
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.laptop.org/pipermail/sugar/attachments/20080610/bd4da696/attachment-0001.htm