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

Omnibox search should automatically enable/disable when users change the clipboard write permission #110

Open
tdulcet opened this issue Dec 16, 2020 · 3 comments
Labels
bug Something isn't working

Comments

@tdulcet
Copy link
Contributor

tdulcet commented Dec 16, 2020

Bug description

Starting with Firefox 84, users can manage optional permissions for installed add-ons through the Add-ons Manager. The Omnibox search should automatically enable/disable when users change the clipboard write permission.

image

Steps to reproduce

  1. Enable the "Integrate into address bar" option on the options page.
  2. Disable the "Input data into clipboard" permission on the permissions page (see above screenshot).

Actual behavior

Omnibox searches are still enabled, but the emojis are not copied to the clipboard.

Expected behavior

Omnibox searches should be disabled or at least an error should be displayed.

System

Operating system and version: All
Browser and version: Firefox 84
Add-on version: 2.0

Possible solution

The extension can listen for browser.permissions.onAdded and browser.permissions.onRemoved API events. Note that these require at least Firefox 77.

@tdulcet tdulcet added the bug Something isn't working label Dec 16, 2020
@rugk
Copy link
Owner

rugk commented Dec 16, 2020

Oh oh, you’re right! Was not aware of this change.

Actually, fortunately, as it was already implemented in Chrome/ium, I already implemented it when this was created in this module:
https://github.com/TinyWebEx/PermissionRequest/blob/97825931f1893968b8da13adf4513e7922fff559/PermissionRequest.js#L485-L492

Though of course it is not tested and bugs can occur…

@rugk
Copy link
Owner

rugk commented Dec 16, 2020

Though I’m not sure whether that is really the problem/solution. Is not the settings page (re)loaded when you click “settings” in Firefox? So should not it be a fresh new site?

@rugk
Copy link
Owner

rugk commented Dec 16, 2020

Though I’m not sure whether that is really the problem/solution. Is not the settings page (re)loaded when you click “settings” in Firefox? So should not it be a fresh new site?

Just tried myself: no it is not.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants