-
Notifications
You must be signed in to change notification settings - Fork 26
Handle case if import with brackets is passed #59
Conversation
@Semigradsky @alexgagnon can you please take a look? |
…a into postcss-master
Updates 22 11 2021
lib/media-ast-from-string.js
Outdated
* remove spaces after left and before right bracket if spaces exist | ||
*/ | ||
function trimNearBrackets(string) { | ||
return string.replace('( ', '(').replace(' )', ')') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What if it will be two or more spaces?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
good point!
@alex-knyazev I guess we need trim all spaces
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Semigradsky fixed, please check, thanks
handle multiple spaces
@jonathantneal can you please take a look? |
@Semigradsky can you please take a look? |
Someday the pull request may get merged. Until then we use the fork. csstools/postcss-custom-media#59
Someday the pull request may get merged. Until then we use the fork. csstools/postcss-custom-media#59
Hi @alex-knyazev, @the-stas, @lipemat, This plugin was moved to the csstools mono repo : https://github.com/csstools/postcss-plugins/tree/main/plugins/postcss-custom-media#readme This PR was a great start but we wanted to handle a few more edge cases. |
Fix issues with whitespace in `@media( --var )` which previously required a fork. csstools/postcss-custom-media#59 (comment)
Fixes issues with whitespace in `@media( --var )` which previously required a fork. csstools/postcss-custom-media#59 (comment)
No description provided.