BaGoMa icon indicating copy to clipboard operation
BaGoMa copied to clipboard

Added working GUI for Python 2.6+ versions (<Python3)

Open ekianjo opened this issue 11 years ago • 5 comments

It needs PyZenity as a dependency, but that's free to distribute anyway.

ekianjo avatar Dec 15 '13 17:12 ekianjo

I tried testing this but http://www.brianramos.com/ (the source for PyZenity) seems to be down.

gburca avatar Dec 17 '13 15:12 gburca

You can find it on the python packages website as well.

On Wednesday, December 18, 2013, Gabriel Burca wrote:

I tried testing this but http://www.brianramos.com/ (the source for PyZenity) seems to be down.

— Reply to this email directly or view it on GitHubhttps://github.com/gburca/BaGoMa/pull/7#issuecomment-30761592 .

Raphael

ekianjo avatar Dec 17 '13 16:12 ekianjo

Here it is on pypi : https://pypi.python.org/pypi/PyZenity/0.1.4 - it's not the latest version but it should work nonetheless.

ekianjo avatar Dec 18 '13 08:12 ekianjo

I was looking to pull this in, but there's a few things that need to be fixed first:

  • The indentation is inconsistent (some lines use tabs, others spaces) and needs to match the rest of the code (4-spaces per level)
  • There's no handling for the "Cancel" buttons.
  • The user is forced to provide a password. The password dialog should only show up when/if the underlying script prompts for a password. If the user has already provided a password (through the config file, which is more secure), bagoma.py will not prompt again for a password, and neither should the GUI.
  • Any command line arguments should be passed on to the underlying script instead of being discarded.
  • If the user provided the email on the command line, there's no point in prompting for it again.
  • Executing "python bagoma.py ..." fails when in a virtualenv.
  • The underlying script output is not shown in the GUI.

gburca avatar Feb 10 '14 01:02 gburca

Thanks for the comments, I will be looking at this in more details and fix my code.

ekianjo avatar May 13 '14 04:05 ekianjo