Skip to content

Bug-1958596 dark_theme manifest key support#26511

Merged
rebloor merged 5 commits intomdn:mainfrom
rebloor:Bug-1958596-dark_theme-manifest-key-support
May 9, 2025
Merged

Bug-1958596 dark_theme manifest key support#26511
rebloor merged 5 commits intomdn:mainfrom
rebloor:Bug-1958596-dark_theme-manifest-key-support

Conversation

@rebloor
Copy link
Contributor

@rebloor rebloor commented Apr 14, 2025

Summary

Adds browser compatibility data for the dark_theme manifest key introduced in Firefox 68 (Bug 1525762) response to comment on Comment 15 on Bug 1958596

Related issues

@rebloor rebloor added the data:webext Compat data for Browser Extensions. https://developer.mozilla.org/Add-ons/WebExtensions label Apr 14, 2025
@rebloor rebloor self-assigned this Apr 14, 2025
@github-actions github-actions bot added the size:l [PR only] 101-1000 LoC changed label Apr 14, 2025
"__compat": {
"support": {
"chrome": {
"version_added": "≤59"
Copy link
Member

Choose a reason for hiding this comment

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

Chrome doesn't support any of the dark_theme properties.

Removed features with no support
@rebloor rebloor marked this pull request as ready for review April 16, 2025 04:20
@rebloor rebloor requested a review from Rob--W April 16, 2025 04:20
"edge": "mirror",
"firefox": {
"version_added": "68",
"notes": "Deprecated in Firefox 89. For details, see the [Changes to themeable areas of Firefox in version 89](https://blog.mozilla.org/addons/2021/04/19/changes-to-themeable-areas-of-firefox-in-version-89/) blog post ."
Copy link
Member

Choose a reason for hiding this comment

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

Can we drop removed properties altogether, since it was never documented anyway?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@Rob--W FYI tab_background_separator and toolbar_field_separator are documented under the theme key. Should I plan on removing them?

@github-actions
Copy link
Contributor

github-actions bot commented May 9, 2025

Tip: Review these changes grouped by change (recommended for most PRs), or grouped by feature (for large PRs).

@rebloor rebloor merged commit 01bbc52 into mdn:main May 9, 2025
6 checks passed
@rebloor rebloor deleted the Bug-1958596-dark_theme-manifest-key-support branch May 9, 2025 03:03
@mdn-bot mdn-bot mentioned this pull request May 9, 2025
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 size:l [PR only] 101-1000 LoC changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants