Eel icon indicating copy to clipboard operation
Eel copied to clipboard

Fix JS memory leak + modernize `eel.js`

Open guillaume-mueller opened this issue 11 months ago • 0 comments

Seeing eel.js for the memory leak fix, I felt its readability could be improved, so I proceed.

I tested it against Python 3.10.16 on Linux. I had to modify conftest.py to use Chromium instead of Chrome. Maybe it would be nice to include such a modification to not force to install Chrome.

If we go ahead, I may want to refactor and modernize eel.js more for concision leading to better maintainability. Are there any rules of ordering classes elements you would like to favor ?

guillaume-mueller avatar Jan 28 '25 19:01 guillaume-mueller