Thanks guys for solve this thing!, it was not clear to me if it was some
bug in the compiler, in bindings or in Jewel components :)

El jue., 3 ene. 2019 a las 23:14, Greg Dove (<greg.d...@gmail.com>)
escribió:

> Yeah, I've made a few XXDataBinding propagations in the past, but I don't
> like doing that blindly and was not ready to test them yet.
>
> I realised it was not optimised. But I think its better working than not,
> and probably the number of bindings with two or more events is low (I
> expect it is, otherwise presumably we would have heard about this earlier).
> So optimising seemed lower priority to me... but if you want to focus on
> that go for it!
>
>
> On Fri, Jan 4, 2019 at 10:59 AM Alex Harui <aha...@adobe.com.invalid>
> wrote:
>
> > OK, thanks.  That made me want to go in and do some refactoring in the
> > code.  Changes like that eventually need propagation to the other
> > XXDataBinding classes and we might want to create a
> > MultipleChangeEventDataBinding option.  GenericBinding will work for now
> > but it is the heaviest last resort.
> >
> > -Alex
> >
> > On 1/3/19, 12:18 PM, "Greg Dove" <greg.d...@gmail.com> wrote:
> >
> >     I took a look at this and just pushed a change to
> ContainerDataBinding
> > that
> >     fixes it.
> >
> >
> >
> >     On Fri, Jan 4, 2019 at 7:59 AM Greg Dove <greg.d...@gmail.com>
> wrote:
> >
> >     >
> >     > Thanks Alex, I missed that - will remove it and retest!
> >     >
> >     >
> >     > On Fri, Jan 4, 2019 at 7:56 AM Alex Harui <aha...@adobe.com.invalid
> >
> >     > wrote:
> >     >
> >     >> Greg, I think Carlos is saying the example should work without the
> >     >> check1.selected=false line, but he had to add it to get the
> example
> > to work.
> >     >>
> >     >> -Alex
> >     >>
> >     >> On 1/3/19, 10:42 AM, "Greg Dove" <greg.d...@gmail.com> wrote:
> >     >>
> >     >>     Carlos, I tried the example and (so far) I don't see the
> issue.
> > I get
> >     >> the
> >     >>     visual selection state of the checkbox and the text content
> > binding
> >     >> both
> >     >>     updating when the popup opens or closes.
> >     >>
> >     >>
> >     >>     On Fri, Jan 4, 2019 at 7:27 AM Alex Harui
> > <aha...@adobe.com.invalid>
> >     >> wrote:
> >     >>
> >     >>     > Is a binding being setup for closePopUp?  Check the debug
> .js
> > files
> >     >> to see
> >     >>     > if that event is listed in the _bindings array.  If it is
> > there,
> >     >> debug into
> >     >>     > it and see if a binding actually gets set up.
> >     >>     >
> >     >>     > -Alex
> >     >>     >
> >     >>     > On 1/3/19, 10:08 AM, "Carlos Rovira" <
> carlosrov...@apache.org
> > >
> >     >> wrote:
> >     >>     >
> >     >>     >     Hi,
> >     >>     >
> >     >>     >     I'm facing a binding issue. This is an example from
> >     >>     > PopUpPlayGround.mxml in
> >     >>     >     Tour de Jewel:
> >     >>     >
> >     >>     >     <j:CheckBox localId="check1" text="open/close the popup:
> >     >> {popup1.open}"
> >     >>     >     selected="{popup1.open}"/>
> >     >>     >
> >     >>     >     <j:PopUp id="popup1" className="somePopUpContent"
> > modal="true"
> >     >>     >                                 open="{check1.selected}"
> >     >>     >
> >  openPopUp="openPopUpHandler(event)"
> >     >>     >
> >     >>  closePopUp="closePopUpHandler(event)" />
> >     >>     >
> >     >>     >     if you check the CheckBok, the popup1 opens and the
> check
> > label
> >     >> shows
> >     >>     >     "open/close
> >     >>     >     the popup: true"
> >     >>     >
> >     >>     >     then closing the popup, makes the label turns to:
> > "open/close
> >     >> the
> >     >>     > popup:
> >     >>     >     false"
> >     >>     >
> >     >>     >     but, here's the problems the CheckBox is not deselected
> > with
> >     >>     >
> >     >>     >     selected="{popup1.open}"
> >     >>     >
> >     >>     >     is this right?
> >     >>     >
> >     >>     >     Notice that I put two event handlers that throws when
> > open and
> >     >> close:
> >     >>     >
> >     >>     >     public function openPopUpHandler(event:Event):void
> >     >>     >             {
> >     >>     >                 trace("[openPopUp] ", event.target,
> > popup1.content);
> >     >>     >             }
> >     >>     >             public function
> > closePopUpHandler(event:Event):void
> >     >>     >             {
> >     >>     >                 trace("[closePopUp] ", event.target,
> >     >> popup1.content);
> >     >>     >                 check1.selected = false;
> >     >>     >             }
> >     >>     >
> >     >>     >     So I'm getting the check unselected when this line runs:
> >     >>     >
> >     >>     >     check1.selected = false;
> >     >>     >
> >     >>     >     But I'm wondering why binding works for popup.open label
> > but no
> >     >> for the
> >     >>     >     check.
> >     >>     >
> >     >>     >     You can check the example here (PopUp section):
> >     >>     >
> >     >>     >
> >     >>     >
> >     >>
> >
> https://na01.safelinks.protection.outlook.com/?url=http%3A%2F%2Froyale.apache.org%2Ftourdejewel%2F&amp;data=02%7C01%7Caharui%40adobe.com%7Cda2d4142bd6d4b657f0508d671b8a3bb%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C636821435166542668&amp;sdata=wBJVpBSjgeehIc2kVlz4b3MspULaIu0mLpl8nsLTI4A%3D&amp;reserved=0
> >     >>     >
> >     >>     >     (ensure is not cached and see the checkbox in the left
> in
> > the
> >     >> "Jewel
> >     >>     > Popup
> >     >>     >     (modal)")
> >     >>     >
> >     >>     >     You can check the complete code here:
> >     >>     >
> >     >>     >
> >     >>     >
> >     >>
> >
> https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Froyale-asjs%2Fblob%2Fdevelop%2Fexamples%2Froyale%2FTourDeJewel%2Fsrc%2Fmain%2Froyale%2FPopUpPlayGround.mxml&amp;data=02%7C01%7Caharui%40adobe.com%7Cda2d4142bd6d4b657f0508d671b8a3bb%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C636821435166552676&amp;sdata=t%2BDcFPT5fF80h78Jxp4L8H8tKGkEHLAmhNrKbjrkRao%3D&amp;reserved=0
> >     >>     >
> >     >>     >     Thoughts?
> >     >>     >
> >     >>     >
> >     >>     >
> >     >>     >
> >     >>     >
> >     >>     >
> >     >>     >     --
> >     >>     >     Carlos Rovira
> >     >>     >
> >     >>     >
> >     >>
> >
> https://na01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fabout.me%2Fcarlosrovira&amp;data=02%7C01%7Caharui%40adobe.com%7Cda2d4142bd6d4b657f0508d671b8a3bb%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C636821435166552676&amp;sdata=GWoNPr7omCaO8Mrz9yhV14XwiNsLeJQ5Y3ia0q%2BM7%2Fg%3D&amp;reserved=0
> >     >>     >
> >     >>     >
> >     >>     >
> >     >>
> >     >>
> >     >>
> >
> >
> >
>


-- 
Carlos Rovira
http://about.me/carlosrovira

Reply via email to