>> Perhaps the patch should be to make select component do it as a mixin ;)
I don't want to break backwards compatibility, Select nad Checkbox component need to have different JS init and onChange method, I don't see any advantage of using mixin here. > I may be wrong but wasn't that zoneupdate mixin added in 5.3? There is only a new ZoneRefresh mixin. Denis On Aug 12, 2011, at 1:30 AM, Andreas Andreou wrote: > That makes sense to me too - see > http://tapestry.1045711.n5.nabble.com/Parameters-in-implementation-mixins-td3395407.html > > I may be wrong but wasn't that zoneupdate mixin added in 5.3? If > that's the case, then it's > just a matter of using it ... and perhaps deciding what to do with > select (which looks to be > the exception). > > On Fri, Aug 12, 2011 at 01:58, Josh Canfield <[email protected]> wrote: >> Perhaps the patch should be to make select component do it as a mixin ;) >> >> On Thu, Aug 11, 2011 at 1:01 PM, Denis Stepanov >> <[email protected]> wrote: >>>> What about a mixin instead of changing the component themselves? That's >>>> what I'd do. >>> >>> Select component doesn't implement it using a mixin. >>> >>> I would like to have consistent behaviour in core components, if a Select >>> could update a zone on change why other form's component can't. >>> >>> Denis >>> >>> On 11.8.2011, at 21:46, Thiago H. de Paula Figueiredo wrote: >>> >>>> On Thu, 11 Aug 2011 14:41:59 -0300, Denis Stepanov >>>> <[email protected]> wrote: >>>> >>>>> Anyone? >>>> >>>> What about a mixin instead of changing the component themselves? That's >>>> what I'd do. >>>> >>>>> >>>>> On 10.8.2011, at 13:40, Denis Stepanov wrote: >>>>> >>>>>> https://issues.apache.org/jira/browse/TAP5-1602 >>>>>> >>>>>> Someone with commit access please review the patch. >>>>>> >>>>>> Denis >>>>>> >>>>>> On Aug 8, 2011, at 11:20 AM, Denis Stepanov wrote: >>>>>> >>>>>>> Thanks, I have implemented this by using a mixin in our code. >>>>>>> >>>>>>> I just what to have native support for Radio and Checkbox valueChaned >>>>>>> event in the next Tapestry version. >>>>>>> >>>>>>> Denis >>>>>>> >>>>>>> On Aug 8, 2011, at 10:39 AM, Михаил Слободянюк wrote: >>>>>>> >>>>>>>>> >>>>>>>>> If you want use radio as like select try this >>>>>>>>> http://code.google.com/p/orion-cpu/source/browse/trunk/orion-web/src/main/java/ua/orion/web/components/RadioSelect.java >>>>>>>>> >>>>>>>>> I would like to add support for the valueChanged event from >>>>>>>>> RadioGroup and >>>>>>>>>> Checkbox component just like it is in Select component, is there >>>>>>>>>> anyone who >>>>>>>>>> thinks it should be done and eventually will accept a patch? >>>>>>>>>> >>>>>>>>>> >>>>>>>>> Don't forget copy >>>>>>>> http://code.google.com/p/orion-cpu/source/browse/trunk/orion-web/src/main/resources/ua/orion/web/components/RadioSelect.js >>>>>>>> >>>>>>>> Mihail >>>>>>> >>>>>> >>>>> >>>> >>>> >>>> -- >>>> Thiago H. de Paula Figueiredo >>>> Independent Java, Apache Tapestry 5 and Hibernate consultant, developer, >>>> and instructor >>>> Owner, Ars Machina Tecnologia da Informação Ltda. >>>> Consultor, desenvolvedor e instrutor em Java, Tapestry e Hibernate >>>> http://www.arsmachina.com.br >>>> >>>> --------------------------------------------------------------------- >>>> To unsubscribe, e-mail: [email protected] >>>> For additional commands, e-mail: [email protected] >>>> >>> >>> >>> --------------------------------------------------------------------- >>> To unsubscribe, e-mail: [email protected] >>> For additional commands, e-mail: [email protected] >>> >>> >> >> --------------------------------------------------------------------- >> To unsubscribe, e-mail: [email protected] >> For additional commands, e-mail: [email protected] >> >> > > > > -- > Andreas Andreou - [email protected] - http://blog.andyhot.gr > Apache Tapestry PMC / http://chesstu.be owner > Open Source / JEE Consulting > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [email protected] > For additional commands, e-mail: [email protected] > --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
