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

Feature suggestion: Add option to open the direct image URL address instead of "moz-extension" one #27

Open
ericpa06 opened this issue Apr 17, 2022 · 3 comments

Comments

@ericpa06
Copy link

ericpa06 commented Apr 17, 2022

Hi. It's me again, nagging you with some feature that may or not to be useful (thanks for implementing the other feature I suggested). I was wondering if it would be possible to add an option that makes the extension to open the direct URL instead of the "moz-extension" one, like.

Instead of this: moz-extension://8596c4cc-02a1-4f8c-9a51-ebca812f47aa/img.html#!https://pic2.imginn.com/whatever.png

Go directly to the URL and open this: https://pic2.imginn.com/whatever.png

image

Why would this be useful? Well, some people use extensions, such as "Search by Image" that can't access "moz-extension" addresses. So for instance, if you want to open an image to than reverse search that image, it doesn't work, you click in it and that context menu entry doesn't appear, you have to go there, clean the moz-extension part, go to the direct URL. It just seem that it would be more practices if there was an option like this.

@johanponken
Copy link

Just FYI, if you press Ctrl-S (Save), the pure URL will load.

@Hexerin
Copy link

Hexerin commented Apr 25, 2023

Adding my support to this being an option. The hotkey workaround that @johanponken brought up does work, but personally I would turn this option on as I would always want results going to the actual image URL.

@ericpa06
Copy link
Author

ericpa06 commented Apr 27, 2023

For anyone wanting to change this behavior, change the bj.js file:

			let u;
			if (bypassMode !== 'off' && oneResult && oneResult.t !== "VIDEO") {
				u = "/img.html#!" + oneResult.e;
			} else {
				u = "/inline.html";
			}

Remove the following part: "/img.html#!" + . So the code will be:

			let u;
			if (bypassMode !== 'off' && oneResult && oneResult.t !== "VIDEO") {
				u =  oneResult.e;
			} else {
				u = "/inline.html";
			}

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