gentle optimization of PR #549

Revert "optimize loop performance"

This reverts commit 98fdf3202901017dcdf16cde910ddd896aab519d.

Signed-off-by: dotnetCarpenter <jon.ronnenberg@gmail.com>

optimize loop performance

Fix going down prototype chain

This fixed the a.undo is not a function error. The for loop was going down into  the array's prototype chain.

Merge pull request #548 from RmiTtro/fix-transform-anim

Make transform more consistent when animated. Fixes bug with animated matrices

Fix the bug where relative matrix transforms are not animated

Also, add the possibility to make uniform skew transform.

Fix a bug with the animation of a scale transform

Fix the bug where animating a scale transform also caused a translation
when there was already a transform in place.

update doc for SVG() in alignment with comment: https://github.com/svgdotjs/svg.js/issues/139#issuecomment-256987267

changelog

Merge pull request #546 from RmiTtro/fix-and-improve-fx-dequeue

Fix and improve the dequeue method of the FX module

Fix and improve the dequeue method of the FX module

The first improvement is to stop using setTimeout to perform the delay. It is
now performed in a manner similar to the start method. This should be a lot
more reliable than using setTimeout and allow some nice goodies like
being able to pause it or affect its duration using the speed method.

The second improvement is to have the delay method add an empty situation
(with its duration set to the duration of the delay) to the queue. This change
allows dequeue not to have to treat delay added to the queue as something
special, now its just a situation.

changelog, run gulp cause of #504

Merge pull request #504 from tkiss/animations_es6-7_fix

Fix broken animations, if using polyfills for es6/7 proposals

Merge pull request #545 from RmiTtro/fix-skew-transform

Fix the implementation of the skew transform

Fix the implementation of the skew transform

Also fix a bug where when calling scale with 3 parameters, cx was not set
with the right value.

Merge pull request #538 from RmiTtro/make-fx-at-reverse-aware

Make SVG.FX.at reverse aware

Make the code in the FX module simpler

Here are the changes that have been made:
  - The loop counter is now incrementing (from 0 to loops)
  - The loop counter increment even when loops is true
  - Add absPos, the absolute position of an animation which is its position in
 the context of its complete duration (including delay and loops)
  - Make the methods speed, pause/resume affect the delay
  - The method step no longer needs to recalculate the value of situation.start
  - Add a second parameter to the method at to allow it to receive an absolute
  position
  - Fix the bug where the method at invert the passed position while the
  animation is running backward

Also, I modified the tests of the FX module that required timing to use
Jasmine Clock, so now it takes less than 1 second to run all the tests!

changelog

Merge pull request #543 from RmiTtro/individual-transform-separated-by-comma

Make matrixify work with transformation chain separated by commas

Make matrixify work with transformation chain separated by commas

According to the SVG spec, transformation chain can be separated by whitespace
and/or commas. The method matrixify was not working with transformation chain
separated by commas. This commit should fix that bug.

update links in package to the new project home

update README links to point to the new project home

fix spelling

Changed author email

bump 2.3.6

added test for silently fixing wrong point parameters to SVG.PointArray + better descriptions for SVG.PointArray tests

fix leading/trailing space test case

odd number of points is invalid in SVG
See https://svgwg.org/svg2-draft/shapes.html#DataTypePoints

handle leading and trailing spaces

replace call to filter function with better regex

correct naming of test

speed up travis tests by caching the node_modules folder - should shave off 23 seconds of test run time

hotfix (#529)

fix wrong doc for local testing (#537)

* fix wrong doc for local testing

* fix html ending - thanks @RmiTtro

changelog

Merge pull request #536 from RmiTtro/fx-speed-like-other-lib

Make SVG.FX.speed act more like others animations libraries

Change the SVG.FX attribute spd to _speed

This change is to follow the convention already put forward by the _target
attribute.

Fix Build and Test section

Fix command to build and added a small section about testing

Add speed to SVG.Element and also add documentation to speed

Make the tests of timeToPos() and posToTime() easier to understand

Make tests relating to speed easier to understand

Make speed(0) pause the animation

Make SVG.FX.speed act more like others animations libraries

For example, a call .speed(2) would make the animation 2 times faster.
If you then call .speed(4), this would make the animation 4 times faster.
And a call .speed(1) would make it go back to normal speed.

changelog

Merge pull request #534 from RmiTtro/fix-afterall-test

Fix the test of SVG.FX.afterAll

use default linux image, change name of nvm node version to stable instead of node - it is the same version but stable is a better name, build the source before testing in case someone only push changes to the source and don't build

Fix the test of SVG.FX.afterAll

changelog updated

Merge pull request #532 from RmiTtro/fix-fx-loop

Make SVG.FX.loop modify the last situation instead of the current one

Rebuild of the library

Add tests for SVG.FX.loop

Make SVG.FX.loop modify the last situation instead of the current one

Fixed error in README

Update LICENSE.txt

Updated date in license

bump 2.3.5

Merge pull request #529 from dotnetCarpenter/pointarray

string from PointArray is now correctly parsed (fixes  #525)

update README for version 2.3.5 which I believe this PR will be part of

all tests pass - All your base are belong to us

fixed id in test which is different in headless browser

Removed wrong tests in sugar.js, fixed remaining tests. Made fill and stroke a nop when no argument given

remove redundant test

re-include dist folder

remove dist folder from PR

sorry the last commit was nonsens. This one is proper tested.

changing implementation according to review by @Fuzzyma

fixes #525 by accepting/parsing coordianate strings where x and y is seperated by comma and or by space - we count the shape coordinates until (and including) the maximum even number

Added failing test for PointArray

Merge pull request #527 from dotnetCarpenter/unittest

Unittests with travis. Thx @dotnetCarpenter

fixed specs for SVG.Point, fixes #530

tests now pass in Firefox

reinstate version number - changed it by mistake

Merge branch 'unittest' of https://github.com/dotnetCarpenter/svg.js into unittest

use 'npm version' instead of setting versions manually

remove version from bower and use git tag instead - use 'npm version premajor|major|minor|patch' to proper tag and set version in package.json

3.0.0-0

use the 'latest' version of firefox

use a newer ubuntu image and the latest firefox (49.0)

Update document up to include travis status and updated README and CHANGELOG - I think this PR will be in the 3.0 release

reinstate semi colon in banners

update badge and log firefox version on travis

be sure that firefox is not running before test starts

install newest firefox on travis currently we use 38.0 but the current version is 49.0

switch back to firefox for testing

halt dev of chrome base solution

chromium-browser is already the newest version. Uncomment to enable update of the chromium browser on travis.

fix syntax error

test with newer version of chrome

--amend

configure chromium to be chrome on travis

probing for chrome on travis

Many of the failing tests are rounding errors. Different browsers delivered different results. Thats why we took chrome as base.

remove redundant npm install - travis do this for us because we're a node project

Firefox 31 is really old - we want to test against a newer browser

Apparently travis is running Firefox 31 which doesn't support 'let'

add travis badge

add the CSS as a fixture - not sure if its needed for the tests but here it is

updated stand-alone jasmine to 2.5.2

try to run travis

cli unit test setup done

trying out karma-fixture

fix missing ) which caused a subtle syntax error