-
Notifications
You must be signed in to change notification settings - Fork 6
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
The icon for the plugin does not show up in the address bar, but rather only as a plugin icon. #16
Comments
Hi @derickr - As you say you should be able to pin it, but you should also be able to position it anywhere that a control is valid. e.g. If that isn't what you mean, could you grab a screenshot of the icon in the position you are talking about? It could be there is some manifest trick to this I am not aware of if you mean inside the actual address input. |
Ah ok, thanks for the clarification - will have to take a look how that is handled! |
I did try to compare the manifests, but the only big thing looked to be the manifest version (2 vs 3)
|
It needs testing but I think this can be resolved with something like changing...
to
in the manifest...I would need to look into it more, it could be mozilla specific, etc - but that seems to be it :) See https://developer.mozilla.org/en-US/docs/Mozilla/Add-ons/WebExtensions/manifest.json/page_action |
I think the old code has this:
"show_matches" : [ "*://*/*" ]
But besides that, how would I test this?
|
I actually have a draft PR with the changes made, so you could just pull that run the build on it. To test, you would go to that branch and..
As an aside the old code with: https://developer.mozilla.org/en-US/docs/Mozilla/Add-ons/WebExtensions/manifest.json/action |
I've tried this out, and it works — I see it in the address bar again! |
Describe the bug
The old helper would show up in the address bar, right next to the star icon (to bookmark). This new one only shows up when you click on the "Extension" (puzzle piece) icon. It is possible to pin it to the toolbar, but not in the same place as the original extension did.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
A way of getting it back in the address bar.
ain your problem.
Desktop (please complete the following information):
The text was updated successfully, but these errors were encountered: