Re: Experiment with UIBase change to classList

2018-03-13 Thread Alex Harui
Hi Carlos, I took a quick look. 1) I don't see how if the user did: myComp.className = "Carlos"; myComp.className = ""; That 'Carlos' would be removed from the classList. 2) I still don't like that we are calling split and apply on classList even for simple assignment of a single className

Re: Experiment with UIBase change to classList

2018-03-13 Thread Carlos Rovira
Hi Alex, ok, I saw the problems of using a list as I was creating it for that reason I try to do this as easy. to solve that we can introduce a few lines more that handles strings. This will be less code that current solution and handle both possibilities is done in more parts of the framework

[VOTE] Release Apache Royale 0.9.2 RC2

2018-03-13 Thread Alex Harui
Hi, This is vote for the 0.9.2 release of Apache Royale. The release candidate can be found here; https://dist.apache.org/repos/dist/dev/royale/0.9.2/rc2/ Before voting please review the section,'What are the ASF requirements on approving a release?', at:

Re: Experiment with UIBase change to classList

2018-03-13 Thread Piotr Zarzycki
Carlos, The thing is that we are loosing our time here. We were discussing solutions for that in the separate thread - it took as more than 60 emails. It's starting again. You have something which is working, it's require to add your custom logic in the component. Building a component end up with

[DISCUSS] Discuss Release Apache Royale 0.9.2 RC2

2018-03-13 Thread Alex Harui
> >This is the discussion thread. RC1 never happened due to a bad >RELEASE_NOTES file. > >Changes in this RC include: >- Updating the top-level RELEASE_NOTES from royale-asjs/releasemgr > >The MenuExample fails to build, so the Approval Script will not run >examples. > >Thanks, >Alex Harui >

Re: [DISCUSS] Discuss Release Apache Royale 0.9.2 RC2

2018-03-13 Thread Piotr Zarzycki
Hi Alex, I got first shot with ApprovalScript and got this [1]. I have ant rat jar in place, do you have any idea what am I missing ? [1] https://paste.apache.org/Ld1G Thanks, Piotr 2018-03-13 8:49 GMT+01:00 Alex Harui : > > > >This is the discussion thread. RC1

Re: Experiment with UIBase change to classList

2018-03-13 Thread Harbs
Hi Carlos, I definitely appreciate the work you are doing. I’m swamped with work right now, so I don’t have the time to spend helping you. (Sorry about that.) :-( I think the discussions here are about pretty minor points. You can certainly implement jewel how you think makes sense, but if you

Re: Experiment with UIBase change to classList

2018-03-13 Thread Piotr Zarzycki
In my example orders matters. Setup first className than your property. On Tue, Mar 13, 2018, 12:39 Harbs wrote: > Hi Carlos, > > I definitely appreciate the work you are doing. I’m swamped with work > right now, so I don’t have the time to spend helping you. (Sorry

Re: Experiment with UIBase change to classList

2018-03-13 Thread Harbs
FYI, here’s one article which discusses collections: https://dev.opera.com/articles/efficient-javascript/ > On Mar 13, 2018, at 1:36 PM, Harbs wrote: > > I don’t know why you keep asserting this. > > The general

Re: Experiment with UIBase change to classList

2018-03-13 Thread Piotr Zarzycki
I personally said - Go and try, report back. I have gave you an real world examples where classList failed. Try and post the results. 2018-03-13 11:49 GMT+01:00 Carlos Rovira : > Hi, > > it's very hard to me to invest lot of time both in tryin to develop > something

Re: [DISCUSS] Discuss Release Apache Royale 0.9.2 RC2

2018-03-13 Thread Piotr Zarzycki
Guys, In order to test Maven artifacts in actions I have prepared settings template. [1] 1) Place link to staged artifacts in url tags - Link is available in VOTE thread 2) In your application pom change version of your dependencies to "0.9.2" 3) Run mvn clean install -s settings-rc-vote.xml

Re: Experiment with UIBase change to classList

2018-03-13 Thread Carlos Rovira
Hi Piotr, just tested without do any other change to the code posted yesterday and is working (what surprised me) gives: *PRIMARY* what is completely right what does not work is that is what Alex said, and can be easily resolved with a if-else that handles multiple classes in a string I

Re: Experiment with UIBase change to classList

2018-03-13 Thread Harbs
I don’t know why you keep asserting this. The general rule with live collections in general is to avoid them when possible. Collections need to be resolved which adds overhead. The less interactions with a DOM, the better. Native JS is almost always going to more efficient than browser/DOM

Re: Experiment with UIBase change to classList

2018-03-13 Thread Harbs
> On Mar 13, 2018, at 1:48 PM, Carlos Rovira wrote: > > Cause I'm dealing with this since 2016 with MDL?, if this is not enough > experience and time with an API, don't know what should I do to reflect > that is the part of Royale where I always move and maybe have the

Re: Experiment with UIBase change to classList

2018-03-13 Thread Carlos Rovira
Hi Harbs 2018-03-13 12:39 GMT+01:00 Harbs : > Hi Carlos, > > I definitely appreciate the work you are doing. I’m swamped with work > right now, so I don’t have the time to spend helping you. (Sorry about > that.) :-( > > Thanks Harbs, but don't worry, I think at this time

Re: Experiment with UIBase change to classList

2018-03-13 Thread Carlos Rovira
Hi Piotr, thanks for your words, but is difficult to work on something when you believe in your vision and others no, and more over when all the facts you see corroborates that vision. It's difficult to maintain live the moto in that scenario. but anyway for you Kindly words Carlos 2018-03-13

Re: Experiment with UIBase change to classList

2018-03-13 Thread Carlos Rovira
Hi, it's very hard to me to invest lot of time both in tryin to develop something useful in the look and feel field for us where no other is doing work, trying to explain and discuss all issues I find without get any traction. It's like to face a wall all the time. Maybe I'm wrong with my

Re: Experiment with UIBase change to classList

2018-03-13 Thread Piotr Zarzycki
Carlos, In my opinion you are not facing the wall from US. You are facing the wall from lack of volounteers who can help, do the job. Believe me your Jewel effort in my list of tasks is almost on the Top. I have to fiinish planned work in TranspiledActionScript first and I hope to join. When it

Re: Experiment with UIBase change to classList

2018-03-13 Thread Carlos Rovira
So, you if is == you expect that setting className in royale you remove all inclusive typeNames? Harbs, className is not equal to class in HTML 2018-03-13 14:08 GMT+01:00 Harbs : > className in Royale == class in HTML. > > > On Mar 13, 2018, at 2:55 PM, Carlos Rovira

Re: Experiment with UIBase change to classList

2018-03-13 Thread Carlos Rovira
I think we're getting to the point in this discussion. For me as a user, I expect to use className property to "add", and not override all I have for that reason in MDL and now in Royale we decided to create properties (that use to be boolean) like "primary" or in MDL "fab" to add or remove those

Re: Experiment with UIBase change to classList

2018-03-13 Thread Carlos Rovira
Solving the multiple string value problem: This: *PRIMARY* with this change COMPILE::JS protected function setClassName(value:String):void { var classes:Array = value.split(" "); element.classList.add.apply(element.classList, classes); } I think this was all the problems we have right?

Re: Experiment with UIBase change to classList

2018-03-13 Thread Carlos Rovira
Hi Harbs 2018-03-13 13:34 GMT+01:00 Harbs : > > I don’t think you understood what Piotr was saying. > > The order of setting the *property* and *className* matters. He was not > talking about the order of class names in a class list. > > I think is what I did right? change

Re: [DISCUSS] Discuss Release Apache Royale 0.9.2 RC2

2018-03-13 Thread Harbs
The top-level release notes seem to be missing a lot of improvements. Here’s what I get in the top-level RELEASE_NOTES: https://paste.apache.org/WumV And here’s what’s in Royale-asjs RELEASE_NOTES.md: https://paste.apache.org/JEgq

Re: Experiment with UIBase change to classList

2018-03-13 Thread Carlos Rovira
Hi Piotr, that's one of the advantages of a collection, order doesn't matter! :) output: *PRIMARY* this is one of the reason to change, since you'll end trying to figure what comes in first or not. Do you need more evidence? Thanks 2018-03-13 12:48 GMT+01:00 Piotr Zarzycki

Re: Experiment with UIBase change to classList

2018-03-13 Thread Harbs
No. className is supposed to *replace* the entire classList minus the internally managed ones (i.e. typeNames). Your code drastically changes the current behavior. You cannot use add for that and replacing the classList will destroy your custom class names. > On Mar 13, 2018, at 2:34 PM,

Re: Experiment with UIBase change to classList

2018-03-13 Thread Harbs
className in Royale == class in HTML. > On Mar 13, 2018, at 2:55 PM, Carlos Rovira wrote: > > I think we're getting to the point in this discussion. > > For me as a user, I expect to use className property to "add", and not > override all I have > for that reason in

Re: Experiment with UIBase change to classList

2018-03-13 Thread Carlos Rovira
Hi Harbs 2018-03-13 12:54 GMT+01:00 Harbs : > > > On Mar 13, 2018, at 1:48 PM, Carlos Rovira > wrote: > > > > Cause I'm dealing with this since 2016 with MDL?, if this is not enough > > experience and time with an API, don't know what should I do

Re: [DISCUSS] Discuss Release Apache Royale 0.9.2 RC2

2018-03-13 Thread Alex Harui
On 3/13/18, 5:12 AM, "Harbs" wrote: >The top-level release notes seem to be missing a lot of improvements. > >What populates the main RELEASE_NOTES file? > >Do we care that it seems to be incomplete? A human populates both release notes files. I didn't want to

Re: Experiment with UIBase change to classList

2018-03-13 Thread Alex Harui
Hi Carlos, I do not think you are considering all of the scenarios in your proposed code. I'm sad that I have to delineate them again, but I will try. 1) In Basic there are two sets of strings: The fixed set from typeNames that should "never" change. And the className set from the user that

Re: Experiment with UIBase change to classList

2018-03-13 Thread Carlos Rovira
Hi Alex, 2018-03-13 17:50 GMT+01:00 Alex Harui : > Hi Carlos, > > I do not think you are considering all of the scenarios in your proposed > code. I'm sad that I have to delineate them again, but I will try. > > 1) In Basic there are two sets of strings: The fixed set

Re: Experiment with UIBase change to classList

2018-03-13 Thread Alex Harui
After I sent this, I saw one of your code changes that changed typeNames to a getter/setter. Please consider that right now "typeNames" is aggregated in the constructor so that subclasses do not need to override createElement. That means that Button might set typeNames="Button" and ImageButton

Re: [VOTE] Release Apache Royale 0.9.2 RC2

2018-03-13 Thread Piotr Zarzycki
+1 (Binding) - Maven artifacts has been checked with my examples - I did build from sources by Maven - Other files looks good. Maven found some unapproved license - I will report that in the Discussion thread. Windows 10. Thanks, Piotr 2018-03-13 9:08 GMT+01:00 Alex Harui

Re: Experiment with UIBase change to classList

2018-03-13 Thread Carlos Rovira
Hi Alex, 2018-03-13 19:20 GMT+01:00 Alex Harui : > Hi Carlos, > > I took a quick look. > > 1) I don't see how if the user did: > > myComp.className = "Carlos"; > myComp.className = ""; > > Right this case is missing, we need to add it, but seems simple since is deal

Re: Experiment with UIBase change to classList

2018-03-13 Thread Alex Harui
Hi Carlos, Just so I'm clear, you believe that UIBase.as in the jewel-ui-set branch addresses all of these issues? I've just been watching commits, so if you think that's the case then I will look at the current state of your UIBase. Thanks, -Alex On 3/13/18, 10:14 AM, "carlos.rov...@gmail.com

Re: Experiment with UIBase change to classList

2018-03-13 Thread Carlos Rovira
2018-03-13 18:03 GMT+01:00 Alex Harui : > After I sent this, I saw one of your code changes that changed typeNames > to a getter/setter. Please consider that right now "typeNames" is > aggregated in the constructor so that subclasses do not need to override >

Re: Experiment with UIBase change to classList

2018-03-13 Thread Carlos Rovira
Hi Alex, right, I tested it in JewelExample. If you find something that does not conform to what you thinked please let me know to address it. I think it should not be very difficult to handle some isolated case with what we have thanks 2018-03-13 18:24 GMT+01:00 Alex Harui

Re: [DISCUSS] Discuss Release Apache Royale 0.9.2 RC2

2018-03-13 Thread Niclas Hedhman
Correction; KEYS file MUST be in the distribution directory as well. On Wed, Mar 14, 2018 at 1:48 PM, Niclas Hedhman wrote: > > It may be that everyone is ignoring Justin, so I thought I would provide > the link; http://www.apache.org/dev/release-distribution.html#sigs-and >