simple-plotting icon indicating copy to clipboard operation
simple-plotting copied to clipboard

Adding option to display graph in Quick Look and avoid output to STDOUT

Open jolros opened this issue 11 years ago • 2 comments

Hi,

I thought this was a great idea for a utility, thanks! I wanted to avoid file cleanup and a manual call to "qlmanage" or "open" in my own process, so I added an option (-p) that runs Quicklook on a temporary file instead of printing to STDOUT. It's backwards compatible with the existing command usage.

No worries if you don't want this functionality, but it was useful for me.

(I deleted my initial pull request and added this one in order to include documentation in the README file)

Thanks again.

jolros avatar Dec 19 '13 19:12 jolros

It’s not a good idea to include calls to a specific program. That’s what the command line and stdout is for.

Profpatsch avatar Oct 23 '14 13:10 Profpatsch

Tip: all those lines of code can be replaced by:

#!/bin/bash

plot "$@" | qlmanage -p -

Profpatsch avatar Oct 23 '14 13:10 Profpatsch