Sync dark mode on browser_info object

[philippjfr]

Eventually useful when we want to dynamically toggle between dark and light themes.

[MarcSkovMadsen]

Would this enable that if the user has not set ?theme=... as query args then pn.config.theme is set to his/ hers browser preferences? Today its just set to default.

And that if ?theme=... is set it takes precedence.

That would be a big improvement.

[philippjfr]

That's already possible in principle.

[MarcSkovMadsen]

That's already possible in principle.

How? And could it (please) be the default behaviour?

[philippjfr]

How?

pn.config.theme = 'dark' if state.browser_info.dark_mode else 'default'

And could it (please) be the default behaviour?

Maybe in future, but not in a patch release.

[philippjfr]

pre-commit.ci autofix

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 82.22%. Comparing base (4e146af) to head (e5eec5f).
Report is 13 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #7382      +/-   ##
==========================================
+ Coverage   81.42%   82.22%   +0.80%     
==========================================
  Files         337      337              
  Lines       50514    50557      +43     
==========================================
+ Hits        41131    41571     +440     
+ Misses       9383     8986     -397     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.