Skip to content

Fix function nesting#27

Closed
vjeux wants to merge 1 commit intoshellscape:masterfrom
vjeux:fix_nesting
Closed

Fix function nesting#27
vjeux wants to merge 1 commit intoshellscape:masterfrom
vjeux:fix_nesting

Conversation

@vjeux
Copy link
Copy Markdown

@vjeux vjeux commented May 20, 2017

The parenthesis is not added at the right place because it's using a single value instead of a stack in order to backtrack.

Fixes #26

The parenthesis is not added at the right place because it's using a single value instead of a stack in order to backtrack.

Fixes shellscape#26
@shellscape
Copy link
Copy Markdown
Owner

The Pull Request Template is not optional. Please fill in the missing info.

I believe this branch https://github.com/shellscape/postcss-values-parser/tree/fix-nested-funcs fixes the issue your pull request addresses, please check that branch against your tests.

@shellscape
Copy link
Copy Markdown
Owner

Closing favor or f4a7896. Thanks for taking the time to open a PR.

@shellscape shellscape closed this May 26, 2017
@vjeux
Copy link
Copy Markdown
Author

vjeux commented May 26, 2017

Awesome, thanks for getting it fixed :)

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.

2 participants