[Seaside-dev] Re: [Seaside Commits] Seaside 3.0: Seaside-Tools-Web-obi.84.mcz

[Nick Ager]

Julian says:

Name: Seaside-Tools-Web-Nick.88
Author: Nick
Time: 13 June 2010, 10:11:32 pm
UUID: d62c23e6-967d-478a-add3-12ba15b45d1f
Ancestors: Seaside-Tools-Web-Nick.87

(julian at the keyboard)
Made configuration actions operate on the current dispatcher, not the
default dispatcher. Remove + expand links. Cleaned up title and path
generation to make sure it includes the selected handler and does not use
the serverPath, etc. Added Clear Default action and Open Handler action.


On 13 June 2010 09:29, Julian Fitzell <jfitzell at gmail.com> wrote:

> Oh... I don't know; that's certainly not how it was originally
> intended to be used. It was intended to simply store the name of the
> dispatcher. If you set the root's default to 'javascript' and the
> javascript dispatcher's default to 'jquery', you'd have the same
> effect, no?
>
> Personally, I think I'd prefer having additional UI support for
> setting the default handler for each dispatcher in the chain if that
> seems like an important feature. But maybe others have another
> opinion?
>
> Julian
>
> On Sun, Jun 13, 2010 at 8:12 AM, Gerhard Obermann <obi068 at gmail.com>
> wrote:
> > Fixed with Seaside-Tools-Web-obi.85.
> >
> > Still not sure if the default dispatcher should be changed if I change
> the
> > default application to e.g. JQuery which is not part of the root
> dispatcher.
> > At the moment the default dispatcher would use "/javascript/jquery" as
> > default name for the JQuery application.
> >
> > Gerhard
> >
> >
> >
> > On Sat, Jun 12, 2010 at 11:36 PM, Julian Fitzell <jfitzell at gmail.com>
> wrote:
> >>
> >> I don't think that can be right... #url will return a full URL if
> >> serverName, etc. are set... why not use WADispatcher>>handlerAt:
> >> combined with #defaultName to get the default dispatcher and compare
> >> for identity? I'm also a bit confused by the use of #defaultDispatcher
> >> here... don't we need to be checking the default name of whatever
> >> dispatcher we're currently viewing?
> >>
> >> Also, there's no EntryPoint class anymore so the parameter name is a
> >> bit confusing. :)
> >>
> >> Julian
> >>
> >> On Sat, Jun 12, 2010 at 8:36 PM,
> >> <squeak-dev-noreply at lists.squeakfoundation.org> wrote:
> >> > Gerhard Obermann uploaded a new version of Seaside-Tools-Web to
> project
> >> > Seaside 3.0:
> >> > http://www.squeaksource.com/Seaside30/Seaside-Tools-Web-obi.84.mcz
> >> >
> >> > ==================== Summary ====================
> >> >
> >> > Name: Seaside-Tools-Web-obi.84
> >> > Author: obi
> >> > Time: 12 June 2010, 7:40:06 pm
> >> > UUID: b06710cd-7b0a-4c4f-8c1e-f2243f40df05
> >> > Ancestors: Seaside-Tools-Web-obi.83
> >> >
> >> > - compare urls instead of strings to get the default application
> >> >
> >> > =============== Diff against Seaside-Tools-Web-obi.83 ===============
> >> >
> >> > Item was changed:
> >> >  ----- Method: WAConfigurationBrowser>>renderRowName:named:on: (in
> >> > category 'rendering-rows') -----
> >> >  renderRowName: anEntryPoint named: aString on: html
> >> >        html anchor
> >> >                class: 'selected' if: anEntryPoint = selection;
> >> >                callback: [ self configure: anEntryPoint ];
> >> >                with: [
> >> >                        anEntryPoint isDispatcher
> >> >                                ifTrue: [        html  text: aString ,
> >> > '/' ]
> >> >                                ifFalse: [
> >> >                                        html text: aString.
> >> > +                                       (WAUrl absolute: self
> >> > defaultDispatcher defaultName) = anEntryPoint url
> >> > -                                       self defaultDispatcher
> >> > defaultName = anEntryPoint url greaseString
> >> >                                                 ifTrue: [ html span
> >> >
> >> >  title: 'Default application';
> >> >
> >> >  with: ' (*)'] ] ]!
> >> >
> >> >
> >> > _______________________________________________
> >> > commits mailing list
> >> > To unsubscribe, email commits-leave at lists.seaside.st
> >> > http://lists.seaside.st/listinfo/commits
> >> >
> >> _______________________________________________
> >> seaside-dev mailing list
> >> seaside-dev at lists.squeakfoundation.org
> >> http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev
> >
> >
> > _______________________________________________
> > seaside-dev mailing list
> > seaside-dev at lists.squeakfoundation.org
> > http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev
> >
> >
> _______________________________________________
> seaside-dev mailing list
> seaside-dev at lists.squeakfoundation.org
> http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.squeakfoundation.org/pipermail/seaside-dev/attachments/20100613/391515d1/attachment.htm