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&data=02%7C01%7Caharui%40adobe.com%7Cda2d4142bd6d4b657f0508d671b8a3bb%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C636821435166542668&sdata=wBJVpBSjgeehIc2kVlz4b3MspULaIu0mLpl8nsLTI4A%3D&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&data=02%7C01%7Caharui%40adobe.com%7Cda2d4142bd6d4b657f0508d671b8a3bb%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C636821435166552676&sdata=t%2BDcFPT5fF80h78Jxp4L8H8tKGkEHLAmhNrKbjrkRao%3D&reserved=0 > > >> > > > >> > Thoughts? > > >> > > > >> > > > >> > > > >> > > > >> > > > >> > > > >> > -- > > >> > Carlos Rovira > > >> > > > >> > > > >> > > > https://na01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fabout.me%2Fcarlosrovira&data=02%7C01%7Caharui%40adobe.com%7Cda2d4142bd6d4b657f0508d671b8a3bb%7Cfa7b1b5a7b34438794aed2c178decee1%7C0%7C0%7C636821435166552676&sdata=GWoNPr7omCaO8Mrz9yhV14XwiNsLeJQ5Y3ia0q%2BM7%2Fg%3D&reserved=0 > > >> > > > >> > > > >> > > > >> > > >> > > >> > > > > > > > -- Carlos Rovira http://about.me/carlosrovira