fsmosca / Python-Easy-Chess-GUI

A Chess GUI based from Python using PySimpleGUI and Python-Chess.
GNU Lesser General Public License v3.0
161 stars 52 forks source link

Thank you for writing this! #36

Closed PySimpleGUI closed 9 months ago

PySimpleGUI commented 4 years ago

Really appreciate you taking the time to make this game (and using PySimpleGUI in the process). I use it a lot as a successful example. The screenshots in your readme are awesome. Thanks for your hard work!

fsmosca commented 4 years ago

Thanks for your PySimpleGUI and the chess demo, without it, it would have been too long for me to get a working chess GUI in python.

I am currently looking on the issue of Linux support. I am using WSLinux on windows 10.

PySimpleGUI commented 4 years ago

I'll have to look, but I think that problem with disable is fixed in 4.15.

Also, Linux icons must be PNGs, not ICOs. Or maybe GIFs. I just recall that ICO file don't work with Linux. Maybe something I should document if it's not in there now.

Let me know any other problems you run into and you'll get priority support!

fsmosca commented 4 years ago

Thanks, png icons for linux are working.

Still there is issue in window.Disable() with PSG v4.15.2, both in Python 3.6 and 3.7 on win10 WSLinux Ubuntu 18.04.

For Python 3.7.6

Traceback (most recent call last):
  File "python_easy_chess_gui.py", line 3534, in <module>
    main()
  File "python_easy_chess_gui.py", line 3530, in main
    pecg.main_loop()
  File "python_easy_chess_gui.py", line 2827, in main_loop
    window.Disable()
  File "/home/ubu2019/.local/lib/python3.7/site-packages/PySimpleGUI/PySimpleGUI.py", line 6379, in Disable
    self.TKroot.attributes('-disabled', 1)
  File "/usr/lib/python3.7/tkinter/__init__.py", line 1788, in wm_attributes
    return self.tk.call(args)
_tkinter.TclError: bad attribute "-disabled": must be -alpha, -topmost, -zoomed, -fullscreen, or -type
fsmosca commented 1 year ago

At the moment, the issue on _tkinter.TclError: bad attribute "-disabled": must be -alpha, -topmost, -zoomed, -fullscreen, or -type is fixed. I use Hide()/UnHide instead of Disable/Enable. The disadvantage is that hiding the window will hide the GUI. But this is better than crashing the GUI in linux and perhaps mac.

fsmosca commented 1 year ago

Current display in windows 10.

image

WSL - Ubuntu 20.04

image

ahmadmousily commented 1 year ago

the window.disable() is not working for me in linux, can anyone point out possible alternatives for this thing ?

fsmosca commented 1 year ago

Just use the latest script. window.disable is now replaced by hide/unhide.

ahmadmousily commented 1 year ago

I know that hide/unhide works on links, but i don't want the first window to get hidden I want it to be there and disabled, as the user have to take some data from it.