RFE: make scaling of imported bitmap images based on image resolution optional

[su_v]

Follow-up report to Bug #165952 “png resolution not imported”

Feature request:
Implement a global opt-out preference for this feature (-> allow to always import 1:1 as it used to be before the the fix for bug #165952). The default would be to import based on image resolution if present (current state), and in the preferences pane one could choose to opt out (no popup question on import, just a global setting).

Rationale:
The bonus of recent new feature (always respect an image resolution if present in the imported bitmap image and scale the image accordingly) might depend somewhat on drawing tasks and make most sense when importing high-res digital images.
For other tasks (e.g. when working with GUI mockups and icon previews), and for users who want to import all images 1:1 (one image pixel = 1 SVG user unit), this new feature simply has added a couple of extra steps (to undo the scaling), which can't be done on multiple imported images at once, nor otherwise automated.

Workaround:
A possible alternative - if adding yet another preference setting is considered evil - could be for example a new command (verb) to reset the scale of selected images to 1:1 (those users who need it could then map it to a custom keyboard shortcut).

[Daniel Wagenaar (dawagenaar)]

I've written a patch to implement this request. See bug #165952 comment #40.

[su_v (suv-lp)]

Merge request tested successfully with Inkscape 0.48+devel r11680 on OS X 10.7.4.

[jazzynico (jazzynico)]

Patch committed revision 11681.
Thanks Daniel!

[su_v (suv-lp)]

@Daniel - many thanks for your contributions! Would you be interested to join the Inkscape Developers team [1]? Since two of your patches have been accepted and committed now (in your case two branches have been merged), you can get direct access to the bzr repository to commit future fixes or new features yourself.

[ At some point in time, way back, the devs had agreed on a threshold of two bug fixes for such right. The top rule for trunk is that it must always be useable and in an unbroken state. Everyone on the dev team uses it for production work, so just keep this in mind. Also, as of today, 'chill' phase of the release plan for 0.49 is setting in [2]. ]

---
[1] 'Join the team' here: <https://launchpad.net/~inkscape.dev>
[2] <http://inkscape.13.n6.nabble.com/Inkscape-0-49-Proposed-Release-Plan-td4965052.html>

[Hans-Karl (jchbraun)]

This is a great features. BMP, TIFF and JPEG will be supported ?
( a-long-request-from-myself, from https://bugs.launchpad.net/inkscape/+bug/165952 )
Thanks !

[Hans-Karl (jchbraun)]

From https://skydrive.live.com/?cid=09706d11303fa52a&id=9706D11303FA52A%21217
R11870 (Windows build) imports ONLY png at the good resolution. Not jpg or bmp. Always 90 dpi for this types.
Patch seems incomplete.

[su_v (suv-lp)]

@Hans-Karl - please file a new report about your issues with support of image resolution in current Windows development snapshot builds. Possibly the current devlibs for the Windows port of Inkscape don't include some of the required dependencies to support the new feature from bug #165952.

Note: this request (bug #1052796) was about optionally _disabling_ the new feature, and works as expected.