Skip to content

Fix: #651 'Providing entities in SvgDocument.Open does not work' issue. #652

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

Merged
merged 5 commits into from
Jan 31, 2020

Conversation

H1Gdev
Copy link
Contributor

@H1Gdev H1Gdev commented Jan 29, 2020

Reference Issue

Fixes #651.
Ref. #649 and #90.

What does this implement/fix? Explain your changes.

Any other comments?

Copy link
Member

@mrbean-bremen mrbean-bremen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice easy fix! Could you add a regression test for this (may just use the example itself for that)?

@H1Gdev
Copy link
Contributor Author

H1Gdev commented Jan 29, 2020

@mrbean-bremen

may just use the example itself for that

Is it possible to run Entities in regression test ?

@mrbean-bremen
Copy link
Member

Is it possible to run Entities in regression test ?

I meant not directly, but just read the file from the sample and use the same open call (without saving it back). You can add a test to check the elements that use the elements to have the correct style.

@H1Gdev H1Gdev force-pushed the entities branch 2 times, most recently from a553518 to f9324e8 Compare January 31, 2020 03:50
Copy link
Member

@mrbean-bremen mrbean-bremen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@mrbean-bremen mrbean-bremen merged commit d4d5483 into svg-net:master Jan 31, 2020
@H1Gdev H1Gdev deleted the entities branch January 31, 2020 06:36
@H1Gdev H1Gdev mentioned this pull request Feb 2, 2020
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.

Providing entities in SvgDocument.Open does not work
2 participants