GeopJr/Collision

Allow launching multiple instances

teackot opened this issue · 4 comments

Is your feature request related to a problem? Please describe.
I find this app useful to check if two files are identical, right now I have to copy two hashes into a text editor and compare them there.

Describe the solution you'd like
Allow opening multiple windows of the app.

Thanks for raising this issue!

Allowing multiple instances does sound interesting and I'll definitely get on it, however your use case seems to be already covered:

  1. Open the first file by clicking on the "Open a File" button in the welcome view
    Screenshot of Collision's welcome view

  2. Select the verify tab from the headbar:
    Screenshot of Collision's Hash view
    Screenshot of Collision's Verify view

  3. Click on the button under the title "File" and select the file you want to check against:
    Screenshot of Collision's Verify view, a file was selected under the file checking section that matches the main opened file
    Screenshot of Collision's Verify view, a file was selected under the file checking section that doesn't match the main opened file

If that is not your use case, please explain it below!
If that was your use case, it appears that the current design of the Verify view is not clear enough...

(just to explain the usage of the verify tab: they are tools to check against the current open file

the text filed checks if the input matches any of the hashes of foo.txt
the file button checks if a hash of the selected file matches any of the hashes of foo.txt)

Oh! I guess I'm just blind and didn't check out the verify tab in a hurry, otherwise it's pretty obvious what this tab does. Anyway, I think multiple instances is still a good thing to have

With the ability to open files directly from the command line, you are “theoretically” able to open multiple files with Collision.
But for now, the program just doesn't handle it correctly!
It seems that launching a new instance of Collision......merges the window content with the already-existing window! xD

First file Second file Sum of both
Capture d’écran du 2022-11-21 20-52-41 Capture d’écran du 2022-11-21 20-52-53 Capture d’écran du 2022-11-21 20-53-40

Nice catch! Thanks for the report!

It should be fixed as of #124