Comment 81 for bug 1389723

Revision history for this message
Alvin Penner (apenner) wrote :

Hi,
 I tested the patch called v0.3 from comment 80. I tested it on the 0.91.x branch using essentially the same test as in the comment at:
http://inkscape.13.x6.nabble.com/Doc-scaling-compatibility-issue-Bug-1389723-td4976911.html#a4976997
The result of the test looks quite reasonable to me. I think this would be worth implementing. If you choose yes to both options then you get a result that is similar to what you would get if you import a pdf file. For a pdf import the import layer has a scale factor of 90/72 caused by a unit change from pt to px. For the current dpiswitcher you get a similar scale factor caused by a dpi change. If you want to do some more editing using the new "96 dpi" units, then you can create a new layer and draw on the new layer in the new 96 dpi units and the result will be represented in the XML file as expected.
 I have only two very minor comments, both of them purely cosmetic:
First of all the user has four possible options coming from two separate questions, namely "viewbox or no" and "scale or no". Of these four options only about 3 or so seem reasonable to me. I can't think of any situation where the user might want to say yes to viewbox and no to scaling. I wonder if it would be possible to ask the scaling question first, since it is the most important one, and then, only if the user says yes, then ask the viewbox question next?
 Secondly I see that the rev number is showing up in the DOS output. I wonder if it would be worthwhile to also add it in the menu item Extensions->Document->DOC Info. It could be added with something that looks like this:
print "version: " + str(self.document.getroot().get(inkex.addNS('version',u'inkscape')))

 Finally it would be really helpful to get feedback from as many other people as possible, since this is likely to be a very serious issue in the new release.
 And thanks for your hard work, Jabier!