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

Fixed hatch clipping. #2001

Closed
wants to merge 1 commit into from
Closed

Conversation

pelson
Copy link
Member

@pelson pelson commented May 14, 2013

No description provided.

@pelson
Copy link
Member Author

pelson commented May 14, 2013

Replaces #1898

@mdboom
Copy link
Member

mdboom commented May 14, 2013

👍 Seems like a pretty straightforward fix. Can you rebase this against v1.2.x, as I believe the bug exists there as well?

@pelson
Copy link
Member Author

pelson commented May 14, 2013

I've just done this locally, but the test results change (because of the color handling added recently on master).

For example with v1.2.x:

The svg rendering of one of the images in this PR looks like:

clip_path_clipping_svg

On this basis, I'm not sure its worth the extra overhead of putting it against v1.2.x.

Thoughts?

@mdboom
Copy link
Member

mdboom commented May 14, 2013

We can put different test images on each branch. I think better that than to have hatches continue to be broken. I'd like to put out a rock-solid 1.2.2 around the same time as 1.3.0...

@pelson pelson mentioned this pull request May 14, 2013
@pelson
Copy link
Member Author

pelson commented May 14, 2013

I'd like to put out a rock-solid 1.2.2 around the same time as 1.3.0...

I can't refuse a reasonable request 😄 - #2003

@pelson pelson closed this May 14, 2013
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.

None yet

2 participants