Discussion:
[Kst-plot/kst] 38ef82: Add --pngHeight and --pngWidth to the command line...
Barth Netterfield
2016-08-20 19:00:12 UTC
Permalink
Branch: refs/heads/master
Home: https://github.com/Kst-plot/kst
Commit: 38ef82ee5d3235403ef136037c1b58f6ec4815a8
https://github.com/Kst-plot/kst/commit/38ef82ee5d3235403ef136037c1b58f6ec4815a8
Author: Barth Netterfield <***@astro.utoronto.ca>
Date: 2016-08-19 (Fri, 19 Aug 2016)

Changed paths:
M src/libkstapp/commandlineparser.cpp
M src/libkstapp/commandlineparser.h
M src/libkstapp/mainwindow.cpp

Log Message:
-----------
Add --pngHeight and --pngWidth to the command line.

Now you can set the size of the output images.
Ben Lewis
2016-08-21 04:59:32 UTC
Permalink
Hi Barth,

The --pngHeight and --pngWidth options work well. Thanks for pushing this feature request through so
quickly! It will be getting a good workout over the next few days.

I just have one small suggestion. If only one size option is used (height OR width) I would expect
the aspect ratio to be maintained. The current behaviour is to use the default size for whichever
dimension is omitted. This is not particularly useful as the same thing can be achieve by specifying
both dimensions. This is just a "nice to have" as I do not have a need for this functionality at the
moment.

Regards, Ben
Post by Barth Netterfield
Branch: refs/heads/master
Home: https://github.com/Kst-plot/kst
Commit: 38ef82ee5d3235403ef136037c1b58f6ec4815a8
https://github.com/Kst-plot/kst/commit/38ef82ee5d3235403ef136037c1b58f6ec4815a8
Date: 2016-08-19 (Fri, 19 Aug 2016)
M src/libkstapp/commandlineparser.cpp
M src/libkstapp/commandlineparser.h
M src/libkstapp/mainwindow.cpp
-----------
Add --pngHeight and --pngWidth to the command line.
Now you can set the size of the output images.
Loading...