Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Double click to open a EPUB on Windows? #2163

Open
seaspeak opened this issue May 11, 2024 · 14 comments
Open

Double click to open a EPUB on Windows? #2163

seaspeak opened this issue May 11, 2024 · 14 comments

Comments

@seaspeak
Copy link

Windows or Store version Thorium.Setup.2.4.1 won't open the clicked EPUB.
I am always using Thorium.Setup.2.3.0-alpha.1.3658607447.exe, It is the only version I konw, could open the double-clicked EPUB.

@panaC
Copy link
Member

panaC commented May 14, 2024

Isn't a registry issue ? could you clear your registry epub file association, to let the window store make the association again.
I'm on mac, not a windows user

@seaspeak
Copy link
Author

seaspeak commented May 14, 2024

Not an association problem, I guess. 2.4.1 brings up the main window (as below) and even not import the book/epub and not open the epub.

image

@panaC
Copy link
Member

panaC commented May 14, 2024

You cannot import an epub and open it in Thorium 2.4.1 at all ?

@seaspeak
Copy link
Author

seaspeak commented May 14, 2024

If I double-click a new epub, version Thorium.2.3.0-alpha.1.3658607447 will import the new epub and open it in a seperated window. But 2.4 just open the main window, and doesn't even import the new epub.

And yes, I can import a new epub by drag&drop or add in the main window.

@panaC panaC self-assigned this May 14, 2024
@panaC
Copy link
Member

panaC commented May 14, 2024

Ok i will check on my side

@danielweck
Copy link
Member

We will not be able to debug/troubleshoot this from Thorium APPX deployed to the Microsoft Windows Store, but hopefully we will be able to reproduce with the NSIS executable which installs Thorium for either all users or the current local one (i cannot remember what the Electron Builder does)

@panaC
Copy link
Member

panaC commented May 16, 2024

It works on my windows 11 pro machine.
I remove by security any application associated with epub file association in Registry Editor.
I tested both with a local Thorium build npm run prepackage:win && npm run package:win and Official Github Release 2.4.1 (https://github.com/edrlab/thorium-reader/releases/download/v2.4.1/Thorium.Setup.2.4.1.exe)

File association (double click, or open with) works with Thorium already open or not.

I should be abble to reproduce to help you.
I keep the issue open

@danielweck
Copy link
Member

Thank you for testing @panaC
You don't have to manually invoke the "pre" (or "post") build scripts, they are automatically resolved by NPM when calling npm run package:win

@danielweck
Copy link
Member

So could the problem experienced by some users be limited to the Windows Store version? Or maybe the NSIS executable installer is affected when users install the Windows Store version in parallel too?

@seaspeak
Copy link
Author

I should be abble to reproduce to help you. I keep the issue open

I am sorry, I am confused. Can you reproduce the problem or you don't have the problem on your site?

I tested the Official Github Release 2.4.1 in Windows 10 Sandbox, so that I don't have to worry about cleaning the registry or user folders. I got the same result.

Video_2024-05.mp4

@danielweck
Copy link
Member

just out of interest, are you able to test with Windows 11?

@seaspeak
Copy link
Author

Just tested on my colleague's notebook running Windows 11. It works.
I tested release 2.4.1 on Windows 10 Sandbox and a fresh virtual machine, all are not working. So I may be 'confident' to say it is really not working on Window 10. Am I really the only one still using Windows 10? Why isn't anyone feedback this question?

@panaC
Copy link
Member

panaC commented May 16, 2024

Surely a breaking change with Electron, sorry !

@danielweck
Copy link
Member

Thorium, just like other apps built with Electron, doesn't control at a low-level how file associations are created. I guess Electron Builder broke some Windows 10 backward compatibility ... or maybe the NSIS installer which probably is responsible for configuring the registry.

@panaC panaC removed their assignment May 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants