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

[Julian Fitzell]

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
>
>