Skip to content

Conversation

rebloor
Copy link
Contributor

@rebloor rebloor commented Sep 4, 2025

Description

Addresses feedback provided by @Rob--W in #40955 (review)

@rebloor rebloor requested review from dotproto and Rob--W September 4, 2025 17:19
@rebloor rebloor self-assigned this Sep 4, 2025
@rebloor rebloor requested a review from a team as a code owner September 4, 2025 17:19
@rebloor rebloor added the Content:WebExt WebExtensions docs label Sep 4, 2025
@github-actions github-actions bot added the size/xs [PR only] 0-5 LoC changed label Sep 4, 2025
@rebloor
Copy link
Contributor Author

rebloor commented Sep 4, 2025

Apologies @Rob--W as the other PR was approved, I didn't go looking for additional feedback before merging.

Copy link
Contributor

github-actions bot commented Sep 4, 2025

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.

manifest.json/content_scripts is also missing mention of case sensitivity.

The release notes also fail to mention that the APIs that take them are now case insensitive.

- : An array of objects. Each object has either a property named `file`, which is a URL starting at the extension's manifest.json and pointing to a CSS file to register, or a property named `code`, which is some CSS code to register.
- `cssOrigin` {{optional_inline}}
- : `string`. The origin of the CSS to inject. This affects the cascading order (priority) of the injected stylesheets. Takes the values `"author"` and `"user"`. Defaults to `"author"`.
- : `string`. The style origin for the injection, either `"user"`, to add the CSS as a user stylesheet, or `"author"`, to add it as an author stylesheet. Defaults to ``"author"`. This property is case insensitive in Firefox and Safari.
Copy link
Member

Choose a reason for hiding this comment

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

Case insensitivity is only in Firefox 144 and later.

Safari does not support the contentScripts API. The scripting API is the cross-browser version of it.

- : `array` of `string`. The list of CSS files to be injected into matching pages. These are injected in the order they appear in this array.
- `cssOrigin` {{optional_inline}}
- : `string`. The origin of the CSS to inject. This affects the cascading order (priority) of the injected stylesheets. Takes the values `"author"` and `"user"`. Defaults to `"author"`.
- : `string`. The style origin for the injection, either `"user"`, to add the CSS as a user stylesheet, or `"author"`, to add it as an author stylesheet. Defaults to ``"author"`. This property is case insensitive in Firefox and Safari.
Copy link
Member

Choose a reason for hiding this comment

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

Case insensitive in Firefox 144 and later.

@rebloor rebloor requested a review from a team as a code owner September 5, 2025 17:16
@rebloor rebloor requested review from pepelsbey and removed request for a team September 5, 2025 17:16
@github-actions github-actions bot added Content:Firefox Content in the Mozilla/Firefox subtree size/s [PR only] 6-50 LoC changed and removed size/xs [PR only] 0-5 LoC changed labels Sep 5, 2025
@rebloor rebloor requested a review from Rob--W September 5, 2025 17:16
@rebloor
Copy link
Contributor Author

rebloor commented Sep 5, 2025

@Rob--W this should fix the issues, sorry about that.

</ul>
<li><code>"user"</code>, to add as a user stylesheet.</li>
<li><code>"author"</code>, to add as an author stylesheet.</li>
</ul>
Copy link
Contributor

Choose a reason for hiding this comment

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

[mdn-linter] reported by reviewdog 🐶

Suggested change
</ul>
</ul>

@rebloor rebloor requested a review from Rob--W September 7, 2025 16:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content:Firefox Content in the Mozilla/Firefox subtree Content:WebExt WebExtensions docs size/s [PR only] 6-50 LoC changed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants