jahnf / Projecteur

Linux Desktop Application for the Logitech Spotlight device (and similar devices) - Digital Laser Pointer
MIT License
379 stars 33 forks source link

Add settings to commands #44

Closed mlouielu closed 4 years ago

mlouielu commented 4 years ago

Hi there, first of all, thanks for this project, its awesome!

I'm using KDEConnect-android + KDEConnect, which provide a simple spotlight function in it. I've made a modification in KDEConnect to make it use Projecteur as the spotlight backend, which use projecteur -c spot=on/off to toggle the spotlight function.

It is nice to have all the settings can pass by the commands -c, so it is possible for others to config at runtime, I'll provide a prototype PR for this.

How do you folks think?

jahnf commented 4 years ago

Thanks, I guess settings via command options would make Projecteur more usable in combination with scripts. I'll have a look at the PR tomorrow.

jahnf commented 4 years ago

I wasn't aware of the presenter functionality in KDE Connect, interesting. Also I left some comments in #45 - let me know what you think.

jahnf commented 4 years ago

Setting of values is now available from the command line (not yet shown in the help - next commit...) But the issue with zoom still remains. If the zoom gets activated while the spot is active - no screenshot can be taken - so either there is an outdated screen image or no screen image (if the zoom never was activated since start of projecteur)

mlouielu commented 4 years ago

Nice works! Thanks for the commit!