Summary

Enable system-matching light mode/dark mode by default.

Review Request #14167 — Created Sept. 12, 2024 and updated Sept. 16, 2024, 9:05 p.m.

Information

Owner

chipx86

Repository

Review Board

Branch

release-7.1.x

Bugs

Depends On

Reviewers

Groups

reviewboard

People

Description

Review Board 7.1 will now ship with system theme-matching on by default,
instead of light mode. This will enable dark mode if the system is set
to dark mode, and light mode otherwise.

With this, dark mode is no longer in beta.

Unless a user has explicitly changed their theme, they'll use
system theme-matching as the new default. If they've explicitly opted
into either light or dark mode, they'll continue using that instead.

Testing Done

Verified that my default theme was matching my system.
Checked the Appearance tab and saw the options we want to let users

pick from: Default (system theme), Dark Mode, and Light Mode.
All unit tests pass.

Commits

Summary	ID
Enable system-matching light mode/dark mode by default. Review Board 7.1 will now ship with system theme-matching on by default, instead of light mode. This will enable dark mode if the system is set to dark mode, and light mode otherwise. With this, dark mode is no longer in beta. Unless a user has explicitly changed their theme, they'll use system theme-matching as the new default. If they've explicitly opted into either light or dark mode, they'll continue using that instead.	f55b177c298c6d1edd11e42d24ab743e819dc274

Summary

Enable system-matching light mode/dark mode by default.

Review Board 7.1 will now ship with system theme-matching on by default, instead of light mode. This will enable dark mode if the system is set to dark mode, and light mode otherwise. With this, dark mode is no longer in beta. Unless a user has explicitly changed their theme, they'll use system theme-matching as the new default. If they've explicitly opted into either light or dark mode, they'll continue using that instead.

f55b177c298c6d1edd11e42d24ab743e819dc274

Issues

Description	From	Last Updated
line too long (82 > 79 characters) Column: 80 Error code: E501	reviewbot	Sept. 12, 2024, 4:42 p.m.
'reviewboard.themes.ui.default.LightUITheme' imported but unused Column: 1 Error code: F401	reviewbot	Sept. 12, 2024, 4:42 p.m.
Hmm, what's up with this file? Something you forgot to commit earlier?	david	Sept. 16, 2024, 8:50 a.m.
Would it be more consistent to just say System theme rather than Use system theme?	OfficeStapler	Sept. 16, 2024, 8:31 p.m.

flake8 failed.

JSHint passed.

flake8

reviewboard/themes/tests/test_ui_theme_registry.py (Diff revision 1)
The issue has been resolved. Show all issues
```
line too long (82 > 79 characters)

Column: 80
Error code: E501
```
reviewboard/themes/tests/test_ui_theme_registry.py (Diff revision 1)
The issue has been resolved. Show all issues
```
'reviewboard.themes.ui.default.LightUITheme' imported but unused

Column: 1
Error code: F401
```

Change Summary:

Removed an unused import.

Commits:

	Summary	ID
	Enable system-matching light mode/dark mode by default. Review Board 7.1 will now ship with system theme-matching on by default, instead of light mode. This will enable dark mode if the system is set to dark mode, and light mode otherwise. With this, dark mode is no longer in beta. Unless a user has explicitly changed their theme, they'll use system theme-matching as the new default. If they've explicitly opted into either light or dark mode, they'll continue using that instead.	1895ba880803a144289c31a5eb4400fd6ddda1f2
	Enable system-matching light mode/dark mode by default. Review Board 7.1 will now ship with system theme-matching on by default, instead of light mode. This will enable dark mode if the system is set to dark mode, and light mode otherwise. With this, dark mode is no longer in beta. Unless a user has explicitly changed their theme, they'll use system theme-matching as the new default. If they've explicitly opted into either light or dark mode, they'll continue using that instead.	f55b177c298c6d1edd11e42d24ab743e819dc274

Diff:

Revision 2 (+306 -20)

Show changes

	reviewboard/accounts/tests/test_theme_form.py
	reviewboard/themes/code/theme_reviewboard.py
	reviewboard/themes/tests/test_ui_theme_registry.py
	reviewboard/themes/ui/default.py
	reviewboard/themes/ui/registry.py

Checks run (2 succeeded)

flake8 passed.

JSHint passed.

Fix it!

reviewboard/themes/code/theme_reviewboard.py (Diff revision 2)

An issue was opened. Show all issues

Hmm, what's up with this file? Something you forgot to commit earlier?

chipx86 Sept. 16, 2024, 9:05 p.m.

Definitely not supposed to be there. I must have somehow merged in something from an in-progress branch...

Fix it!

reviewboard/themes/ui/default.py (Diff revision 2)

An issue was opened. Show all issues

Would it be more consistent to just say System theme rather than Use system theme?

chipx86 Sept. 16, 2024, 9:05 p.m.

Probably would, thanks! Let me make that change and see how it reads in the UI.