-
Notifications
You must be signed in to change notification settings - Fork 47.5k
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 SVG image, other related cleanup #4157
Conversation
@@ -157,6 +157,7 @@ var ReactDOM = mapObject({ | |||
defs: 'defs', | |||
ellipse: 'ellipse', | |||
g: 'g', | |||
iamge: 'image', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
iamge
is a typo.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bah, I caught it and fixed it but forgot to save the file :(
Sure. |
Support SVG image, other related cleanup
Great. Thanks |
When will be this available in npm? |
@abhilashsajeev This is slated for the 0.14 beta release, which is coming very soon. If you're anxious to try it out, you can always check out the source code and do a grunt build. |
@jimfb Okey Jim. What is the expected date of release for 0.14 ? I need to use it in production. |
@abhilashsajeev If you need it urgently, you can always patch your local copy of React to include this change. 0.14 final will probably be released sometime in July. |
@spicyj July first week is fine. If I patch my existing version to production will there be any problem like breaking of stable parts etc? |
No, not July first week. Sometime in July. |
Oh.. Then I should probably go with patch. There was some problem with using d3 also with react. |
I got tired of dealing with PRs trying to add image. Then I decided to cleanup while I was there since we've had to do a few followup changes when changes in getMarkupWrap weren't made in both places.