Re: git commit: [flex-asjs] [refs/heads/develop] - Uploads are assumed to be POST

2017-07-11 Thread Harbs
d wanted to bring up for a long time: I think trace statements should disappear in the release JS build. Should we put all the JS trace code inside an if(goog.DEBUG) block? Thanks, Harbs

[FalconJX] Debugging minified code

2017-07-11 Thread Harbs
, Harbs [1]https://github.com/google/closure-compiler/wiki/Debugging-Compiled-Code <https://github.com/google/closure-compiler/wiki/Debugging-Compiled-Code>

Re: [FlexJS]Runtime type checking and PAYG

2017-07-11 Thread Harbs
ds during development that have more code paths that check for common >>> mistakes is certainly within the charter. We want to maximize developer >>> productivity. >>> >>> I think Google Closure Compiler supports a debug flag you can use in >>> COMPILE

Re: git commit: [flex-asjs] [refs/heads/develop] - Uploads are assumed to be POST

2017-07-11 Thread Harbs
needs to be some way to invoke the app… Harbs > On Jul 11, 2017, at 11:40 PM, Greg Dove wrote: > > Guys, we certainly have been here before. > > From a js release 'size' perspective, I don't think it matters whether we > use constants or liteterals, I think t

Re: [FlexJS]Runtime type checking and PAYG

2017-07-11 Thread Harbs
mpile outside of COMPILE::JS blocks > that might be ok. > > -Alex > > On 7/11/17, 1:07 PM, "Harbs" wrote: > >> I just wasted over an hour because I was initializing a BinaryData with a >> string instead of an ArrayBuffer. >> >> I would like

Re: [FlexJS]Runtime type checking and PAYG

2017-07-11 Thread Harbs
tps://stackoverflow.com/questions/11402218/removing-debug-code-from-inside-a-function-using-closure-compiler-simple-optimis> > On Jul 12, 2017, at 12:37 AM, Harbs wrote: > > Sounds interesting. > > I’m not sure I understand how this would work. I am willing to explore this >

[FlexJS]Runtime type checking and PAYG

2017-07-11 Thread Harbs
catch this in the compiler, because the argument is different for SWF and JS. Is this a violation of PAYG? It’s sort-of just in case code, but not really because it’s protecting against errors. Thoughts? Other solutions? Harbs

Re: [1/5] git commit: [flex-asjs] [refs/heads/develop] - FLEX-35337 place tool tip anchor point in the centre rather than the bottom right

2017-07-11 Thread Harbs
Why are you assuming that this behavior is wrong? Don’t you think a functional change should require discussion? Thanks, Harbs > On Jul 11, 2017, at 8:40 AM, jmcl...@apache.org wrote: > > Repository: flex-asjs > Updated Branches: > refs/heads/develop e93c11d7c -> 1d73de00b

Re: git commit: [flex-asjs] [refs/heads/develop] - Uploads are assumed to be POST

2017-07-11 Thread Harbs
+1. I also think that we have bigger fish to fry first. My point was not to force one way or the other. Rather that the way it currently stands there’s a valid reason to use string literals. I was not suggesting changing anything. Thanks, Harbs > On Jul 11, 2017, at 6:33 PM, Josh Tynj

Re: [FlexJS]MouseEvent.ROLL_OVER (was Re: [2/5] git commit...)

2017-07-11 Thread Harbs
pointer-events:none; Definitely the right way to go. > On Jul 11, 2017, at 6:01 PM, piotrz wrote: > > Alex, > > What do you mean by Mouse "transparent" ? We still need to use some event > mouse (enter/leave) where after roll over on the element tooltip should be > shown. > > Piotr > > > >

Re: [FlexJS]MouseEvent.ROLL_OVER (was Re: [2/5] git commit...)

2017-07-11 Thread Harbs
To be clear: That css should be applied to the ToolTip element. > On Jul 11, 2017, at 6:54 PM, Harbs wrote: > > pointer-events:none; > > Definitely the right way to go. > >> On Jul 11, 2017, at 6:01 PM, piotrz wrote: >> >> Alex, >> >> What do

Re: [FlexJS]MouseEvent.ROLL_OVER (was Re: [2/5] git commit...)

2017-07-11 Thread Harbs
That should have been MOUSE_ENTER rather than MOUSE_OVER. > On Jul 11, 2017, at 6:58 PM, Harbs wrote: > > It’s been supported by every browser since 2014. I think we should switch to > that. > > I was pondering whether we should standardize on ROLL_OVER or MOUSE_OVER. I &

Re: [FlexJS]MouseEvent.ROLL_OVER (was Re: [2/5] git commit...)

2017-07-11 Thread Harbs
was only supported by IE. If we agree it > is safe to use on all of our target browsers, we can switch to using it. > > I don't think it will solve the flicker problem. I think the tooltips > need to be "mouse transparent". > > My 2 cents, > -Alex > >

Re: Flash Builder desktop debugging issue

2017-07-11 Thread Harbs
Same for me. > On Jul 11, 2017, at 4:18 PM, Jeffry Houser wrote: > > To debug a browser based applications, I use Firefox. I couldn't get a > debugger to work in any other browser.

Re: [FlexJS]MouseEvent.ROLL_OVER (was Re: [2/5] git commit...)

2017-07-11 Thread Harbs
performance issues. My question here is why are we not using the standard mouseenter and are using a hacky solution for custom ROLL_OVER events instead. I think Alex wrote that code, so I’m going to wait until he can respond. Thanks, Harbs > On Jul 11, 2017, at 1:50 PM, piotrz wrote: >

Re: git commit: [flex-asjs] [refs/heads/develop] - Uploads are assumed to be POST

2017-07-11 Thread Harbs
PONSE_STATUS','httpResponseStatus');fqa.prototype.h={names:[{name:'HTTPConstants',i:IF,kind:g}]};w(IF,fqa); elsewhere: IF='org.apache.flex.net.HTTPConstants’, That’s 807 bytes. That’s quite a penalty for avoiding typing “POST”… No idea what wiki you are referring to. Ha

CSS compiler issues

2017-07-11 Thread Harbs
I just created a couple of JIRAs on compiler issues that have been bugging me for a while.[1][2] Neither of these are super-important, but it would be nice if it worked better. Harbs [1]https://issues.apache.org/jira/browse/FLEX-35340 <https://issues.apache.org/jira/browse/FLEX-35340&

Re: [FlexJS]MouseEvent.ROLL_OVER (was Re: [2/5] git commit...)

2017-07-11 Thread Harbs
mouseover is needed if you need bubbling. That’s sometimes the correct behavior. > On Jul 11, 2017, at 1:19 PM, piotrz wrote: > > It seems to. MouseOver is launching really often - Maybe Enter/Leave would be > really better. > Need test to see whether we have same behavior having Enter/Leave as

Re: [FlexJS]MouseEvent.ROLL_OVER (was Re: [2/5] git commit...)

2017-07-11 Thread Harbs
and-mouseeventmouse_over-in-as3/> > On Jul 11, 2017, at 12:19 PM, piotrz wrote: > > Harbs, > > I have found following examples [1][2][3]. Maybe it will help us. > > [1] > https://siongui.github.io/2012/08/07/javascript-compare-mouseenter-mouseleave-mou

Re: git commit: [flex-asjs] [refs/heads/develop] - Uploads are assumed to be POST

2017-07-11 Thread Harbs
outweigh the “correctness” of using constants. Thanks, Harbs > On Jul 11, 2017, at 11:46 AM, piotrz wrote: > > I just think that we should use const whenever we can, but won't fight for > that if no one think similar. :) If I found such code around my interest my > natural mov

[FlexJS]MouseEvent.ROLL_OVER (was Re: [2/5] git commit...)

2017-07-11 Thread Harbs
and ROLL_OUT events for any elements which have listeners registered for them. This seems horribly inefficient and I don’t understand why it’s necessary. It seems to me that mouseenter and rollOver (as well as mouseleave and rollOut) are analogous. Am I missing something? Thanks, Harbs > On

Re: [2/5] git commit: [flex-asjs] [refs/heads/develop] - FLEX-35338 stop cursor from spinning / CPU from spiking

2017-07-11 Thread Harbs
unnecessary mouse events when mousing in and out of the tool tip. On the other hand, placing it inside the target element might effect layout and the like. It’s possible that this can be solved using absolute/fixed positioning and zindex. Thoughts? Harbs > On Jul 11, 2017, at 11:16 AM, Jus

Re: [2/5] git commit: [flex-asjs] [refs/heads/develop] - FLEX-35338 stop cursor from spinning / CPU from spiking

2017-07-11 Thread Harbs
re the MouseEvent constants wrong for Javascript? > On Jul 11, 2017, at 10:38 AM, Harbs wrote: > > Shouldn’t the event be triggered by rollover rather than mouseover? > > That seems like the issue to me… > >> On Jul 11, 2017, at 10:13 AM, Alex Harui wrote: >> &g

Re: [2/5] git commit: [flex-asjs] [refs/heads/develop] - FLEX-35338 stop cursor from spinning / CPU from spiking

2017-07-11 Thread Harbs
to rollOver. That should mean that mouseenter should be the better option. [2]https://www.quirksmode.org/js/events_mouse.html <https://www.quirksmode.org/js/events_mouse.html> > On Jul 11, 2017, at 10:48 AM, Harbs wrote: > > No. I’m wrong. > > First of all, it’s m

Re: [2/5] git commit: [flex-asjs] [refs/heads/develop] - FLEX-35338 stop cursor from spinning / CPU from spiking

2017-07-11 Thread Harbs
Did you try mouseenter? (See my other responses) It seems like that should solve the problem. > On Jul 11, 2017, at 11:01 AM, Justin Mclean wrote: > > Hi, > > I not sure how this could be consider "just in case" when previous the > tooltip didn’t work and had an infinite loop? Before it was i

Re: [2/5] git commit: [flex-asjs] [refs/heads/develop] - FLEX-35338 stop cursor from spinning / CPU from spiking

2017-07-11 Thread Harbs
Shouldn’t the event be triggered by rollover rather than mouseover? That seems like the issue to me… > On Jul 11, 2017, at 10:13 AM, Alex Harui wrote: > > This seems like "just-in-case" code. I could see that it might be needed > in a different bead for Express though. > > Without this change

Re: [FlexJS] Bindable and consts

2017-07-10 Thread Harbs
that I keep putting off until we’re in a better place. Harbs > On Jul 10, 2017, at 6:54 PM, Alex Harui wrote: > > How urgent is this? I'm trying to figure out why the compiler did not > respond properly to bad MXML. > > I thought public APIs were all exported so the

Re: [FlexJS] States Bug?

2017-07-10 Thread Harbs
It does. It was code that was imported from a regular Flex component. It worked in the old app (although it was probably not necessary). > On Jul 10, 2017, at 6:45 PM, Alex Harui wrote: > > If you have time, see if it works in regular Flex. > > Thanks, > -Alex > > On

Re: [FlexJS] technical debt

2017-07-10 Thread Harbs
How can you use filtering to change what the dashboard shows? > On Jul 10, 2017, at 2:02 PM, Justin Mclean wrote: > > Hi, > >> I was trying to set up an alternate set because it’s easier to have two sets >> to compare rather than having long drawn-out discussion. > > As opposed to using the f

Re: [FlexJS] Bindable and consts

2017-07-10 Thread Harbs
? Thanks, Harbs > On Jul 9, 2017, at 11:27 PM, Harbs wrote: > > I just discovered something: > > Foo.as: > package com.acme.foobaz.model{ > [Bindable]public class Foo{ > static public const BAZ:String = “baz”; > } > } > > In some other

Re: [FlexJS] States Bug?

2017-07-10 Thread Harbs
The bug (if it is one) seems to be when using nested includeIn attributes. What happens is that you get two objects with the same key (i.e. the same object is included twice in the class) > On Jul 10, 2017, at 11:07 AM, yishayw wrote: > > It looks like the compiler gets confused when we add ne

Re: [FlexJS] Question about Events/possible bugs

2017-07-10 Thread Harbs
It could, but there’s no reason to. The Clipboard class is just a proxy to the native Flash Clipboard class on the SWF side. > On Jul 10, 2017, at 11:02 AM, piotrz wrote: > > Harbs, > > I think this import could be outside COMPILE::JS. > > Piotr > > &

Re: [FlexJS] Question about Events/possible bugs

2017-07-10 Thread Harbs
GraphicContainer was another one I was involved in, so I know that’s an omission as well. (committed) > On Jul 10, 2017, at 11:16 AM, Harbs wrote: > > Good catch. > > Without looking at them, I’d guess that they are bugs. Clipboard was my error > and I just commit

Re: [FlexJS] Question about Events/possible bugs

2017-07-10 Thread Harbs
Good catch. Without looking at them, I’d guess that they are bugs. Clipboard was my error and I just committed a fix for that. Thanks, Harbs > On Jul 10, 2017, at 11:06 AM, Greg Dove wrote: > > I noticed a problem with VerticalFlexLayout in IE11 (and presumably older > IE browser

Re: [FlexJS] technical debt

2017-07-10 Thread Harbs
That’s a good idea, and I will probably do so next time I’m inclined to spend time on this. > On Jul 10, 2017, at 10:43 AM, Justin Mclean wrote: > > If you are trying things out rather than experimenting on the live server you > can run Sonar Cube locally.

Re: [FlexJS] technical debt

2017-07-10 Thread Harbs
time for this now so I’m going to just leave this alone for the time being. Thanks, Harbs > On Jul 10, 2017, at 10:43 AM, Justin Mclean wrote: > > Hi, > >> I have permissions to edit profiles, but I don’t seem to have permissions to >> assign profiles. > > You d

Re: [FlexJS] technical debt

2017-07-10 Thread Harbs
I have permissions to edit profiles, but I don’t seem to have permissions to assign profiles. I created another profile and I’m waiting for INFRA to look at my permissions again. > On Jul 10, 2017, at 9:28 AM, piotrz wrote: > > Hi Justin, > > Thank you. > > Harbs, >

[FlexJS] Bindable and consts

2017-07-09 Thread Harbs
com.acme.foobaz.model.Foo becomes k) k.BAZ even though k.BAZ was optimized away and k.BAZ is undefined. Why does [Bindable] on a class cause the bracket notation to be used? Harbs

Re: [FlexJS]DropDownList dataProvider

2017-07-09 Thread Harbs
I fixed it. If someone feels that it’s not a bug we can revert the commit... Next issue… ;-) > On Jul 9, 2017, at 9:27 PM, Harbs wrote: > > It was always like that, but I think it’s incorrect. > > You can go back in the history, but you need to follow the changes across the

Re: [FlexJS]DropDownList dataProvider

2017-07-09 Thread Harbs
It was always like that, but I think it’s incorrect. You can go back in the history, but you need to follow the changes across the rename from HTML. SourceTree makes that pretty easy. > On Jul 9, 2017, at 9:00 PM, piotrz wrote: > > I cannot check when those changes was introduced, cause for so

Re: [FlexJS]DropDownList dataProvider

2017-07-09 Thread Harbs
Yeah. I noticed that too. > On Jul 9, 2017, at 8:20 PM, piotrz wrote: > > Additionally I see in ArraySelectionModel which has been used in DropDownList > in dataProvider setter this one: > > if (value === this._dataProvider) >return; > > Piotr > > > > - > Apache Flex PMC > piotrzarz

Re: [FlexJS]DropDownList dataProvider

2017-07-09 Thread Harbs
://issues.apache.org/jira/browse/FLEX-35336 <https://issues.apache.org/jira/browse/FLEX-35336> Harbs > On Jul 9, 2017, at 7:19 PM, Harbs wrote: > > Yes. I’ll try to put together a test case. > >> On Jul 9, 2017, at 6:57 PM, piotrz wrote: >> >> Actually could

Re: [FlexJS]DropDownList dataProvider

2017-07-09 Thread Harbs
Yes. I’ll try to put together a test case. > On Jul 9, 2017, at 6:57 PM, piotrz wrote: > > Actually could you provide small example ? I would like to make sure myself > that is a bug. > > Thanks, > Piotr > > > > - > Apache Flex PMC > piotrzarzyck...@gmail.com > -- > View this message in

[FlexJS]DropDownList dataProvider

2017-07-09 Thread Harbs
). I leaning to classifying it as a bug because it seems like having a requirement that the bindable property should be initialized is overly stringent. Thoughts? Harbs

Re: [FlexJS] technical debt

2017-07-09 Thread Harbs
Actually I did not miss that. The Flex code is an external project: https://code.google.com/archive/p/jsf-flex/ <https://code.google.com/archive/p/jsf-flex/> I’ll be happy to communicate. Thanks, Harbs > On Jul 9, 2017, at 1:39 PM, Justin Mclean wrote: > > Hi, > >

Re: [FlexJS] technical debt

2017-07-09 Thread Harbs
rift. You can see the source code here.[2] As you can see on the SonarQube dashboard[3], there are a total of 21 projects using SonarQube. 5 of those are Flex projects. Of the remaining 16 projects, it’s very easy to see that none of them are using the Flex language. Thanks, Harbs [1]https://

Re: [FlexJS] Use of typeof

2017-07-09 Thread Harbs
eck for "window" and a few > other things. > > HTH, > -Alex > > On 7/5/17, 1:54 PM, "Harbs" wrote: > >> No. I was trying to use process to check whether it’s running in a Node >> runtime (such as Node or Electron). window does not have proce

Re: [FlexJS] technical debt

2017-07-08 Thread Harbs
should be constant is not something that Flex ever enforced. I don’t really see the value for this rule, but I could go either way. There are some other rules which should be discussed, but we can do that on the wiki. Feel free to revert my changes, but the discussion should be about defaults for

Re: FlexJS Tests (was Re: [FlexJS] technical debt)

2017-07-07 Thread Harbs
the JS run, but some manual > intervention is currently required. > > -Alex > > On 7/7/17, 3:16 AM, "Harbs" wrote: > >> The checkintests target currently fails. I’m guessing the build file was >> not updated for dual. >> >> Selenium has a JS API[1]

Re: FlexJS Tests (was Re: [FlexJS] technical debt)

2017-07-07 Thread Harbs
liking Mustella and some > thoughts about how to make FlexUnit run on both platforms. > > My personal preference is that tests should be written in MXML and/or AS > and run on both platforms. Bonus points if existing tests can be run > unmodified or mostly unmodified. > > -Alex

Re: [FlexJS][VS Code] dual issues

2017-07-07 Thread Harbs
Sounds great! Looking forward! Harbs > On Jul 7, 2017, at 2:12 AM, Josh Tynjala wrote: > > Okay, I figured out how to make VSCode switch to JS APIs instead of SWF > APIs for completion and things. In the next version of the extension, when > the targets compiler option

Re: [FlexJS] technical debt

2017-07-07 Thread Harbs
I don’t see any options like that (logged in using my Apache account). Does someone need to add permissions? > On Jul 7, 2017, at 10:49 AM, Justin Mclean wrote: > > Hi, > >>> and how do we change them? > > In the sonar cube web interface (same url as before) under rules/quality > profile. It

Re: [FlexJS] technical debt

2017-07-07 Thread Harbs
Considering you did not answer the most significant part of my question. I’m asking it again. > On Jul 6, 2017, at 9:13 PM, Harbs wrote: > > and how do we change them?

[FlexJS][VS Code] dual issues

2017-07-06 Thread Harbs
are specific to SWF-only output: a. I’m not getting JS specific classes recognized. b. I’m getting lots of incompatible overrides (to Flash methods) c. I’m getting overriding function not marked for override errors on functions that are implemented in flash (i.e. public function get scaleX():Number) d. conflicts between variables and Flash getters. e. etc... Thanks, Harbs

FlexJS Tests (was Re: [FlexJS] technical debt)

2017-07-06 Thread Harbs
Where are the instructions on how to use it? If I know how to write tests, I’d be much better about doing so… > On Jul 6, 2017, at 11:53 PM, Alex Harui wrote: > > I have put in place a test infrastructure

Re: [FlexJS] technical debt

2017-07-06 Thread Harbs
Yup. For example: I’d say an extremely large percentage of the “technical debt” in TLF is there for a reason. Do NOT “fix” any of that… > On Jul 6, 2017, at 11:53 PM, Alex Harui wrote: > > But IMO, for right now, if "it ain't broke, don't fix it".

Re: [FlexJS] 0.8.0 not working in IntelliJ IDEA

2017-07-06 Thread Harbs
VS Code is reporting all kinds of issues that are not problems since the switch to dual. Maybe that’s related as well? > On Jul 6, 2017, at 9:06 PM, Josh Tynjala wrote: > > I don't see why -targets wouldn't work when it works elsewhere, but I just > tried -compiler.targets too, and that also fa

Re: [FlexJS] technical debt

2017-07-06 Thread Harbs
A large percentage of the complaints are inaccurate. The reason the technical debt increased is largely because of the new TLF code. Who made these rules and how do we change them? We can make it look much better by just changing some of the rules… Harbs > On Jul 6, 2017, at 2:24 AM, Jus

Re: [FlexJS] Use of typeof

2017-07-05 Thread Harbs
think that would offer a solution everywhere. > On Jul 5, 2017, at 7:48 PM, Alex Harui wrote: > > Sure, I know it wouldn't work at runtime, but it sounded like Harbs > couldn't even get the compiler to accept window["process"] which it should. > > So, it

Re: [FlexJS] Mobile Depends on Flash 11.4?

2017-07-04 Thread Harbs
Where is that from? In build.properties I see this: # flex-sdk-description values release.name = Apache Flex (FlexJS) # this is the version that appears in the flex-sdk-description tag # and on the package name. This is the publicly known version of FlexJS release.version = 0.9.0 # fb.release.

Re: [FlexJS] Use of typeof

2017-07-04 Thread Harbs
in case I’m missing something.) Harbs > On Jul 5, 2017, at 12:00 AM, Harbs wrote: > > Looks like it. > > I see this in missing.js: > > /** > * @export > * This gets mapped to org.apache.flex.utils.Language.trace() by the compiler > * @param {...} r

Re: [FlexJS] Use of typeof

2017-07-04 Thread Harbs
{!Process} * @const */ var process; However, it seems like a drag to have to add a typedef every time a developer needs to check for the existence of a global that we did not think of. > On Jul 4, 2017, at 9:13 PM, Harbs wrote: > > Thanks. Here’s what I see: > > if(typeof window

Re: [FlexJS] Use of typeof

2017-07-04 Thread Harbs
n order to get trace() working in > Node.js, I had to figure out how to find the console object on window > versus global. I feel like I remember using typeof, but maybe it was > something else. Take a look at the implementation of Language.trace() to > see what I did. > > -

Re: Installer Problems

2017-07-04 Thread Harbs
Thanks for taking care of that. :-) > On Jul 4, 2017, at 8:10 PM, piotrz wrote: > > Issue has been fixed. Everyone who had the problems can try to download > FlexJS. > > Thanks, > Piotr > > > > - > Apache Flex PMC > piotrzarzyck...@gmail.com > -- > View this message in context: > http:/

[FlexJS] Use of typeof

2017-07-04 Thread Harbs
you to do that and complains that process is an undefined property. In the browser you can use window[“process”] == undefined and in node you can (theoretically) use global[“process”] == undefined. I can’t think of a generic way to do this though. Thoughts? Harbs

Re: Installer Problems

2017-07-04 Thread Harbs
https://www.npmjs.com/package/flexjs > On Jul 4, 2017, at 2:13 PM, piotrz wrote: > > Do we have some alternative to provide users where we can download by SDK > with all dependencies ? > > > > - > Apache Flex PMC > piotrzarzyck...@gmail.com > -- > V

Re: TLF Branch merged back

2017-07-03 Thread Harbs
BTW: Sorry for forgetting to merge Falcon as well and thanks Alex for taking care of it. > On Jul 3, 2017, at 4:41 PM, Harbs wrote: > > I just merged the TLF branch back into develop. > > TLF is still “alpha quality”, so unless you do a LOT of custom work on a > propr

Re: Installer Problems

2017-07-03 Thread Harbs
Yeah. That’s probably the issue with AIR 25 being too big… > On Jul 3, 2017, at 10:08 PM, piotrz wrote: > > Alex, Olaf, > > I just tried after some time installer and it's downloaded FlexJS and > compiler, but it failed in uncompressing Adobe Air 25 [1]. > > With configuration FlexJS 0.8.0 and

Re: Installer Problems

2017-07-03 Thread Harbs
However,. it looks like it still download it from Downloading Apache FlexJS from:http://www-us.apache.org/dist/flex/flexjs/0.8.0/binaries/ > On Jul 3, 2017, at 9:27 PM, Harbs wrote: > > I tried and this time it worked. > > Earlier today I got the same error that others reporte

Re: Installer Problems

2017-07-03 Thread Harbs
I tried and this time it worked. Earlier today I got the same error that others reported. > On Jul 3, 2017, at 9:06 PM, piotrz wrote: > > Hi Alex, > > I just tried and got error [1] same as previous I think. > > [1] https://paste.apache.org/90UQ > > Thanks, > Piotr > > > > - > Apache

[Legal] Code from Stack Overflow

2017-07-03 Thread Harbs
March 2016, where should attribution be put? Should a link be included in the code? A mention of the author in NOTICE? Should we include a link back to the SO post? Should this be asked on legal? Harbs [1]https://stackoverflow.com/a/2401861 [2]https://meta.stackexchange.com/questions/272956/a-new

Feature Branches (was Re: TLF Branch merged back)

2017-07-03 Thread Harbs
preference is to only bother with feature branches for changes that have a decent chance of breaking things. I don’t really care about naming convetions, but naming a branch with a JIRA id for cross-referencing JIRA fixes makes sense. Thanks, Harbs > On Jul 3, 2017, at 4:29 PM, piotrz wrote: > &

TLF Branch merged back

2017-07-03 Thread Harbs
those in when a feature is complete. Is that right? Harbs

Re: git commit: [flex-asjs] [refs/heads/tlf] - First implementation of FileProxy which takes on some of the functions of FileReference in Flash. More beads coming soon.

2017-07-03 Thread Harbs
Not sure. It relies on Network. Adding it to Storage would add a Network dependency to Storage. > On Jul 3, 2017, at 2:40 PM, piotrz wrote: > > Another thing which come up to my mind is - Maybe this FileProxy > implementation should be included in Storage module. > > Just thinking loudly. >

Re: [ANNOUNCE] Apache FlexJS 0.8.0 and Apache Flex FalconJX 0.8.0 Released

2017-07-03 Thread Harbs
Fixed. > On Jul 3, 2017, at 11:15 AM, piotrz wrote: > > Hi Harbs, > > Thanks! Awesome! > > Let me know and I will at least test it. Didn't have enough time todays > morning to look into that. > > Piotr > > > > - > Apache Flex PMC &g

Re: [ANNOUNCE] Apache FlexJS 0.8.0 and Apache Flex FalconJX 0.8.0 Released

2017-07-03 Thread Harbs
This is probably the issue: http://archive.apache.org/dist/flex/flexjs/0.5.0/binaries/"; file="apache-flex-flexjs-0.5.0-bin"/> http://archive.apache.org/dist/flex/flexjs/0.6.0/binaries/"; file="apache-flex-flexjs-0.6.0-bin"/> http://archive.apache.org/dist/flex/flexjs/0.6.0/binaries/"; file="apa

Re: [ANNOUNCE] Apache FlexJS 0.8.0 and Apache Flex FalconJX 0.8.0 Released

2017-07-03 Thread Harbs
Did Apache change something with preferred mirrors? > On Jul 3, 2017, at 10:08 AM, Harbs wrote: > > It looks like something is messed up with the mirror. > > I’ve never spent the time to figure out how the installer works… :-( > >> On Jul 3, 2017, at 9:35 AM, piot

Re: [ANNOUNCE] Apache FlexJS 0.8.0 and Apache Flex FalconJX 0.8.0 Released

2017-07-03 Thread Harbs
It looks like something is messed up with the mirror. I’ve never spent the time to figure out how the installer works… :-( > On Jul 3, 2017, at 9:35 AM, piotrz wrote: > > I just tried it and also have problems [1] > > [1] https://paste.apache.org/wWPR > > Piotr > > > > - > Apache Flex

Re: [FlexJS] PAYG definitions and guidance, Please participate

2017-07-02 Thread Harbs
was being extreme with his use of PAYG and splitting up of beads. However, I went along with it, and with time, I see the wisdom of this approach. Thanks, Harbs > On Jul 2, 2017, at 4:44 PM, yishayw wrote: > > Alex Harui-2 wrote >> One thing I wanted put up >> for discuss

Re: [FlexJS] PAYG definitions and guidance, Please participate

2017-07-02 Thread Harbs
have layout beads which go in both directions relative to what we currently have: Leaner beads which are more performant, and more robust beads which have more ease of layout. Harbs > On Jul 2, 2017, at 9:44 AM, Alex Harui wrote: > > Documentation is always a good thing, but as

Re: [FlexJS] PAYG definitions and guidance, Please participate

2017-07-02 Thread Harbs
need better documentation and guides. Unfortunately most of us are not great writers. :-( Harbs > On Jul 2, 2017, at 1:40 AM, Greg Dove wrote: > > Thanks Alex, > > I look forward to your feedback. It would be great to see (from you or > anyone else) - within this

Re: [FlexJS] question about porting an Adobe Flex 3 project to HTML+JS

2017-06-30 Thread Harbs
They have build scripts for both Ant and Maven. You can probably build them using any one of the other build chains, but those are the only build scripts included. Harbs > On Jun 30, 2017, at 6:07 PM, Allen YANG wrote: > > Piotr, > Thank you. Are these examples all built on M

Re: [FalconJX] Slower builds after dual merge

2017-06-30 Thread Harbs
I initially thought that I was getting slower build times, but I’m no longer so sure. Using these options, I’m getting roughly 10 second build times --debug=true --targets=JSFlex That’s about the same as I was getting before the switch to dual. Using debug=false sometimes takes over 3 minutes.

Re: [FlexJS] question about porting an Adobe Flex 3 project to HTML+JS

2017-06-30 Thread Harbs
help. Good luck! Harbs > On Jun 30, 2017, at 3:19 PM, Allen YANG wrote: > > Hi Team, > I have a Flex 3 project that was built on Adobe Flash Builder 3.6 (SDK Flex > 3.6A); it needs to be ported to support a non-flash environment. This > program was not originally written

Re: git commit: [flex-asjs] [refs/heads/develop] - Fix merge error removing support for numeric font size.

2017-06-29 Thread Harbs
Yes. > On Jun 29, 2017, at 12:54 PM, Justin Mclean wrote: > > Did you actually try using that class / compiling it?

Re: [FLEXJS][FALCONJX] Variable Initialization, equality, etc.

2017-06-29 Thread Harbs
be interested in hearing his findings. Thanks, Harbs > On Jun 29, 2017, at 1:07 PM, Justin Mclean wrote: > > HI, > >> There are tests for both, and you DON’T get type conversions unless it’s >> required. Browsers are smarter than that. > > See [1] and follow th

Re: git commit: [flex-asjs] [refs/heads/develop] - Fix merge error removing support for numeric font size.

2017-06-29 Thread Harbs
Check your basic-manifest.xml for Core. Do you see

Re: [FLEXJS][FALCONJX] Variable Initialization, equality, etc.

2017-06-29 Thread Harbs
> On Jun 29, 2017, at 12:19 PM, Justin Mclean wrote: > > Hi, > >> The JS version of the test is here: >> http://home.apache.org/~aharui/FlexJS/FLEX35330/bin/js-release/ >> >> The SWF version of the test is here: >> http://home.a

Re: git commit: [flex-asjs] [refs/heads/develop] - Fix merge error removing support for numeric font size.

2017-06-29 Thread Harbs
No. It’s there. Make sure you have commit af1f5c9 in your build of Core. > On Jun 29, 2017, at 12:21 PM, Justin Mclean wrote: > > Hi, > >> That’s why you should use > > You can’t in this case as it doesn’t compile with the following error: > > This tag could not be resolved to an ActionScri

Re: [FLEXJS][FALCONJX] Variable Initialization, equality, etc.

2017-06-29 Thread Harbs
was that Chrome was roughly twice as fast as Firefox and the Flash version took about 3 times as long in Chrome. I did not bother testing the Flash version in Firefox because I have the debug Flash plugin installed there. Harbs > On Jun 29, 2017, at 10:23 AM, Alex Harui wrote: > > Now

Re: git commit: [flex-asjs] [refs/heads/develop] - Fix merge error removing support for numeric font size.

2017-06-29 Thread Harbs
I’m not sure what your point is. That’s why you should use SimpleCSSValuesImpl is supposed to be a simple implementation which works the same in Flash and HTML. AllCSSValuesImpl is supposed to be an implementation which works with valid HTML CSS. Thanks, Harbs > On Jun 29, 2017, at 11:34

Re: git commit: [flex-asjs] [refs/heads/develop] - Fix merge error removing support for numeric font size.

2017-06-29 Thread Harbs
A font weight of 600 will fail in Flash. It’s CSS that only works in HTML. > On Jun 29, 2017, at 10:49 AM, Justin Mclean wrote: > > Hi, > >> Considering we have AllCSSValuesImpl, I think that should be the default for >> apps which are expecting HTML CSS. > > It’s not about HTTML CSS. > > Th

Re: git commit: [flex-asjs] [refs/heads/develop] - Fix merge error removing support for numeric font size.

2017-06-29 Thread Harbs
fine. Harbs > On Jun 29, 2017, at 10:26 AM, Justin Mclean wrote: > > Hi, > > But you did agreed that the code in question should stay in CSSValuesImpl. > > From you last email in that thread: > > "I agree with Justin on this. I don’t see why numeric f

Re: git commit: [flex-asjs] [refs/heads/develop] - Fix merge error removing support for numeric font size.

2017-06-29 Thread Harbs
And it was about AllCSSStyles and not AllCSSValuesImpl. I see no reason to not use that one. > On Jun 29, 2017, at 10:14 AM, Harbs wrote: > > My suggestion was to copy it and prune the ones that are not used. > >> On Jun 29, 2017, at 9:49 AM, Justin Mclean

Re: git commit: [flex-asjs] [refs/heads/develop] - Fix merge error removing support for numeric font size.

2017-06-29 Thread Harbs
My suggestion was to copy it and prune the ones that are not used. > On Jun 29, 2017, at 9:49 AM, Justin Mclean wrote: > > Hi, > >> AllCSSValuesImpl - is not enough performant cause of lack of strict >> comparison ? > > Just the large number of styles.When Ha

Re: [FlexJS,TLF] Tlf branch and Maven build

2017-06-28 Thread Harbs
Yes. You need that branch. > On Jun 28, 2017, at 9:34 AM, yishayw wrote: > > I'll try switching falcon to tlf branch. I was using release up to now. > > > > -- > View this message in context: > http://apache-flex-development.247.n4.nabble.com/FlexJS-TLF-Tlf-branch-and-Maven-build-tp62452

Re: [FlexJS,TLF] Tlf branch and Maven build

2017-06-27 Thread Harbs
falcon changes that are required.) > On Jun 27, 2017, at 9:46 PM, piotrz wrote: > > Hi Harbs, > > I see that your TLF branch is failing on Maven build [1] with following > stack: > > Loading configuration: > F:\jenkins\jenkins-slave\wo

[FlexJS] Use of innerHTML

2017-06-27 Thread Harbs
]? Harbs [1]https://developer.mozilla.org/en-US/docs/Web/API/Element/innerHTML#Security_considerations <https://developer.mozilla.org/en-US/docs/Web/API/Element/innerHTML#Security_considerations> [2]https://developer.mozilla.org/en-US/docs/Web/API/ParentNode/append <https://developer.mo

Re: git commit: [flex-asjs] [refs/heads/tlf] - Reverted strict equality

2017-06-26 Thread Harbs
used) and was able to locate the error quickly. Our specific case was one where there was not getter defined at all for the value. > On Jun 25, 2017, at 8:30 PM, Harbs wrote: > > However, after thinking about this some more, I’m not sure that the SWF > behavior is actually desir

Re: git commit: [flex-asjs] [refs/heads/tlf] - Reverted strict equality

2017-06-25 Thread Harbs
I’m not sure how you will do that unless you use every part of the SDK. My point was that I would like to try it on my own app which does have quite extensive SDK use (as well as many, many other classes). I will do that if and when we have these compiler options in Falcon. > On Jun 26, 2017, a

<    1   2   3   4   5   6   7   8   9   10   >