Re: [FlexJS] BarChart component added - render issue

2014-02-12 Thread Erik de Bruin
I'm not going to be able to look into this before the end of the week. Can you please file a JIRA issue and assign it to me, so it doesn't get lost in the deluge of email that is currently joining the Occupy My Inbox movement. EdB On Sun, Feb 9, 2014 at 8:16 AM, Erik de Bruin wrote: > I'm ba

Re: [FlexJS] BarChart component added - render issue

2014-02-08 Thread Erik de Bruin
I'm back home, I'll try to reproduce and if needed fix FalconJX this week. EdB On Friday, February 7, 2014, Alex Harui wrote: > > > On 2/7/14 12:02 AM, "OmPrakash Muppirala" > > wrote: > >But, when I compile the example app, the file: > >flex-asjs\examples\BarChartExample\bin\js-debug\org\apa

Re: [FlexJS] BarChart component added - render issue

2014-02-07 Thread OmPrakash Muppirala
On Feb 7, 2014 7:59 AM, "Alex Harui" wrote: > > Are you guys only getting this warning when using FB or does the warning > also show up when building from ant? I am getting same error with the ant bold as well. Thanks, Om > > On 2/7/14 7:01 AM, "Peter Ent" wrote: > > >I got all the code and re

Re: [FlexJS] BarChart component added - render issue

2014-02-07 Thread Alex Harui
On 2/7/14 12:02 AM, "OmPrakash Muppirala" wrote: >But, when I compile the example app, the file: >flex-asjs\examples\BarChartExample\bin\js-debug\org\apache\flex\charts\bea >ds > >seems to have been modified with a whole bunch requires removed. There >are >only these three: > >goog.provide('org

Re: [FlexJS] BarChart component added - render issue

2014-02-07 Thread Peter Ent
Assuming that I've built and installed things into the right places, I did not get this warning building the example using ant in the BarChartExample directory. So just using FB. I'm not yet familiar enough the configuration differences to see why this would be the case. --peter On 2/7/14 10:59 A

Re: [FlexJS] BarChart component added - render issue

2014-02-07 Thread Alex Harui
Are you guys only getting this warning when using FB or does the warning also show up when building from ant? On 2/7/14 7:01 AM, "Peter Ent" wrote: >I got all the code and rebuilt. The issue Om was trying to fix is >definitely fixed - the charts look great in both AS and JS. > >I am however, get

Re: [FlexJS] BarChart component added - render issue

2014-02-07 Thread Peter Ent
I got all the code and rebuilt. The issue Om was trying to fix is definitely fixed - the charts look great in both AS and JS. I am however, getting this warning when cross-compiling the BarChartExample from within Flash Builder: /Users/pent/Documents/Apache Flex/BarChartExample/bin/js-debug/org/a

Re: [FlexJS] BarChart component added - render issue

2014-02-07 Thread OmPrakash Muppirala
On Thu, Feb 6, 2014 at 11:09 PM, Alex Harui wrote: > > > On 2/6/14 10:49 PM, "OmPrakash Muppirala" wrote: > > >On Feb 6, 2014 10:42 PM, "Alex Harui" wrote: > >> > >> Ack. Read the error incorrectly. It may not be BarChartLayout that is > >> incorrect. Do you know which .js file is actually c

Re: [FlexJS] BarChart component added - render issue

2014-02-06 Thread Alex Harui
On 2/6/14 10:49 PM, "OmPrakash Muppirala" wrote: >On Feb 6, 2014 10:42 PM, "Alex Harui" wrote: >> >> Ack. Read the error incorrectly. It may not be BarChartLayout that is >> incorrect. Do you know which .js file is actually complaining? >> > >Yes, it is from XAxisBead.js Hmm. Are you all s

Re: [FlexJS] BarChart component added - render issue

2014-02-06 Thread OmPrakash Muppirala
On Feb 6, 2014 10:42 PM, "Alex Harui" wrote: > > Ack. Read the error incorrectly. It may not be BarChartLayout that is > incorrect. Do you know which .js file is actually complaining? > Yes, it is from XAxisBead.js > > On 2/6/14 10:29 PM, "OmPrakash Muppirala" wrote: > > >On Feb 6, 2014 10:0

Re: [FlexJS] BarChart component added - render issue

2014-02-06 Thread Alex Harui
Ack. Read the error incorrectly. It may not be BarChartLayout that is incorrect. Do you know which .js file is actually complaining? On 2/6/14 10:29 PM, "OmPrakash Muppirala" wrote: >On Feb 6, 2014 10:08 PM, "Alex Harui" wrote: >> >> >> >> On 2/6/14 9:38 PM, "OmPrakash Muppirala" wrote: >>

Re: [FlexJS] BarChart component added - render issue

2014-02-06 Thread OmPrakash Muppirala
On Feb 6, 2014 10:08 PM, "Alex Harui" wrote: > > > > On 2/6/14 9:38 PM, "OmPrakash Muppirala" wrote: > > >On Feb 6, 2014 8:24 PM, "Alex Harui" wrote: > >> > >> > >> > >> On 2/6/14 6:04 PM, "OmPrakash Muppirala" wrote: > >> > >> >App works fine even though I am getting a warning during the JS >

Re: [FlexJS] BarChart component added - render issue

2014-02-06 Thread Alex Harui
On 2/6/14 9:38 PM, "OmPrakash Muppirala" wrote: >On Feb 6, 2014 8:24 PM, "Alex Harui" wrote: >> >> >> >> On 2/6/14 6:04 PM, "OmPrakash Muppirala" wrote: >> >> >App works fine even though I am getting a warning during the JS >> >compilation >> >part: >> > >> >WARNING - Bad type annotation. Un

Re: [FlexJS] BarChart component added - render issue

2014-02-06 Thread OmPrakash Muppirala
On Feb 6, 2014 8:24 PM, "Alex Harui" wrote: > > > > On 2/6/14 6:04 PM, "OmPrakash Muppirala" wrote: > > >App works fine even though I am getting a warning during the JS > >compilation > >part: > > > >WARNING - Bad type annotation. Unknown type > >org.apache.flex.charts.beads.layouts.BarChartLayo

Re: [FlexJS] BarChart component added - render issue

2014-02-06 Thread Alex Harui
On 2/6/14 6:04 PM, "OmPrakash Muppirala" wrote: >App works fine even though I am getting a warning during the JS >compilation >part: > >WARNING - Bad type annotation. Unknown type >org.apache.flex.charts.beads.layouts.BarChartLayout > [java] var /** @type >{org.apache.flex.charts.beads.l

Re: [FlexJS] BarChart component added - render issue

2014-02-06 Thread OmPrakash Muppirala
On Mon, Feb 3, 2014 at 6:37 AM, Peter Ent wrote: > Hi Om, > > Sorry for the delayed response. BarChartLayout.changeHandler is the place. > You might need to look at the BarChartView bead as well. > > Thanks so much! > Peter > > Just checked in a fix. Please take a look. App works fine even thou

Re: [FlexJS] BarChart component added - render issue

2014-02-03 Thread Peter Ent
Hi Om, Sorry for the delayed response. BarChartLayout.changeHandler is the place. You might need to look at the BarChartView bead as well. Thanks so much! Peter On 1/31/14 7:17 PM, "OmPrakash Muppirala" wrote: >On Fri, Jan 31, 2014 at 11:31 AM, Peter Ent wrote: > >> Had a chat with Alex. For

Re: [FlexJS] BarChart component added - render issue

2014-01-31 Thread OmPrakash Muppirala
On Fri, Jan 31, 2014 at 11:31 AM, Peter Ent wrote: > Had a chat with Alex. For this particular issue, we were thinking that > Flex's scrollPolicy should be carried over to FlexJS, but enhanced: > > scrollPolicy: on | off | auto | hidden > > which map as follows - > > on: overflow:scroll, but

Re: [FlexJS] BarChart component added - render issue

2014-01-31 Thread Peter Ent
Had a chat with Alex. For this particular issue, we were thinking that Flex's scrollPolicy should be carried over to FlexJS, but enhanced: scrollPolicy: on | off | auto | hidden which map as follows - on: overflow:scroll, but scrollbars visible always off:overflow:visible auto: overflo

Re: [FlexJS] BarChart component added - render issue

2014-01-31 Thread Alex Harui
On 1/31/14 7:03 AM, "Peter Ent" wrote: >Hi, > >First, I can confirm that Om's suggestion of *adding* the createElement >override to the cross-compiled/generated BarChart.js does fix the problem. > >Secondly - how do we make it possible to allow customization of the >JavaScript that gets cross-c

Re: [FlexJS] BarChart component added - render issue

2014-01-31 Thread Alex Harui
Ah, sorry. Somehow I thought you were only adding one line to an existing createElement method. So, in FlexJS, if you have to modify the cross-compiled JS for a class in the FlexJSJX project, that will usually mean that that class can't be in FlexJSJX and has to be moved back to FlexJSUI and have

Re: [FlexJS] BarChart component added - render issue

2014-01-31 Thread Peter Ent
Hi, First, I can confirm that Om's suggestion of *adding* the createElement override to the cross-compiled/generated BarChart.js does fix the problem. Secondly - how do we make it possible to allow customization of the JavaScript that gets cross-compiled? This is bound to be necessary and clearly

Re: [FlexJS] BarChart component added - render issue

2014-01-31 Thread OmPrakash Muppirala
On Jan 30, 2014 11:32 PM, "Alex Harui" wrote: > > > > On 1/30/14 11:07 PM, "OmPrakash Muppirala" wrote: > > >On Jan 30, 2014 10:18 PM, "Alex Harui" wrote: > >> > >> > >> > >> On 1/30/14 9:11 PM, "OmPrakash Muppirala" wrote: > >> > >> >On Thu, Jan 30, 2014 at 8:30 PM, Alex Harui wrote: > >> > >

Re: [FlexJS] BarChart component added - render issue

2014-01-30 Thread Alex Harui
On 1/30/14 11:07 PM, "OmPrakash Muppirala" wrote: >On Jan 30, 2014 10:18 PM, "Alex Harui" wrote: >> >> >> >> On 1/30/14 9:11 PM, "OmPrakash Muppirala" wrote: >> >> >On Thu, Jan 30, 2014 at 8:30 PM, Alex Harui wrote: >> > >> >> Om, something is still not configured right. BarChart.js should

Re: [FlexJS] BarChart component added - render issue

2014-01-30 Thread OmPrakash Muppirala
On Jan 30, 2014 10:18 PM, "Alex Harui" wrote: > > > > On 1/30/14 9:11 PM, "OmPrakash Muppirala" wrote: > > >On Thu, Jan 30, 2014 at 8:30 PM, Alex Harui wrote: > > > >> Om, something is still not configured right. BarChart.js should be > >>coming > >> from frameworks/js/FlexJS/libs. > > > > > >I

Re: [FlexJS] BarChart component added - render issue

2014-01-30 Thread Alex Harui
On 1/30/14 9:11 PM, "OmPrakash Muppirala" wrote: >On Thu, Jan 30, 2014 at 8:30 PM, Alex Harui wrote: > >> Om, something is still not configured right. BarChart.js should be >>coming >> from frameworks/js/FlexJS/libs. > > >I dont see the libs folder here: >https://git-wip-us.apache.org/repos/a

Re: [FlexJS] BarChart component added - render issue

2014-01-30 Thread OmPrakash Muppirala
On Thu, Jan 30, 2014 at 8:30 PM, Alex Harui wrote: > Om, something is still not configured right. BarChart.js should be coming > from frameworks/js/FlexJS/libs. I dont see the libs folder here: https://git-wip-us.apache.org/repos/asf?p=flex-asjs.git;a=tree;f=frameworks/js/FlexJS;h=868b43d18106

Re: [FlexJS] BarChart component added - render issue

2014-01-30 Thread Alex Harui
Om, something is still not configured right. BarChart.js should be coming from frameworks/js/FlexJS/libs. There is no createElement call in my copy. -Alex On 1/30/14 8:18 PM, "OmPrakash Muppirala" wrote: >On Jan 30, 2014 5:36 PM, "OmPrakash Muppirala" >wrote: >> >> On Thu, Jan 30, 2014 at 1:

Re: [FlexJS] BarChart component added - render issue

2014-01-30 Thread OmPrakash Muppirala
On Jan 30, 2014 5:36 PM, "OmPrakash Muppirala" wrote: > > On Thu, Jan 30, 2014 at 1:43 PM, Peter Ent wrote: >> >> I've pushed changes that should have the BarChart example working for both >> AS and JS. There is an issue with the axis appearing below the chart that >> causes the chart area (the d

Re: [FlexJS] BarChart component added - render issue

2014-01-30 Thread OmPrakash Muppirala
On Thu, Jan 30, 2014 at 1:43 PM, Peter Ent wrote: > I've pushed changes that should have the BarChart example working for both > AS and JS. There is an issue with the axis appearing below the chart that > causes the chart area (the dataGroup) to be larger than the chart space. > This is an issue

Re: [FlexJS] BarChart component added - render issue

2014-01-30 Thread Peter Ent
I've pushed changes that should have the BarChart example working for both AS and JS. There is an issue with the axis appearing below the chart that causes the chart area (the dataGroup) to be larger than the chart space. This is an issue with getting the HTML/CSS right and any help/advice or chang

Re: [FlexJS] BarChart component added - render issue

2014-01-30 Thread Peter Ent
I have the BarChart example working in both AS and JS. The AS worked right away but I discovered an issue with sizing the NonVirtualDataGroup in JavaScript. Or rather, the lack of sizing it. I am tracking that down now. There are places in FlexJS where I don't have the size calculation quite right

Re: [FlexJS] BarChart component added - render issue

2014-01-29 Thread Alex Harui
OK, things appear to be compiling and running again on the AS side. Make sure the BarChartExample works and you think it should. Thanks -Alex On 1/29/14 2:12 PM, "Alex Harui" wrote: >It is stale because it was using the custom bar chart and not the one I >moved to FLexJSJX. I have it updated

Re: [FlexJS] BarChart component added - render issue

2014-01-29 Thread Alex Harui
It is stale because it was using the custom bar chart and not the one I moved to FLexJSJX. I have it updated locally but there is a problem now because we're trying to use the Ifactory code gen in the compiler and it isn't working right. -Alex On 1/29/14 1:49 PM, "Peter Ent" wrote: >I wonder w

Re: [FlexJS] BarChart component added - render issue

2014-01-29 Thread Peter Ent
The ActionScript side works for me but JavaScript produces what Om is seeing, so I will look into that. --peter On 1/29/14 4:12 PM, "OmPrakash Muppirala" wrote: >On Wed, Jan 29, 2014 at 1:07 PM, Alex Harui wrote: > >> I just discovered that the BarChartExample is stale. It isn't using the >>

Re: [FlexJS] BarChart component added - render issue

2014-01-29 Thread Peter Ent
I wonder why it is stale? I'll take a look as well since it will help me remember and help with any JS issues. -peter On 1/29/14 4:07 PM, "Alex Harui" wrote: >I just discovered that the BarChartExample is stale. It isn't using the >BarChart now checked into FlexJSJX. I'm going to try to get i

Re: [FlexJS] BarChart component added - render issue

2014-01-29 Thread OmPrakash Muppirala
On Wed, Jan 29, 2014 at 1:07 PM, Alex Harui wrote: > I just discovered that the BarChartExample is stale. It isn't using the > BarChart now checked into FlexJSJX. I'm going to try to get it to compile > and run in AS and then maybe Peter will have time to look at the JS side. > > I did wonder h

Re: [FlexJS] BarChart component added - render issue

2014-01-29 Thread Alex Harui
I just discovered that the BarChartExample is stale. It isn't using the BarChart now checked into FlexJSJX. I'm going to try to get it to compile and run in AS and then maybe Peter will have time to look at the JS side. -Alex On 1/29/14 11:59 AM, "Peter Ent" wrote: >Please feel free to make a

Re: [FlexJS] BarChart component added - render issue

2014-01-29 Thread Peter Ent
Please feel free to make adjustments. I have to admit that the particulars of CSS escape me sometimes. Thanks for your help and checking this out. Peter On 1/29/14 2:40 PM, "OmPrakash Muppirala" wrote: >When I compile and run the BarChartExample, in the JS version, this is >what >I am seeing (

Re: [FlexJS] BarChart component added - render issue

2014-01-29 Thread OmPrakash Muppirala
On Wed, Jan 29, 2014 at 11:40 AM, OmPrakash Muppirala wrote: > When I compile and run the BarChartExample, in the JS version, this is > what I am seeing (swf works fine) > > http://jsfiddle.net/cvLqE/ > > To be clear, in the above version, I dont see any bars in either browsers. The one below does

Re: [FlexJS] BarChart component added - render issue

2014-01-29 Thread OmPrakash Muppirala
When I compile and run the BarChartExample, in the JS version, this is what I am seeing (swf works fine) http://jsfiddle.net/cvLqE/ Once I tweak the output html like this http://jsfiddle.net/52uaL/2/ i.e. remove the position: absolute style attribute of 'barChart' div, the bars show up, but are