LogViewer can get the About popup stuck and blocking other features

[Porkepix]

I discovered this accidentally, but while the files manager to open a log is popped (cmd + o), user can still open the about popup, whether by an accidental cmd + h shortcut, or through the menu. At that point, closing the About popup might sometimes work, sometimes not.

But if the user finish to select and open a log file while the About popup is still opened, it seems to completely stuck the Log Viewer: About popup can't be closed anymore, and log seems to not load at all.

All the user can do is closing the Log Viewer completely as it's now unusable.

[tmolitor-stud-tu]

Does this happen with the feature/MLV-QListViewModels branch, too?

[Porkepix]

Is there built binaries somewhere?

[tmolitor-stud-tu]

No, you'll have to clone the repository and start the src/LogViewer.py by hand.

[tmolitor-stud-tu]

Don't forget to switch to the right branch before starting.

[Porkepix]

Ah my bad, I never expected it to be python and thought there was a need for a whole mac building/compiling toolchain (and knowledge of how does it work).

So, issue isn't solved in the branch but curiously if run from the branch window start much smaller than otherwise.

Also, the branch currently have a missing dependency in the requirements.txt, user needs to manually install pycryptodome or pycryptodomex on top of the requirements.txt as it's crashing for failing to find the module otherwise.

[Sobottasgithub]

Hello!
I tested it on the feature/MLV-QListViewModels branch and it's true that when clicking cmd + o and cmd + h at the same time opens both the about popup and the file manager, and the about popup becomes disabled. But the about popup should be closable after closing the file manager. When handling large files the LogViewer on the main branch, tends to freeze until the file is loaded, this isn't a bug tho, we are currently working on fixing this.

To start the src/LogViewer.py by hand you need to pip install the missing requirements that are listed (pycryptodome or pycryptodomex). :)

[Porkepix]

Hello! I tested it on the feature/MLV-QListViewModels branch and it's true that when clicking cmd + o and cmd + h at the same time opens both the about popup and the file manager, and the about popup becomes disabled. But the about popup should be closable after closing the file manager. When handling large files the LogViewer on the main branch, tends to freeze until the file is loaded, this isn't a bug tho, we are currently working on fixing this.

So, waiting to test a future version for now?

To start the src/LogViewer.py by hand you need to pip install the missing requirements that are listed (pycryptodome or pycryptodomex). :)

But then why isn't either of those part of the requirements.txt?