-
Notifications
You must be signed in to change notification settings - Fork 2.2k
Bug-1828220 add browser_action.onUserSettingsChanged event #27355
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
base: main
Are you sure you want to change the base?
Bug-1828220 add browser_action.onUserSettingsChanged event #27355
Conversation
Tip: Review these changes grouped by change (recommended for most PRs), or grouped by feature (for large PRs). |
"__compat": { | ||
"support": { | ||
"chrome": { | ||
"version_added": false |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Chrome added this event in version 130
https://developer.chrome.com/docs/extensions/whats-new#chrome_130_actiononusersettingschanged
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Chrome supports it in the action namespace only, not browserAction:
- https://source.chromium.org/chromium/chromium/src/+/main:chrome/common/extensions/api/action.json;l=409;drc=636dcb39ff241e388b9e4b1ce45ecf656bf0ec77
- https://source.chromium.org/chromium/chromium/src/+/main:chrome/common/extensions/api/browser_action.json;drc=636dcb39ff241e388b9e4b1ce45ecf656bf0ec77
Firefox supports it in both.
webextensions/api/browserAction.json
Outdated
}, | ||
"firefox_android": { | ||
"version_added": "142", | ||
"notes": "Never fires as extensions can't be pinned to the bookmarks toolbar in Firefox for Android." |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I suggest simplifying this statement. It currently implies that Firefox for Android has a bookmarks toolbar that extensions can't be pinned to. At the moment, Firefox for Android does not have a bookmarks toolbar at all.
Summary
Adds details for the addition of the
browser_action.onUserSettingsChanged
event. Addresses the dev-doc-needed requirements of Bug 1828220 Implement onUserSettingsChanged event.Related issues
See mdn/content#40488 for details of the documentation changes.