[FEATURE REQUEST] Make Windows "Open With.." Dialog Load the Image

[ghost]

Checklist

• I have checked that this issue isn't addressed in the FAQ.
• I have checked that this issue isn't addressed in any other issue or pull request.

Related Problem(s)

Right-clicking an image in Windows and selecting open with Upscayl only opens the app, it does not load the image.

Description

App should open with the selected image loaded.

Alternatives

No response

[NayamAmarshe]

That won't be possible for now. I cannot spend my time on a Windows only feature. Electron builder does not seem to support file associations for MacOS and Linux, so unfortunately, I'd have to mark this as unplanned.

PRs are always welcome though, if someone wants to, they can submit a PR for the feature.

[aaronliu0130]

@NayamAmarshe I don't think this is asking for a file association, I think it's when the file is opened with upscayl (which i think is processed as 'upscayl/upscayl.exe "file name.jpg"') the image gets automatically loaded into the select image button

[NayamAmarshe]

@NayamAmarshe I don't think this is asking for a file association, I think it's when the file is opened with upscayl (which i think is processed as 'upscayl/upscayl.exe "file name.jpg"') the image gets automatically loaded into the select image button

That requires file association feature I believe.

[aaronliu0130]

The user can just browse their file manager in the open with-other screen and select anya pp and it gets passed as an argument I believe.

[NayamAmarshe]

Okay, I'll take a look.

[ghost]

Precisely. It doesn't need to get added to a context menu in Windows or anything. It just needs to support a Windows standard executable launch option that has the location of the file to load on startup when going via "Open with" in the right click menu of an image file in Windows explorer.

[aaronliu0130]

I just tested it and in Windows it indeed just passes the file to be opened as the first argument. Reopening for now while NayamAmarshe hasn't stopped looking

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[aaronliu0130]

As a vanilla JS person who only learned React today, the only thing I currently see blocking me from implementing this is the inability to affect the TSX file in index.ts, where arguments are handled. I've tried import and "jsx": "react" (with importing react in the TSX) but it still says Module '../renderer/pages/index' was resolved to 'c:/Users/aaron/Git/upscayl/renderer/pages/index.tsx', but '--jsx' is not set. I really need ideas...

[aaronliu0130]

A few months ago I tried to do this. It's quite a bit harder than I expected and would require us to make the upscayling functions more modular to allow for an alternative entry point.