overlay icon indicating copy to clipboard operation
overlay copied to clipboard

Error on initialization of Window class

Open MaxDerDeveloper opened this issue 4 years ago • 7 comments

I'm initializing the overlay.Window class and get this error:

Traceback (most recent call last): File "C:\Users\User\Desktop\test_overlay.py", line 3, in win = Window() File "C:\Users\User\AppData\Local\Programs\Python\Python38\lib\site-packages\overlay\window.py", line 42, in init self.transparent = kwargs.get('transparent', False) File "C:\Users\Nutzer\AppData\Local\Programs\Python\Python38\lib\site-packages\overlay\window.py", line 125, in transparent self.root.config(bg=bg) File "C:\Users\Nutzer\AppData\Local\Programs\Python\Python38\lib\tkinter_init.py", line 1637, in configure return self.configure('configure', cnf, kw) File "C:\Users\Nutzer\AppData\Local\Programs\Python\Python38\lib\tkinter_init.py", line 1627, in _configure self.tk.call(_flatten((self._w, cmd)) + self._options(cnf)) _tkinter.TclError: unknown color name "systemWindowBody"

MaxDerDeveloper avatar Jun 08 '20 05:06 MaxDerDeveloper

Hi. It seems like you are trying to run the module on Windows (or other operating systems other than Mac OS), which isn't supported by this module. There are other alternatives for creating a floating UI on Windows, such as using pywin32. Overlay was created due to the lack of such methods on Mac OS.

linxuanm avatar Jun 08 '20 08:06 linxuanm

Okay thanks dude

MaxDerDeveloper avatar Jun 08 '20 09:06 MaxDerDeveloper

You should not state that this works on Windows OS, because you said it wasn't tested, so now it is, and it doesn't work, so please remove the line that states that it works, wasn't tested. Good work, though!

elvirag avatar Jun 19 '20 14:06 elvirag

By "not tested" I meant that I am not sure if it would work. The platform section is a specification of the compatibility state of common platforms rather than a list of platforms on which this package functions.

However I do see your concern, and indeed, the current format is rather ambiguous. Thanks for the reminder.

linxuanm avatar Jun 20 '20 00:06 linxuanm

By "not tested" I meant that I am not sure if it would work. The platform section is a specification of the compatibility state of common platforms rather than a list of platforms on which this package functions.

However I do see your concern, and indeed, the current format is rather ambiguous. Thanks for the reminder.

Thanks, and thank you for the change, keep up the good work!

elvirag avatar Jun 21 '20 12:06 elvirag

Uh, surely this shouldn't be closed before it gets fixed?

Heath123 avatar Jun 22 '20 20:06 Heath123

Uh, surely this shouldn't be closed before it gets fixed?

It wasn't planned to be fixed, but now it is. I'll keep this open until the fix is released.

linxuanm avatar Jun 26 '20 05:06 linxuanm