Inkview: multiple issues

[Hachmann]

Linux Mint 18.2 with Inkview 0.92+devel (914e21f, 2017-12-02)

1. no version info available, had to open Inkscape to be able to copy the info here - expecting: inkview -v or inkview --version to work.

2. Window is not resizable (and its contents is huge, longer than my screen) - a scrollbar would be helpful. (edited)

3. Tried downsizing, but -s doesn't scale anything aside from the window - between ~ 25 and 95% of my screen width with a tutorial svg. Values smaller 1 result in the same size as 1, larger values make the window wider, until it goes to 4, there it stops. (edited)

<removed 4.>

5. When running Inkview like this
$/opt/inkscape_master/bin/inkview -r -f /media/myfolder/Inkscape/Branches/documentation/export-inkscape/*.svg

I get memory leak warnings:

** (inkview:21716): WARNING **: Fonts dir '/home/maren/.config/inkscape_master/fonts' does not exist and will be ignored.

** (inkview:21716): WARNING **: GC Warning: Repeated allocation of very large block (appr. size 49152):
        May lead to memory leak and poor performance.

** (inkview:21716): WARNING **: GC Warning: Repeated allocation of very large block (appr. size 49152):
        May lead to memory leak and poor performance.

(Ctrl+C, because the only thing happening is my CPU load going up, and my computer fan starting hum loudly)

Does it try to load all files before starting the slideshow?...

(I did take care to set the environment variables and PATH correctly, so it finds the corresponding master inkscape)

[Patrick Storz (ede123)]

Let's see if we can address this properly in this bug (usually it's much better to file one issue per bug, otherwise some will fall into oblivion and the bug will linger forever):

1. is more of a feature request (Inkview never had that functionality)
   I quickly thoughr about adding it a while back but was not sure
   if it might be confusing (inkview does not really have a version
   number of its own but simply uses whatever Inkscape uses, i.e. one
   could just as easy query "inkscape -V")

2. On Windows the window is never larger than the screen.
   One can make the window larger but not smaller.

3. -s only takes integers, no floats, so 0.5 is not an option
   In the code it's defined as double (so floats should work)
   I can confirm the window does not shrink below the default
   without scale option though.

4. -t doesn't seem to be parsed correctly [...] -t=3
   That's a user error ;-)
   The syntax is either "-t 3" or "--timer=3"

5. Inkview does in fact load all files specified on the command line
   before launching. I think this was the default behavior since forever
   which can be good for slideshows (loading the next page will be faster)
   but I disliked it for most applications, too.
   I intended to rework this at some point (so "preloading" can be an
   option) but it wasn't straightforward to implement, so I decided
   to postpone that feature when I worked on Inkview last.

   The errors you are seeing might be related to that, but I suspect
   it might be indicating and underlying issue (for example I noticed
   "inkview share/tutorials" would crash while the individual files open
   fine.

   With "tutorial-shapes.de.svg" I just tried debugging with gdb and
   it seems to lock-up in a strange repaint loop as I get >16K entries
   that look like the following

#1 0x0000000002e1d0ba in SPCanvas::paintRectInternal(PaintRectSetup const*, Geom::GenericRect<int> const&) ()
   from E:\Temp\Inkscape\inkscape.git\master\build64\inkscape\libinkscape_base.dll

[Patrick Storz (ede123)]

Quick note on 2:
What I described (window can not grow large than the screen) is actually the desired behavior, see [1]. If it grows larger for you that seems to be a bug - as we "do everything correct" as far as I see it might be yet another gtk3 bug though (content scaling in gtk3 is a mess - most of the stuff does not work at all for practical purposes and upstream won't acknowledge it).

Also in that line I *guess* it should allow to scale down but I'll have to debug it properly first before speculating anymore...

https://gitlab.com/inkscape/inkscape/blob/914e21feed1e2f9981455b5a527b269f325188f6/src/svg-view-slideshow.cpp#L61-62

[Hachmann (marenhachmann)]

Sorry for the low quality report...

Glad that timer thing was a user error :)

For the scaling, it was the error message that told me it only accepts integers. Partially user error here, too, because I used the same syntax here.

However, now with the correct syntax, the scale of the SVG does not vary at all. The only thing that changes is the width of the window (and possibly, its height, but I cannot see that). This is for both values like 0.1 or 0.5 (but doesn't go below about 25% of screen, which is the default for no scale given) as well as 1 or 2 or 3 (gets wider, but not wider than screen, stops at about 95% of width...).

Only loads the header background of the selected tutorial when I hit the >| button.

I'll fix the report up a bit. Sorry again!