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.

Sign up for GitHub

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

Jump to bottom

Check supported value for dashed transition property #47

Closed
aChudinov opened this issue Jun 22, 2018 · 7 comments
Closed

Check supported value for dashed transition property #47

aChudinov opened this issue Jun 22, 2018 · 7 comments
Labels
bug

Comments

@aChudinov
Copy link
Contributor

When I invoke supportedValue('transition', 'max-height 100ms linear'), I have false-height 100ms linear as a result.
In general, this happens with any dashed transition property.
@kof kof added the bug label Jun 23, 2018
aChudinov pushed a commit to aChudinov/css-vendor that referenced this issue Jun 26, 2018
cssinjs#47: Fix check for supported dashed transition property value
c172711
kof added a commit that referenced this issue Jul 3, 2018
@kof
Merge pull request #49 from aChudinov/fix-dashed-transition-property
f793d28 
#47: Fix check for supported dashed transition property value
@AleshaOleg
Copy link
Member

#49 merged. @aChudinov thanks!

@aChudinov
Copy link
Contributor Author

@AleshaOleg Thank you 😃

@aChudinov
Copy link
Contributor Author

@AleshaOleg Will you take care of propagating these changes to jss-vendor-prefixer?

@AleshaOleg
Copy link
Member

@aChudinov right now preparing build. Have some problems with webpack@4 :)

@AleshaOleg
Copy link
Member

css-vendor@1.0.4 released

@AleshaOleg
Copy link
Member

@aChudinov wait, what you mean under changes to jss-vendor-prefixer?
https://github.com/cssinjs/jss-vendor-prefixer/blob/master/package.json#L40
This plugin will use the last version automatically. Do you mean just bump a version of css-vendor?
If yes, it's already published :)

@aChudinov
Copy link
Contributor Author

Cool, one more thank :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@kof @AleshaOleg @aChudinov