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

MV3 feature release clarification #19026

Merged
merged 11 commits into from Apr 4, 2023

Conversation

rebloor
Copy link
Collaborator

@rebloor rebloor commented Feb 28, 2023

Summary

Updates support information for Manifest V3 features which were in preview prior to Firefox 109.

Related issues

Related changes to content made in mdn/content#24916

@rebloor rebloor requested a review from Rob--W February 28, 2023 03:07
@github-actions github-actions bot added the data:webext 🎲 Compat data for Browser Extensions. https://developer.mozilla.org/Add-ons/WebExtensions label Feb 28, 2023
@rebloor rebloor marked this pull request as ready for review March 12, 2023 16:09
Copy link
Collaborator

@queengooborg queengooborg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Notes are actually formatted as HTML rather than Markdown, so we'll need to use <a href=""></a> rather than []().

Copy link
Member

@Rob--W Rob--W left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Github does not offer the ability to comment on non-changed lines, so here are more pieces of feedback:

  • Drop mentions of "MV3 preview" from the BCD. The pre-release testing phase is not relevant to devs, especially because MV3 support has been enabled by default in Firefox 109+.
  • manifest/action.js: should refer to the action MDN articles instead of the browser_action one.
    • Not strictly part of this PR, but it would also be nice tot interlink each other (action / browser_action at "See also" in mdn/content )
  • manifest/content_security_policy.json: Like Firefox, Chrome has also dropped the object-src requirement, since version 111 - https://bugs.chromium.org/p/chromium/issues/detail?id=1320785#c10

webextensions/api/scripting.json Outdated Show resolved Hide resolved
webextensions/api/scripting.json Outdated Show resolved Hide resolved
@rebloor rebloor requested a review from Rob--W March 22, 2023 02:07
"support": {
"chrome": {
"version_added": "88",
"notes": "Available for use in Manifest V3 or later."
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Rob--W should I repeat this against all the sub features?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Rob--W do you think we need the "Available for use in Manifest V2 or later." notes against Safari support?

"support": {
"chrome": {
"version_added": "88",
"notes": [
"Available for use in Manifest V3 or later.",
"If an extension defines a browser action, it is not allowed to define a page action as well."
"If an extension defines an action, it is not allowed to define a page action as well."
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Rob--W correct?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Chrome does not support page_action in MV3 at all. So "not allowed to define a page action" is not conditional on the action key being present. In MV2, Chrome does not allow browser_action and page_action to co-exist (and doesn't support the action key either).

FYI: Safari does not support page_action nor browser_action in MV3 either (only action). In MV2 Safari does support page_action and browser_action.

(and for completeness: Firefox supports browser_action only in MV2, action only in MV3, and page_action in MV2+MV3).

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @Rob--W I've removed this note.

@rebloor
Copy link
Collaborator Author

rebloor commented Mar 22, 2023

Not strictly part of this PR, but it would also be nice tot interlink each other (action / browser_action at "See also" in mdn/content )

Is not the "This key replaces browser_action in Manifest V3 extensions." and the reciprocal version in browser_action adequate?

webextensions/manifest/action.json Outdated Show resolved Hide resolved
webextensions/manifest/background.json Outdated Show resolved Hide resolved
@rebloor rebloor requested a review from Rob--W March 29, 2023 22:56
Copy link
Member

@Rob--W Rob--W left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Current changes here look good.

To complete the full scope of "MV3 feature clarification", please also update:

That change is trivial, so feel free to merge after making the change (or submit a follow-up PR if you wish).

@rebloor
Copy link
Collaborator Author

rebloor commented Apr 4, 2023

Thanks @Rob--W, I have made the final change you requested and merged.

@rebloor rebloor merged commit 5d66484 into mdn:main Apr 4, 2023
4 checks passed
@rebloor rebloor deleted the MV3-feature-release-clarification branch April 5, 2023 22:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data:webext 🎲 Compat data for Browser Extensions. https://developer.mozilla.org/Add-ons/WebExtensions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants