Comment 16 for bug 1239682

Revision history for this message
su_v (suv-lp) wrote :

<opinion>
In my understanding, the argument made in comment 14 is flawed - it would be quite safe to assume that any Inkscape SVG without a viewBox attribute has a 'px'-based drawing scale (based on 90dpi) - the display setting for the GUI never changed the drawing scale in a stable release version. r13938 appears to deliberately introduce a breaking change which
- prevents further editing of many old documents which were last saved in Inkscape <= 0.48 with "Default units" other than 'px' for the GUI
- prevents correct adaptation of such files to the internal resolution change (90dpi -> 96dpi) later on
  (once a solution for that is implemented - the lack of such a procedure is a release blocker for 0.92)
- prevents latest trunk being used for bug triage of reports with Inkscape SVG files from Inkscape <= 0.48 as test cases attached

I do hope that this change will be reverted soon (before it starts hurting users of development snapshot builds - files modified due to the changes in r13938 will be broken for further editing in Inkscape 0.48 and 0.91 too, once they had been opened and resaved with a devel build).

Tips for anyone affected by r13938:
* preventing incorrect scaling of old documents without viewBox attribute in trunk rev >= 13938: before opening such files in inkscape trunk rev >= 13938, open them in a plain-text editor instead and make sure that the attribute 'inkscape:document-units' is set to 'px'.
* fixing files resaved with a such modified drawing scale: open in a plain-text editor, delete the added viewBox attribute, change the attribute 'inkscape:document-units' to 'px' and hope that the enforced rescaling didn't already trigger other changes which can't be that easily undone.
</opinion>