Skip to content

Conversation

@Cimbali
Copy link
Contributor

@Cimbali Cimbali commented Apr 15, 2025

Description

Updating this setting which only takes 3 values (since Sept 2022) but those changes never got documented. From the changes in widget/nsXPLookAndFeel.cpp you can see that the "system" theme got removed and "auto" means what "browser" used to mean.

Motivation

It is useful to have docs up-to-date with what they are trying to document.

Additional details

https://hg-edge.mozilla.org/integration/autoland/rev/edaed3aeb0f9

Related issues and pull requests

No related issues found.

This setting only takes 3 values since Sept 2022 (see https://hg-edge.mozilla.org/integration/autoland/rev/edaed3aeb0f9 ) but those changes never got documented. Also not entirely clear as to whether the setting now applies browser or system theme - but I believe it is browser?
@Cimbali Cimbali requested a review from a team as a code owner April 15, 2025 12:33
@Cimbali Cimbali requested review from rebloor and removed request for a team April 15, 2025 12:33
@github-actions github-actions bot added Content:WebExt WebExtensions docs size/s [PR only] 6-50 LoC changed labels Apr 15, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Apr 21, 2025

Preview URLs

(comment last updated: 2025-04-22 02:36:15)

…overridecontentcolorscheme/index.md


Simplify sentence

Co-authored-by: rebloor <git@sherpa.co.nz>
@Cimbali Cimbali requested a review from rebloor April 21, 2025 10:57
Copy link
Contributor

@rebloor rebloor left a comment

Choose a reason for hiding this comment

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

Thanks @Cimbali

@rebloor rebloor merged commit 6dfd513 into mdn:main Apr 22, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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