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

Launchy 2.4.3 doesn't seem to use application setting at all. #121

Closed
matthewtuck opened this issue Dec 7, 2017 · 1 comment · Fixed by #153
Closed

Launchy 2.4.3 doesn't seem to use application setting at all. #121

matthewtuck opened this issue Dec 7, 2017 · 1 comment · Fixed by #153
Assignees

Comments

@matthewtuck
Copy link

matthewtuck commented Dec 7, 2017

It looks like launchy does not use the "application" setting at all, contrary to documentation. It seems to always be ignored, and in a cursory search I can't find anywhere in the source where the Launchy.application method is used.

I was trying this in the context of a data URI which I assume is an unsupported feature. But the documentation suggests I should be able to force this to the browser.

» launchy data:image/png;base64,omitted_data==
ERROR: No application found to handle 'data:image/png'
Try `launchy --help' for more information.

» launchy data:image/png;base64,omitted_data== --application Browser
ERROR: No application found to handle 'data:image/png'
Try `launchy --help' for more information.
@copiousfreetime copiousfreetime self-assigned this Feb 1, 2018
@copiousfreetime
Copy link
Owner

That does appear to be the case. I think that was a goal early on when I pondered launchy doing more than only Browser. Since there is only one application, I think I'll change it so that it is --force-browser or something along those lines.

@copiousfreetime copiousfreetime linked a pull request Feb 4, 2024 that will close this issue
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

Successfully merging a pull request may close this issue.

2 participants