[Seaside-dev] Javascript Condition with else decoration

[phil at highoctane.be]

Le 30 sept. 2014 17:19, "Esteban A. Maringolo" <emaringolo at gmail.com> a
écrit :
>
> IMHO JSIfThenElse should be the name of decoration class, and replace
> all the references to it by JSTernary or similar.
>
> But feel free to change it to whatever you like the most, I'm fine as
> long as there is a way to define a JS if/then/else statement using
> decorators.

Is there anyone having Cliff notes of the JS classes ?

It is actually quite hard to figure out how to make them all work together.

Phil
>
> Regards!
>
>
> Esteban A. Maringolo
>
>
> 2014-09-30 12:14 GMT-03:00 Johan Brichau <johan at inceptive.be>:
> > Esteban,
> >
> > As a short note, I forgot to reply that I was a tad confused by the
naming
> > of the method.
> > I would prefer to have something like #classicalthen: else: where the
> > pattern stays <condition> <then> <else>, while your suggestion
transforms it
> > to <then> <condition> <else>
> >
> > So, then we are down to choosing a name, which is probably why you
ended up
> > with #condition: else:
> >
> > Johan
> >
> > On 30 Sep 2014, at 16:48, Johan Brichau <johan at inceptive.be> wrote:
> >
> > Hi Esteban,
> >
> > I picked up your request and was going to look at it but ran out of
time.
> > The best idea is to create a ticket on
> > https://code.google.com/p/seaside/issues/list
> >
> > thanks
> > Johan
> >
> > On 30 Sep 2014, at 15:24, Esteban A. Maringolo <emaringolo at gmail.com>
wrote:
> >
> > No replies to this request in a week.
> > Can I have write access to the repo?
> >
> > Who is the principal maintainer of Seaside?
> >
> > Esteban A. Maringolo
> >
> >
> > 2014-09-22 17:21 GMT-03:00 Esteban A. Maringolo <emaringolo at gmail.com>:
> >
> > I was trying to implement a classical "if {...} else {...}" object using
> > JSDecorations.
> > Then I found there is a JSCondition, but there is no way to describe an
> > "else" statement.
> >
> > Also there is an JSIfThenElse decoration, but it is implemented as a
ternary
> > operator (x ? y : z) instead of 'if(x){ y} else {z}'
> >
> > So I added an extension to JSObject, named #condition:else: and also a
> > JSConditionElse decoration which produces the expected output.
> >
> > Can this be added to the main trunk of the Javascript-Core package?
> >
> > The changes are attached.
> >
> >
> > With these modifications I was able to transform this Javascript:
> >    $('#ajaxId tbody').on( 'click', 'tr', function () {
> >        if ( $(this).hasClass('selected') ) {
> >            $(this).removeClass('selected');
> >        }
> >        else {
> >            $('#ajaxId tr.selected').removeClass('selected');
> >            $(this).addClass('selected');
> >        }
> >    } );
> >
> > Into this:
> >
> > | selClass |
> > selClass := 'selected'.
> > ^ (html jQuery: '#' , self ajaxId , ' tbody')
> > on: 'click'
> > selector: 'tr'
> > do:
> > ((html jQuery this removeClass: selClass)
> > condition: (html jQuery this hasClass: selClass)
> > else:
> > (JSScript new
> > add: ((html jQuery: '#' , self ajaxId , ' tr.' , selClass) removeClass:
> > selClass);
> > add: (html jQuery this addClass: selClass);
> > yourself))
> >
> >
> > Esteban A. Maringolo
> >
> > ps: An alternative solution would be to modify the JSIfThenElse, but
this
> > would require adding returns to the trueStatement and falseStatement of
this
> > class.
> >
> > _______________________________________________
> > 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/20140930/14e33727/attachment.htm