GUI Plugin Scroll

[JamesHabben]

GUI Plugin Scroll

[Johann-PLW]

This is not something only related to the compiled GUI release on Windows platforms.
I have the same behavior on my Mac. It is a known bug of PySimpleGUI.
iLEAPP forces version 4.16.0 to be installed (PySimpleGUI==4.16.0 in requirements.txt file)
I have upgraded to latest version 4.60.5 and now I am able to scroll when my cursor is inside the plugin box.
But some changes occurred in the GUI
PySimpleGUI v4.16.0

PySimpleGUI v4.60.5

If we use v4.59.0, scroll works as expected and GUI stays the same:

[JamesHabben]

Oh, good find @Johann-PLW. I think the latest with the color changes look better as a proper UI.

[Johann-PLW]

And I was able to increase the size of output log box.

[stark4n6]

The newer version of PySimpleGUI looks slick, does it break anything you've found? I wonder if it's just a simple update

[Johann-PLW]

Everything works fine except:

• After selecting input file/folder and output folder, the plugin window loses the focus so we have to click inside the main window once before to be able to select/unselect modules
• Some adjustments in code to fix this PEP8 warning

[Johann-PLW]

I've just run a few more tests by :

• adjusting the size of output logs
• replacing FindElement with windows[key] or window.find_element as suggested in PEP8 warnings.

Just four lines of code updated in ileappGUI.py file and it works perfectly both with PySimpleGUI 4.0.16 (actual one used by iLEAPP) and the new 4.60.5 version.
So I think I can push this little modifications then we can decide whether we want to use the latest version of PySimpleGUI just by updating requirements.txt file or stay with 4.0.16
In the end, no action required by the user.

[abrignoni]

Based on your work I think we should move to the new version of PYsimpleGUI. Thank you!

…

On Sat, Dec 23, 2023, 1:33 PM Johann POLEWCZYK ***@***.***> wrote: I've just run a few more tests by : - adjusting the size of output logs - replacing FindElement with windows[key] and window.find_element as suggested in PEP8 warnings. Just four lines of code updated in ileappGUI.py file and it works perfectly both with PySimpleGUI 4.0.16 (actual one used by iLEAPP) and the new 4.60.5 version. So I think I can push this little modifications then we can decide whether we want to use the latest version of PySimpleGUI just by updating requirements.txt file or stay with 4.0.16 In the end, no action required by the user. — Reply to this email directly, view it on GitHub <#642 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AG3DPC7MN4ARRJ5G6J3XNRTYK4PZFAVCNFSM6AAAAABBAGQXOGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNRYGM2DOOJWG4> . You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>