Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add multipass tests #1177

Closed
wants to merge 15 commits into from
Closed

Add multipass tests #1177

wants to merge 15 commits into from

Commits on Feb 14, 2021

  1. Add tests for multipass. 

    Fix multipassCounter code to make failed multipass tests pass.
    @strarsis
    strarsis committed Feb 14, 2021
    Copy the full SHA
    40384ef View commit details
    Browse the repository at this point in the history

  2. Add more tests for multipass. 

    Improve multipass related tests.
    @strarsis
    strarsis committed Feb 14, 2021
    Copy the full SHA
    3623fdd View commit details
    Browse the repository at this point in the history

  3. Fkx quotes in addAttributesToSVGElement plugin config.

    @strarsis
    strarsis committed Feb 14, 2021
    Copy the full SHA
    3916f93 View commit details
    Browse the repository at this point in the history

  4. Fix lint errors.

    @strarsis
    strarsis committed Feb 14, 2021
    Copy the full SHA
    d1274df View commit details
    Browse the repository at this point in the history

  5. Change wording of API doc comment

    @marcjansen @strarsis
    marcjansen authored and strarsis committed Feb 14, 2021
    Copy the full SHA
    26e66ed View commit details
    Browse the repository at this point in the history

  6. Fix collapsing of repeated commands in convertPathData plugin 

    This fixes svg#1166
    @jlherren @strarsis
    jlherren authored and strarsis committed Feb 14, 2021
    Copy the full SHA
    aef9825 View commit details
    Browse the repository at this point in the history

  7. Add test for collapsing of repeated commands

    @GreLI @strarsis
    GreLI authored and strarsis committed Feb 14, 2021
    Copy the full SHA
    f35e671 View commit details
    Browse the repository at this point in the history

  8. Fix prefixIds plugin error with animations 

    Fix svg#848
    @ssbb @strarsis
    ssbb authored and strarsis committed Feb 14, 2021
    Copy the full SHA
    7ef64b0 View commit details
    Browse the repository at this point in the history

  9. Correct test

    @GreLI @strarsis
    GreLI authored and strarsis committed Feb 14, 2021
    Copy the full SHA
    3934775 View commit details
    Browse the repository at this point in the history

  10. Perform cleanupIDs optimization when style/script tag is empty 

    cleanupIDs plugin is disabled when a script or style tag is present. I
assume that this is because they might contain an id.

If these tags are empty there is no point in not cleaning the id though.
When running svgo with the option `multipass: true`, it allows us to
first run `inlineStyles` plugin to empty the style tag. Then, we can run
`cleanupIDs` plugin.
    @dcyriller @strarsis
    dcyriller authored and strarsis committed Feb 14, 2021
    Copy the full SHA
    d759f46 View commit details
    Browse the repository at this point in the history

  11. Fix cleanupIDs if defs is last elem of svg

    @alexpavlovich @strarsis
    alexpavlovich authored and strarsis committed Feb 14, 2021
    Copy the full SHA
    9cf1f38 View commit details
    Browse the repository at this point in the history

  12. Add Figma plugin link to Readme

    @opudalo @strarsis
    opudalo authored and strarsis committed Feb 14, 2021
    Copy the full SHA
    ba47879 View commit details
    Browse the repository at this point in the history

  13. Remove unrelated comments.

    @strarsis
    strarsis committed Feb 14, 2021
    Copy the full SHA
    81f8761 View commit details
    Browse the repository at this point in the history

  14. Add test for specific config set for prefixIds with multipass.

    @strarsis
    strarsis committed Feb 14, 2021
    Copy the full SHA
    fb4dd6b View commit details
    Browse the repository at this point in the history

  15. Merge branch 'master' of https://github.com/svg/svgo into tests-multi… 

    …pass
    @strarsis
    strarsis committed Feb 14, 2021
    Copy the full SHA
    b43238b View commit details
    Browse the repository at this point in the history