Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Simple GUI with Zenity / YAD #6

Open
shvchk opened this issue Feb 25, 2019 · 2 comments · May be fixed by #7
Open

Simple GUI with Zenity / YAD #6

shvchk opened this issue Feb 25, 2019 · 2 comments · May be fixed by #7

Comments

@shvchk
Copy link

shvchk commented Feb 25, 2019

Would be nice to be able to just double click AppImage, answer a few questions (device, iso file location, mode) and… done! 😃

Might be useful to add two new arguments to force corresponding mode (e.g. --cli and --gui).

if [[ $DISPLAY ]] could be used to select default mode (or maybe build separate AppImage defaulting to GUI mode).

App can make sure it's ok to output to stdout with if [ -t 1 ], and use log file if not (e.g. app is launched via double click or cron instead of terminal).

I might come up with a PR when I have some time. Would it be better to do GUI as a separate script (kind of wrapper) or part of the main script?

@ValdikSS
Copy link
Owner

ValdikSS commented Mar 8, 2019

I might come up with a PR when I have some time

Sure, that would be great.

Would it be better to do GUI as a separate script (kind of wrapper) or part of the main script?

I think the separate script is better, but either methods work.

@shvchk shvchk linked a pull request May 29, 2019 that will close this issue
@recolic
Copy link
Contributor

recolic commented Nov 25, 2021

I don't think it's a good design to mix core logic and user-interaction in the same tool... In my opinion, I think this script is designed to be a simple tool, to do one thing (pour windows to usb), and it is doing that thing very very well.
https://en.wikipedia.org/wiki/Unix_philosophy

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants