New extension 'Deep Ungroup' includes hard-coded unit conversion based on 90dpi
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Inkscape |
Fix Released
|
Medium
|
jazzynico |
Bug Description
The recently added extension 'Deep Ungroup' (r14926) includes the method '_get_dimension()' (copied from the original Synfig export extension, bundled with Inkscape since 0.91) which hard-codes unit conversion factors based on Inkscape's legacy internal resolution of 90dpi. Likely this should be converted into a more flexible method which retrieves current values somehow via inkex's Effect() class; or at least updated to defaulting to 96dpi.
The same code in synfig_prepare.py (referenced in the comments) was adjusted to the new internal resolution (while keeping the values hard-coded) in rev 13659:
https:/
Please note: I did not investigate which use cases could be affected directly by the assumption of legacy 90dpi.
Related branches
Changed in inkscape: | |
assignee: | nobody → jazzynico (jazzynico) |
importance: | Undecided → Medium |
milestone: | none → 0.92 |
status: | New → Triaged |
Fixed revision 14998.
I've applied the same change that was used in the Synfig extension for now. A better fix should be investigated to 0.93.