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

fix svg fill problem #2151

Closed
wants to merge 1 commit into from
Closed

fix svg fill problem #2151

wants to merge 1 commit into from

Conversation

zetaab
Copy link

@zetaab zetaab commented Feb 22, 2020

Summary

This PR fixes/implements the following bugs/features

Test plan (required)

test codes are available in issue #1629 with this fix I get it working normally.

Demonstrate how the issue/feature can be replicated. For most cases, simply adding an appropriate html/css template into the reftests should be sufficient. Please see other tests there for reference.

Code formatting

Please make sure that code adheres to the project code formatting. Running npm run format will automatically format your code correctly.

Closing issues

Fixes #1629

@zetaab
Copy link
Author

zetaab commented Feb 22, 2020

Before:
screen shot 2018-10-20 at 10 28 55

After:
screen shot 2018-10-20 at 10 29 03

much better

@ferrinweb
Copy link

Urgently needed, is @niklasvh planning to merge?

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.

svg not rendered correctly
2 participants