Re: [Discuss] Start release process 0.9.3

2018-09-03 Thread Harbs
I have item1 > item2 > item3. Item1 has a VerticalLayout This sets item2 to block Item2 has OneFlexibleChildHorizontalLayout which sets itself to flex and sets flex-grow and flex-shrink on item3 VerticalLayout and OneFlexibleChildHorizontalLayout are stepping on each other when setting the

Build failed in Jenkins: royale-asjs #1275

2018-09-03 Thread apacheroyaleci
See -- [...truncated 2.18 MB...] [mxmlc] Dependencies calculated for 'org.apache.royale.core.CSSFontFaceBead' [mxmlc]

Build failed in Jenkins: royale-asjs #1274

2018-09-03 Thread apacheroyaleci
See -- [...truncated 2.18 MB...] [mxmlc] Dependencies calculated for 'org.apache.royale.core.CSSFontFaceBead' [mxmlc]

Re: [Discuss] Start release process 0.9.3

2018-09-03 Thread Alex Harui
Hmm. That change doesn't make sense to me. AIUI, some component has flex layout and a child has block. The child with a vertical layout should be setting block on its children, not itself. Of course, I could be wrong... -Alex On 9/3/18, 3:52 AM, "Harbs" wrote: Yup. That seems to

Re: [royale-asjs] branch feature/new_merge updated: Fixed asconfigc on JewelExample

2018-09-03 Thread Carlos Rovira
Hi, I tried to fix Jewel library asconfig to work with build maven distribution to get code intelligence. I get JewelExample working ok, but instead Jewel has a problem with SWF target. I opened an issue with Josh. This was the reason I modified the tokens in some distribution xml I suppose you

Re: Make Jewel dependent on Basic (Re: [Discuss] Start release process 0.9.3)

2018-09-03 Thread Carlos Rovira
Hi as Piotr said JewelExample is the example to check. Fixing DataGroup should fix the four sections involved: -TablePlayGround -DateComponentsPlayGround -ComboBoxPlayGround -ListPlayGround just as a hint, I move this lines in MainContent.mxml to the bottom of ApplicationMainContent, to let the

Re: Make Jewel dependent on Basic (Re: [Discuss] Start release process 0.9.3)

2018-09-03 Thread Harbs
I see the problem. I’m out of time for tonight. I’ll work on fixing this in the morning. Thanks, Harbs > On Sep 3, 2018, at 3:46 PM, Carlos Rovira wrote: > > The problem here's the changes in DataGroup, we can skip this problem if we > want for now making Jewel DataGroup be equal to old

Re: [royale-asjs] branch feature/new_merge updated: Fixed asconfigc on JewelExample

2018-09-03 Thread Piotr Zarzycki
Wonderful! <3 :) Thank you! pon., 3 wrz 2018 o 20:23 napisał(a): > This is an automated email from the ASF dual-hosted git repository. > > harbs pushed a commit to branch feature/new_merge > in repository https://gitbox.apache.org/repos/asf/royale-asjs.git > > > The following commit(s) were

Re: Make Jewel dependent on Basic (Re: [Discuss] Start release process 0.9.3)

2018-09-03 Thread Piotr Zarzycki
I'm glad that you resolved it! :) pon., 3 wrz 2018 o 20:21 Harbs napisał(a): > Huh. I had an outdated version of asconfigc installed… > > > On Sep 3, 2018, at 9:16 PM, Harbs wrote: > > > > I saw that, but for some reason it did not work for me… > > > >> On Sep 3, 2018, at 9:11 PM, Piotr

Re: Make Jewel dependent on Basic (Re: [Discuss] Start release process 0.9.3)

2018-09-03 Thread Piotr Zarzycki
That token is working for Moonshine as well, so not sure what is wrong on your end. :( pon., 3 wrz 2018 o 20:16 Harbs napisał(a): > I saw that, but for some reason it did not work for me… > > > On Sep 3, 2018, at 9:11 PM, Piotr Zarzycki > wrote: > > > > It's recognized. Today I have fixed

Re: Make Jewel dependent on Basic (Re: [Discuss] Start release process 0.9.3)

2018-09-03 Thread Harbs
Huh. I had an outdated version of asconfigc installed… > On Sep 3, 2018, at 9:16 PM, Harbs wrote: > > I saw that, but for some reason it did not work for me… > >> On Sep 3, 2018, at 9:11 PM, Piotr Zarzycki wrote: >> >> It's recognized. Today I have fixed Jewel Alert example by that commit

Re: Make Jewel dependent on Basic (Re: [Discuss] Start release process 0.9.3)

2018-09-03 Thread Harbs
I saw that, but for some reason it did not work for me… > On Sep 3, 2018, at 9:11 PM, Piotr Zarzycki wrote: > > It's recognized. Today I have fixed Jewel Alert example by that commit [1] > > [1] > https://github.com/apache/royale-asjs/commit/37d24b86a674c61be9688f09492978c74a243f3b > >

Re: Make Jewel dependent on Basic (Re: [Discuss] Start release process 0.9.3)

2018-09-03 Thread Piotr Zarzycki
It's recognized. Today I have fixed Jewel Alert example by that commit [1] [1] https://github.com/apache/royale-asjs/commit/37d24b86a674c61be9688f09492978c74a243f3b Thanks, Piotr pon., 3 wrz 2018 o 20:09 Harbs napisał(a): > I don’t think theme is recognized by asconfigc and I don’t think

Re: Make Jewel dependent on Basic (Re: [Discuss] Start release process 0.9.3)

2018-09-03 Thread Harbs
I don’t think theme is recognized by asconfigc and I don’t think there’s a way to use relative paths or tokens as a compiler option. Am I missing something? > On Sep 3, 2018, at 8:59 PM, Piotr Zarzycki wrote: > > Harbs, > > I believe JewelExample contains TablePlayGround. If you will be

Re: Make Jewel dependent on Basic (Re: [Discuss] Start release process 0.9.3)

2018-09-03 Thread Piotr Zarzycki
Harbs, I believe JewelExample contains TablePlayGround. If you will be playing with that. Can you fix VSCode build as well ? Enough if you add missing parts in asconfig.json. Check what jewel theme is being used in pom.xml. Thanks, Piotr pon., 3 wrz 2018 o 19:54 Harbs napisał(a): > Hi Carlos,

Re: Make Jewel dependent on Basic (Re: [Discuss] Start release process 0.9.3)

2018-09-03 Thread Harbs
Hi Carlos, Thanks for checking. Which Jewel examples should I use to fix this? > On Sep 3, 2018, at 3:46 PM, Carlos Rovira wrote: > > Hi, congratulations! jewel example is running almost at 70% aprox > > Things to fix: > > * Table, this will fix DateChooser and this DateField (DateChooser

Re: Make Jewel dependent on Basic (Re: [Discuss] Start release process 0.9.3)

2018-09-03 Thread Carlos Rovira
Hi, congratulations! jewel example is running almost at 70% aprox Things to fix: * Table, this will fix DateChooser and this DateField (DateChooser uses a Table internally) The problem here's the changes in DataGroup, we can skip this problem if we want for now making Jewel DataGroup be equal to

Build failed in Jenkins: royale-asjs #1273

2018-09-03 Thread apacheroyaleci
See -- [...truncated 2.19 MB...] [mxmlc] Dependencies calculated for 'org.apache.royale.core.CSSFontFaceBead' [mxmlc]

Re: Make Jewel dependent on Basic (Re: [Discuss] Start release process 0.9.3)

2018-09-03 Thread Carlos Rovira
build with maven is ok for me :) I try examples now El lun., 3 sept. 2018 a las 14:21, Harbs () escribió: > Oops. I broke that. > > Should be fixed now… > > > On Sep 3, 2018, at 3:09 PM, Carlos Rovira > wrote: > > > > Hi build with maven fails in MXRoyale: > > > > COMPC > > > > Loading

Re: Make Jewel dependent on Basic (Re: [Discuss] Start release process 0.9.3)

2018-09-03 Thread Harbs
Oops. I broke that. Should be fixed now… > On Sep 3, 2018, at 3:09 PM, Carlos Rovira wrote: > > Hi build with maven fails in MXRoyale: > > COMPC > > Loading configuration: > /Users/carlosrovira/Dev/Royale/Source/royale-asjs/frameworks/projects/MXRoyale/target/compile-swf-config.xml > > >

Build failed in Jenkins: royale-asjs #1272

2018-09-03 Thread apacheroyaleci
See Changes: [piotrzarzycki21] Make sure that Jewel Alert blog example is working fine with VSCode -- [...truncated 2.18 MB...] [mxmlc]

Re: Make Jewel dependent on Basic (Re: [Discuss] Start release process 0.9.3)

2018-09-03 Thread Carlos Rovira
Hi build with maven fails in MXRoyale: COMPC Loading configuration: /Users/carlosrovira/Dev/Royale/Source/royale-asjs/frameworks/projects/MXRoyale/target/compile-swf-config.xml Warning: mx.controls.RichTextEditor is defined by multiple files:

Re: Make Jewel dependent on Basic (Re: [Discuss] Start release process 0.9.3)

2018-09-03 Thread Piotr Zarzycki
Maven build should work - I checked it one hour ago. I will try to get from your plate Tour De Flex. On Mon, Sep 3, 2018, 1:26 PM Harbs wrote: > > > On Sep 3, 2018, at 2:22 PM, Carlos Rovira > wrote: > > > > Hi Harbs > > > > just check what I can do: > > The branch you're working is

Re: Make Jewel dependent on Basic (Re: [Discuss] Start release process 0.9.3)

2018-09-03 Thread Harbs
> On Sep 3, 2018, at 2:22 PM, Carlos Rovira wrote: > > Hi Harbs > > just check what I can do: > The branch you're working is "feature/new_merge" right? Yes. > this builds with maven? Theoretically, but I have not tried. Please let me know. > Can I check it out, build and test Jewel now?

Re: Make Jewel dependent on Basic (Re: [Discuss] Start release process 0.9.3)

2018-09-03 Thread Carlos Rovira
Hi Harbs just check what I can do: The branch you're working is "feature/new_merge" right? this builds with maven? Can I check it out, build and test Jewel now? So I can report what I see? thanks El lun., 3 sept. 2018 a las 13:17, Harbs () escribió: > I’m done for the morning and I’ll be out

Re: Make Jewel dependent on Basic (Re: [Discuss] Start release process 0.9.3)

2018-09-03 Thread Harbs
I’m done for the morning and I’ll be out this afternoon. I’ll try to spend some more time this evening on this. I still need to: * Test the framework against some more of my apps. * Make sure the examples work. * See if the Jewel examples run. * See if Tour de Flex runs. Anyone who can confirm

setting left, top styles vía Javascript has no effect on IE11

2018-09-03 Thread Carlos Rovira
Hi, we have currently 2 bugs in IE11 due to the same problem. 1.- Validor ErrorTip recently done by Yutaro 2.- DateField and ComboBox in Jewel both popups some kind of component that needs to position in a particular x, y position. This is done using Point class and PointUtils with

Re: [Discuss] Start release process 0.9.3

2018-09-03 Thread Harbs
Yup. That seems to work. > On Sep 3, 2018, at 1:45 PM, Harbs wrote: > > It is related to the merge because it works on the revert-refactor branch. > > I’m guessing the order of layouts changed somehow, but I don’t know where > that change is. > > I’m working on a change to VerticalLayout

Re: [Discuss] Start release process 0.9.3

2018-09-03 Thread Harbs
It is related to the merge because it works on the revert-refactor branch. I’m guessing the order of layouts changed somehow, but I don’t know where that change is. I’m working on a change to VerticalLayout which should solve this... > On Sep 3, 2018, at 1:26 PM, Carlos Rovira wrote: > > Ok,

Re: [Discuss] Start release process 0.9.3

2018-09-03 Thread Carlos Rovira
Ok, that layout is from Basic and I think there's no changes from MXRoyale in that part (Jewel for sure doesn't change Basic Layouts since I made Jewel ones based on CSS) So I think whatever problem in Basic Layouts could be a bug or something that seems to work and some change in your app made

Re: [Discuss] Start release process 0.9.3

2018-09-03 Thread Harbs
I found where it’s going off. I have a component with a layout of OneFlexibleChildHorizontalLayout. It’s inside another component with a standard VerticalLayout. OneFlexibleChildHorizontalLayout.layout sets the position to flex and then the VerticalLayout sets it to block. For some reason

Re: [Discuss] Start release process 0.9.3

2018-09-03 Thread Carlos Rovira
Hi Harbs, let me know if you want that I take a look, checking the branch and building it thanks El lun., 3 sept. 2018 a las 12:07, Harbs () escribió: > Foo. Something is setting Container position to “block” when the layout > should be setting it to “flex”… :-( > > > On Sep 3, 2018, at 1:01

Re: [Discuss] Start release process 0.9.3

2018-09-03 Thread Harbs
Foo. Something is setting Container position to “block” when the layout should be setting it to “flex”… :-( > On Sep 3, 2018, at 1:01 PM, Piotr Zarzycki wrote: > > Good news Harbs! Thank you! > > pon., 3 wrz 2018 o 11:58 Harbs > napisał(a): > >> I just got the

Re: [Discuss] Start release process 0.9.3

2018-09-03 Thread Piotr Zarzycki
Good news Harbs! Thank you! pon., 3 wrz 2018 o 11:58 Harbs napisał(a): > I just got the merge branch to work with my app. There’s some visual > differences still. Need to figure that out… > > > On Sep 3, 2018, at 5:53 AM, Alex Harui wrote: > > > > FWIW, I agree with Harbs. Enough time has

Re: [Discuss] Start release process 0.9.3

2018-09-03 Thread Harbs
I just got the merge branch to work with my app. There’s some visual differences still. Need to figure that out… > On Sep 3, 2018, at 5:53 AM, Alex Harui wrote: > > FWIW, I agree with Harbs. Enough time has passed and changes have been made > that it is time to try making Jewel dependent on

Re: Make Jewel dependent on Basic (Re: [Discuss] Start release process 0.9.3)

2018-09-03 Thread Carlos Rovira
Hi Piotr, thanks for asking. Yes, if we want to try link Jewel is ok to link to others like HTML. Carlos El lun., 3 sept. 2018 a las 10:56, Piotr Zarzycki (< piotrzarzyck...@gmail.com>) escribió: > I'm doing Maven build and HTML was failing. It is because it need to depend > again from Basic.

Re: Make Jewel dependent on Basic (Re: [Discuss] Start release process 0.9.3)

2018-09-03 Thread Piotr Zarzycki
I'm doing Maven build and HTML was failing. It is because it need to depend again from Basic. I can make the changes. Carlos are you on that right now ? I don't want to collide with you. Thanks, Piotr pon., 3 wrz 2018 o 10:28 Carlos Rovira napisał(a): > Hi Harbs, > > I'm available for you to

Build failed in Jenkins: royale-asjs_MXRoyale_MXTests #42

2018-09-03 Thread apacheroyaleci
See Changes: [noreply] Update FileReference.as [noreply] Update UIComponent.as [noreply] Update Event.as -- [...truncated 1.86

Re: Make Jewel dependent on Basic (Re: [Discuss] Start release process 0.9.3)

2018-09-03 Thread Carlos Rovira
Hi Harbs, I'm available for you to help you solve. write me when you need. Just a note: for doing this, I'll take the following path: I'll try to compile and run without make Jewel depend on Basic, since is the actual shape. From there I'll try to link Basic. I think that path will provide you a

Re: Make Jewel dependent on Basic (Re: [Discuss] Start release process 0.9.3)

2018-09-03 Thread Harbs
OK. Everything compiles now. I’m going to run the branch against some applications and see what happens. If you could do the same, it would be helpful. Carlos, I probably need some direction on how to test Jewel. Harbs > On Sep 3, 2018, at 10:38 AM, Piotr Zarzycki wrote: > > Ok! Again good

Re: Make Jewel dependent on Basic (Re: [Discuss] Start release process 0.9.3)

2018-09-03 Thread Piotr Zarzycki
Ok! Again good luck! :) pon., 3 wrz 2018 o 09:34 Harbs napisał(a): > The issues that I see with Jewel are related to the changes that were lost > when you merged. > > I’m hoping I can resolve this quickly. Let’s see how over-optimistic I’m > being… ;-) > > Harbs > > > On Sep 3, 2018, at 10:28

Re: royale-extras and the royale-typedefs Maven build

2018-09-03 Thread Harbs
I just committed the changes. Please check that the patches are correct. > On Sep 3, 2018, at 10:20 AM, Harbs wrote: > > Sure. > >> On Sep 3, 2018, at 5:31 AM, Alex Harui wrote: >> >> Hi Harbs, >> >> Can you take the patched google_maps_api_v3_11.js from a local >> royale-typedefs build

Re: Make Jewel dependent on Basic (Re: [Discuss] Start release process 0.9.3)

2018-09-03 Thread Harbs
The issues that I see with Jewel are related to the changes that were lost when you merged. I’m hoping I can resolve this quickly. Let’s see how over-optimistic I’m being… ;-) Harbs > On Sep 3, 2018, at 10:28 AM, Piotr Zarzycki wrote: > > Harbs, > > There is also another way. Maybe it is

Re: Make Jewel dependent on Basic (Re: [Discuss] Start release process 0.9.3)

2018-09-03 Thread Piotr Zarzycki
Harbs, There is also another way. Maybe it is worth that I will spend couple of hours on my branch and try to understand what was wrong. If I would manage to do that - maybe you will be able to add your stuff on top of that ? Do you see value in that approach ? I'm just afraid that doing again

Re: Make Jewel dependent on Basic (Re: [Discuss] Start release process 0.9.3)

2018-09-03 Thread Piotr Zarzycki
Hi Harbs, Looking forward to your changes. I'm just interested in release in whatever state it is and dealing with discussion later on. Btw. I have invested also at least 6h with merge stuff and as you can see it end up with bigger things. Good Luck! :) Thanks, Piotr pon., 3 wrz 2018 o 09:18

Re: royale-extras and the royale-typedefs Maven build

2018-09-03 Thread Harbs
Sure. > On Sep 3, 2018, at 5:31 AM, Alex Harui wrote: > > Hi Harbs, > > Can you take the patched google_maps_api_v3_11.js from a local > royale-typedefs build or maybe from apacheroyaleci and commit it to > royale-extras? I don't want to deal with the technicalities of whether I can >

Re: Make Jewel dependent on Basic (Re: [Discuss] Start release process 0.9.3)

2018-09-03 Thread Harbs
Right now, my priority is getting all three active branches combined with all code working. That means MX/Spark, Jewel, and all the additions on the revert branch all coexisting nicely in the merge branch. I’m spending the better part of today on that. I’d like to hold off on discussing where

Re: Increased size of the framework

2018-09-03 Thread Piotr Zarzycki
Yishay, I just check last successful build and it's gone. Probably it wasn't generated. We are good here, unless you would like to investigate what has happened. However I do not recommend spend on that time. Thanks, Piotr pon., 3 wrz 2018 o 08:40 Yishay Weiss napisał(a): > Piotr, I’m trying

RE: Increased size of the framework

2018-09-03 Thread Yishay Weiss
Piotr, I’m trying to understand the problem. Do you see this in successful builds? From: Piotr Zarzycki Sent: Sunday, September 2, 2018 10:27:11 PM To: dev@royale.apache.org Subject: Re: Increased size of the framework How are you going to remove it ? It is

Make Jewel dependent on Basic (Re: [Discuss] Start release process 0.9.3)

2018-09-03 Thread Carlos Rovira
Hi, considering making Jewel dependent on Basic. I'm trying to put my mind in that place. Since this is a huge effort for me, hope you all try to do the same as me and considering some thoughts, so we can plan something that works for all: Since Basic will be the middle point between Core and