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

Updates to Chrome and Firefox Extensions #10

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

iLama
Copy link
Member

@iLama iLama commented Jul 24, 2023

No description provided.

Manifest V3 Updates:
 - Replace `background.page` with `background.service_worker` and point it to `background.js`
  - Remove `background.html` as it's not necessary
  - Move URL Permissions to `optional_host_permissions`
 - Updates to `background.js`:
    - Change `getClickHandler` to Object Function
    - Replace `escape` with `encodeURIComponent` since `escape` is deprecated
    - Assign the context menu an ID as is required by Manifest V3
    - Add EventListener for context menu item click

Unrelated to V3:
  - Formatting
  - Variable declarations
Firefox's implementation of Manifest V3 has some slight deviations for Chrome so for now we need a version with a Manifest specific to Firefox
@iLama iLama requested a review from bradchoate July 24, 2023 14:59
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 this pull request may close these issues.

2 participants