Skip to content

Fix dxf import's colors, support BYLAYER

What does the merge request do?

Fix some colors in the dxf import, including a common one (using the parent layer's color). This still does not support BYLAYER/BYBLOCK for inserted blocks.

Implementation notes

Removed accidentally duplicated colors in table (which messed up index)

I looked over the test file differences by hand, and compared to LibreCAD.

I haven't touched the get_rgbcolor call in mtext_ctrl, that may need later correction to support bylayer? but I don't any test cases that hit that line (not sure how to produce this text in LibreCAD).

Fix #377 (closed)

Summary for release notes

Correct some colors in dxf import.

Checklist

  • Add unit tests (if applicable)
  • Changes to inkex/ are well documented (No changes)
  • Clean merge request history (only 1 commit)
Edited by Nathan Lee

Merge request reports