Re: [FlexJS] PAYG and dataGroups

2017-07-23 Thread Alex Harui
Different variants of datagroup listview etc can be smarter about data changes. The listview could also share its index properties with beads that know? how to deal with data changes. Get Outlook for Android From: Harbs Sent: Sunday, Jul

Re: [FlexJS] stopImmediatePropagation

2017-07-23 Thread Harbs
Good. I confirmed that this survives minification. > On Jul 24, 2017, at 12:41 AM, Harbs wrote: > > Oh.It seems like there’s an official way to do this: > BrowserEvent.prototpye.getBrowserEvent() > >> On Jul 24, 2017, at 12:34 AM, Harbs > > wrote: >> >> Bah! >> >

Re: [FlexJS] stopImmediatePropagation

2017-07-23 Thread Harbs
Oh.It seems like there’s an official way to do this: BrowserEvent.prototpye.getBrowserEvent() > On Jul 24, 2017, at 12:34 AM, Harbs wrote: > > Bah! > > I just discovered another issue with this branch. > > The goog.BrowserEvent “event_” property is renamed when the code is minified. > Without

Re: [FlexJS] stopImmediatePropagation

2017-07-23 Thread Harbs
Bah! I just discovered another issue with this branch. The goog.BrowserEvent “event_” property is renamed when the code is minified. Without hacking the goog code, using that property is not gonna work… > On Jul 23, 2017, at 1:44 PM, Harbs wrote: > > I feel like I went down the rabbit hole wi

[FlexJS] PAYG and dataGroups

2017-07-23 Thread Harbs
I just ran into a bug in my app as follows: ListView has the following function: protected function rollOverIndexChangeHandler(event:Event):void { if (lastRollOverIndex != -1) { var ir:ISelectableItemRenderer = dataGroup.getItemRendererForIndex(lastRollOverIndex)

[FlexJS]E4X filter functions

2017-07-23 Thread Harbs
I found a case where Flacon is not producing useable output for E4X filters: var folderFolders:XMLList = assetXML.folder.(key.indexOf(folder.key) == 0); var folderImages:XMLList = assetXML.image.(key.indexOf(folder.key) == 0); Outputs: var /** @type {XMLList} */ folderFolders = this.assetXML.chi

Re: [FlexJS] stopImmediatePropagation

2017-07-23 Thread Harbs
I feel like I went down the rabbit hole with this… I think MouseEvent needs to work much like BrowserEvent in that it proxies to the underlying event to get properties, coordinates, etc. What I’m not really sure about is what happens if you want to dispatch a MouseEvent on an element? (i.e. myB

Re: [FlexJS]TileLayout

2017-07-23 Thread Harbs
Agreed, but I think the defaults should match Flash between and HTML. He did not set stretch. Nothing was set. “stretch” is the default css. > On Jul 23, 2017, at 12:23 PM, Alex Harui wrote: > > There doesn't have to be one TileLayout. There can be > TileLayoutWithFlexStart, TileLayoutWithStre

Re: git commit: [flex-asjs] [refs/heads/feature/browser-event] - goog.Event assumes the getter can be called on the event. Default to the super getter if wrappedEvent is not set (yet).

2017-07-23 Thread Alex Harui
It may not be a getter in goog.Event and just a var instead. It might be ok to modify the typedefs to make it a getter. HTH, -Alex On 7/23/17, 1:59 AM, "Harbs" wrote: >This commit should theoretically work, but it looks like there’s a weird >Falcon bug here. > >When outputting the getters for

Re: [FlexJS]TileLayout

2017-07-23 Thread Alex Harui
There doesn't have to be one TileLayout. There can be TileLayoutWithFlexStart, TileLayoutWithStretch, and TileLayoutWithAlignContent with a property that allows you to set it. Peter may remember why he chose "stretch". -Alex On 7/23/17, 1:28 AM, "Harbs" wrote: >I just discovered something whi

Re: git commit: [flex-asjs] [refs/heads/feature/browser-event] - goog.Event assumes the getter can be called on the event. Default to the super getter if wrappedEvent is not set (yet).

2017-07-23 Thread Harbs
This commit should theoretically work, but it looks like there’s a weird Falcon bug here. When outputting the getters for target and currentTarget, the “super” reference is being dropped. I’m assuming it’s because org.apache.flex.events.Event does not actually declare target and currentTarget.

[FlexJS]TileLayout

2017-07-23 Thread Harbs
I just discovered something which is worth bringing up: The default for css flex layouts is align-content: stretch This adds space between elements if the total height (or width) of the elements are less than the parent. I don’t think that’s an appropriate default for a tile layout. The layout