-
Notifications
You must be signed in to change notification settings - Fork 62
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
new attributes for separate fill & stroke opacity #265
Conversation
Travis is failing because of the thing with |
I see it like this: |
db1f5c9 fixes the travis build re |
@cchalmers suggested simply not hiding |
@tkvogt but what if you set, say, both the |
That minor version bump is the reason I didn't take the unhide approach unilaterally. I just wanted the travis build to green-light. But I'm fine with that solution. |
@byorgey The SVG spec says that "object/group opacity can be thought of conceptually as a postprocessing operation" (http://www.w3.org/TR/SVG/masking.html#OpacityProperty) |
@tkvogt Thanks, that makes sense. Mostly I was trying to figure out whether e.g. setting |
In that case I think this is ready to merge. Updating the SVG backend to use these should be easy. Updating other backends to support them sounds much harder. |
new attributes for separate fill & stroke opacity
See #248.
Questions I'd like help thinking about: (1) what should be the semantics of the combination of opacity, fill-opacity, and stroke-opacity attributes? (2) what is the right way to upgrade backends to support these attributes?