Skip to content

Add strokeOpacity to unitless CSS properties #2482

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

Merged
merged 1 commit into from
Nov 8, 2014

Conversation

BinaryMuse
Copy link

No description provided.

@@ -26,6 +26,7 @@ var isUnitlessNumber = {
opacity: true,
order: true,
orphans: true,
strokeOpacity: true,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yea, we should do this. Can you actually add it last in the list and put a comment mentioning that it's for SVG?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We already have fillOpacity in the list.

@BinaryMuse BinaryMuse force-pushed the bkt/fix-stroke-opacity branch from ec2b721 to 96126e9 Compare November 8, 2014 05:20
@BinaryMuse
Copy link
Author

@zpao I've moved both the SVG-related properties to the bottom with a comment.

sophiebits added a commit that referenced this pull request Nov 8, 2014
Add strokeOpacity to unitless CSS properties
@sophiebits sophiebits merged commit 7a162ce into facebook:master Nov 8, 2014
@sophiebits
Copy link
Collaborator

Thank you!

@zpao zpao added this to the 0.12.2 milestone Dec 10, 2014
zpao pushed a commit to zpao/react that referenced this pull request Dec 18, 2014
Add strokeOpacity to unitless CSS properties
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