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.

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

Allow for URL query string params #10

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

backflip
Copy link

This change allows the use of file.svg?--color=blue as an alternative to file.svg param(--color blue).

The main advantage would be valid CSS syntax. I'm currently building a pattern library and I want the consumers of the source to be able to use the styles without this plugin (in case they do not have a PostCSS step ready).

This should fix #8, too.

@backflip
Copy link
Author

I successfully ran the tests on the following local Node versions:

  • 8.17.0
  • 10.24.1
  • 12.22.12
  • 14.21.3
  • 16.20.2
  • 18.18.0
  • 20.8.0

I was unable to either install or run the project on Node 6.17.1. Installing fails with ENOTDIR: not a directory, open 'postcss-svg/node_modules/.staging/@types/q-01acf8e4/package.json' and running it after installing it on Node 8 fails with code ELIFECYCLE [email protected] test:tape: 'postcss-tape'.

@backflip
Copy link
Author

backflip commented Oct 10, 2023

Hm, I think this leads to issues with relative paths. Let me investigate.

UPDATE: Fixed an squashed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

SCSS/SASS support
1 participant