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

support inline-flex. fixes #107 #109

Merged
merged 2 commits into from
Jun 2, 2017
Merged

support inline-flex. fixes #107 #109

merged 2 commits into from
Jun 2, 2017

Conversation

idanen
Copy link

@idanen idanen commented May 7, 2017

No description provided.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling ac33f3f on idanen:master into 7820c71 on 7rulnik:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling ac33f3f on idanen:master into 7820c71 on 7rulnik:master.

@7rulnik
Copy link
Owner

7rulnik commented May 9, 2017

@idanen hey!
Nice catch, thank you. I will merge it tomorrow

@idanen
Copy link
Author

idanen commented May 9, 2017

Added small change to prevent adding -js-display when the rule has browser prefix like display: -webkit-flex;.

Now just adding for flex and inline-flex instead of everything with "flex"

@idanen
Copy link
Author

idanen commented May 9, 2017

Hey @7rulnik , any idea why the coverage check doesn't finish?

@idanen
Copy link
Author

idanen commented May 23, 2017

Hi @7rulnik, do you have an ETA for when will you merge this?

Thanks

@7rulnik
Copy link
Owner

7rulnik commented Jun 2, 2017

@idanen sorry, I was on vacation. Want to release it today.

@7rulnik 7rulnik merged commit 59d95e7 into 7rulnik:master Jun 2, 2017
@7rulnik
Copy link
Owner

7rulnik commented Jun 5, 2017

@idanen just released in v.2.0.0

@idanen
Copy link
Author

idanen commented Jun 5, 2017

Awesome! Thanks for the update 👌

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.

3 participants