Re: [FlexJS] Parallel Effect

2017-01-24 Thread Alex Harui
On 1/23/17, 11:49 PM, "Harbs" wrote: >The question is how far to take this. > >Should every method in Effect be a separate interface? i.e. IPlayable, >IReversable, IPausable, IStoppable, IResumable. That seems a bit >over-the-top to me. > >As IEffect currently stands, Parallel needs the methods

Re: [FlexJS] Parallel Effect

2017-01-23 Thread Harbs
The question is how far to take this. Should every method in Effect be a separate interface? i.e. IPlayable, IReversable, IPausable, IStoppable, IResumable. That seems a bit over-the-top to me. As IEffect currently stands, Parallel needs the methods implemented. Personally, I think Parallel sh

Re: [FlexJS] Parallel Effect

2017-01-22 Thread Alex Harui
On 1/22/17, 7:42 AM, "Yishay Weiss" wrote: >Thanks Piotr and Alex. It’ll be an interesting exercise to see if this >can be made into a strand. My issue with ParallelWithReverse is that >Parallel already implements IEffect which has reverse(). What does that >say about Parallel? Does it or does i

RE: [FlexJS] Parallel Effect

2017-01-22 Thread Yishay Weiss
pache.org> Subject: Re: [FlexJS] Parallel Effect On 1/22/17, 5:05 AM, "piotrz" wrote: >Hi Yishay, > >Personally I'm always for as small class as we can. My +1 for >ParallelWithReverse I agree. It might be worth considering making a subclass of Parallel that is a St

Re: [FlexJS] Parallel Effect

2017-01-22 Thread Alex Harui
On 1/22/17, 5:05 AM, "piotrz" wrote: >Hi Yishay, > >Personally I'm always for as small class as we can. My +1 for >ParallelWithReverse I agree. It might be worth considering making a subclass of Parallel that is a Strand so the reverse, pause and stop can be beads. Are there more than one wa

Re: [FlexJS] Parallel Effect

2017-01-22 Thread piotrz
Hi Yishay, Personally I'm always for as small class as we can. My +1 for ParallelWithReverse Piotr - Apache Flex PMC piotrzarzyck...@gmail.com -- View this message in context: http://apache-flex-development.247.n4.nabble.com/FlexJS-Parallel-Effect-tp58554p58558.html Sent from the Apa