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

Regular Expressions in Replace variable stopped working #964

Closed
greystate opened this Issue Jul 6, 2016 · 4 comments

Comments

Projects
None yet
2 participants
@greystate

greystate commented Jul 6, 2016

I'm using Avocode 2.8.4 on OS X 10.11.5 and I have an issue with "replace" variables:

I have one that looks like this:

avocode-replace

So input is: font-size: (\d+)px; and the replacement is: .font-size($1);, e.g. to convert font-size: 30px; into .font-size(30); (I'm using LESS, so I have a mixin called .font-size() to generate th font-size rules).

This has stopped working - replacements with exact matches and no RegEx rules seem to work fine.

@petrbrzek

This comment has been minimized.

Show comment
Hide comment
@petrbrzek

petrbrzek Jul 6, 2016

Member

@greystate Thanks for the report. I can reproduce it so it should be fixed in next release.

Member

petrbrzek commented Jul 6, 2016

@greystate Thanks for the report. I can reproduce it so it should be fixed in next release.

@petrbrzek petrbrzek added this to the 2.9.0 milestone Jul 6, 2016

@petrbrzek

This comment has been minimized.

Show comment
Hide comment
@petrbrzek

petrbrzek Aug 5, 2016

Member

Fixed in 2.9.0.

Member

petrbrzek commented Aug 5, 2016

Fixed in 2.9.0.

@petrbrzek petrbrzek closed this Aug 5, 2016

@greystate

This comment has been minimized.

Show comment
Hide comment
@greystate

greystate Nov 25, 2016

Beta 2.12.0-community-beta.1 has resurfaced this issue 😿

(I can open a new duplicate issue, if you want @petrbrzek ?)

greystate commented Nov 25, 2016

Beta 2.12.0-community-beta.1 has resurfaced this issue 😿

(I can open a new duplicate issue, if you want @petrbrzek ?)

@petrbrzek

This comment has been minimized.

Show comment
Hide comment
@petrbrzek

petrbrzek Nov 25, 2016

Member

@greystate thanks for noticing us. It will be fixed.

Member

petrbrzek commented Nov 25, 2016

@greystate thanks for noticing us. It will be fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment