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

added LTYPE table parser. Necessary to create patterns for dashed lines #120

Closed
wants to merge 2 commits into from

Conversation

ieskudero
Copy link
Contributor

LTYPE tables are used to get line properties, such as patterns to create dashed lines. This allows to use the lineTypeName property common to all entities. With this dxf viewers can create non continuous lines.

@skymakerolof
Copy link
Owner

Sounds like a good improvement feature-wise! The diff is huge however and from what I can see it is mostly formatting that has changed. Would it be possible to revert that so that the diff only includes implementation changes? Would also be nice with some test case that shows how it is supposed to work.

@ieskudero
Copy link
Contributor Author

My mistake, didn't realize that before uploading it. I will try to fix this, or if I can't I will upload another PR with a use case included.

@ieskudero ieskudero mentioned this pull request Nov 24, 2022
@ieskudero
Copy link
Contributor Author

@skymakerolof you can close this PR. #121 fixes it. By the way, can you upload a new version of the library?

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.

2 participants