Scaling inkview (feature request, patch included)

Bug #1550897 reported by Nicolas Brodu
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Inkscape
Fix Released
Wishlist
Nicolas Brodu
Inkscape-docs
Fix Committed
Low
Hachmann

Bug Description

Hi all,

and thanks for your collective efforts on this fantastic project.

I recently needed to add a scale option to Inkview, so as to change the default window size. For example, for a SVG file with units expressed in mm, the default window is way too small when computed at 90dpi with a screen resolution much larger. Other use cases can be when the SVG viewport, in pixels, is too large to fit on screen. A scale option addresses both problems, so I implemented it.

Attached is the patch that adds the scale option. It's very small, but quite useful.

Best,
Nicolas

Revision history for this message
Nicolas Brodu (g-ubuntuone-c) wrote :
jazzynico (jazzynico)
Changed in inkscape:
assignee: nobody → Nicolas Brodu (g-ubuntuone-c)
importance: Undecided → Wishlist
milestone: none → 0.92
status: New → In Progress
tags: added: inkview preferences
Revision history for this message
jazzynico (jazzynico) wrote :

Thanks for your patch!

We recently dropped getopt in our development version, and thus your modification had to be adapted. The attached patch was created and tested on Xubuntu 15.10 with rev. 14674. I also added a minimal help documentation for the new option.

Revision history for this message
jazzynico (jazzynico) wrote :

A bit too late for 0.92, linking to 0.93.

Changed in inkscape:
milestone: 0.92 → 0.93
Changed in inkscape-docs:
status: New → Triaged
importance: Undecided → Low
assignee: nobody → jazzynico (jazzynico)
tags: added: documentation man
Revision history for this message
jazzynico (jazzynico) wrote :

Adding the inkscape-docs project (inkview man page needs to be updated).

Changed in inkscape-docs:
milestone: none → 0.93
Revision history for this message
Patrick Storz (ede123) wrote :
Changed in inkscape:
status: In Progress → Fix Committed
Revision history for this message
Hachmann (marenhachmann) wrote :

There was still the man page update missing. The option that needed to be added to
https://gitlab.com/inkscape/inkscape/blob/master/man/inkview.pod.in
is:

short_name('s');
long_name("scale");
set_arg_description(N_("NUM"));
set_description(N_("Scale image by factor NUM"));

See https://gitlab.com/inkscape/inkscape/merge_requests/207

Revision history for this message
Hachmann (marenhachmann) wrote :

@Eduard: Do you think anyone would care about Inkview man page translations?...

Revision history for this message
Patrick Storz (ede123) wrote :

Yes, thought about it, too...
In principle it should not be too hard to add - if we have interested translators I don't see a reason why not to do translate it.

Revision history for this message
Hachmann (marenhachmann) wrote :

Created a report, then we can't forget to do it when we want to.

Revision history for this message
Patrick Storz (ede123) wrote :
Changed in inkscape-docs:
status: Triaged → Fix Committed
assignee: jazzynico (jazzynico) → Hachmann (marenhachmann)
Max Gaukler (mgmax)
Changed in inkscape:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.