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

Svg props #172

Merged
merged 4 commits into from
Sep 16, 2018
Merged

Svg props #172

merged 4 commits into from
Sep 16, 2018

Conversation

gregberge
Copy link
Owner

No description provided.

TrySound and others added 4 commits September 15, 2018 18:42
In this diff I introduce the new option `svgProps` which is similar to
svgAttributes but applies values in interpolation style `{true}`.

This allows to specify expressions depending on template values for
example default `currentColor` in `fill` prop.

I mentioned in comment about later fixing api to smooth-code/h2x#13
@gregberge gregberge self-assigned this Sep 16, 2018
@TrySound
Copy link
Contributor

Do you know you could push commits to my branch directly?

@gregberge
Copy link
Owner Author

@TrySound I didn't know! sorry

@gregberge
Copy link
Owner Author

@TrySound OK for you? I will merge it.

@TrySound
Copy link
Contributor

Sure, looks good to me. Waiting for release :)

@gregberge gregberge merged commit 9657110 into master Sep 16, 2018
@gregberge gregberge deleted the svg-props branch September 16, 2018 14:46
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