Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BasicUI] Add the "follow system" theme #1829

Merged
merged 4 commits into from
Apr 13, 2023

Conversation

lolodomo
Copy link
Contributor

@lolodomo lolodomo commented Apr 8, 2023

Closes #1828

3 themes are now available! bright, dark and system.
Theme previously named "default" has been renamed into "bright".
"default" is still accepted for backward compatibility.

Tested with Fiorefox, Chrome and Edge on Windows 10.
Tested with Chrome on Android.

Signed-off-by: Laurent Garnier [email protected]

Closes openhab#1828

3 themes are now available! bright, dark and system.
Theme previously named "default" has been renamed into "bright".
"default" is still accepted for backward compatibility.

Tested with Fiorefox, Chrome and Edge on Windows 10.
Tested with Chrome on Android.

Signed-off-by: Laurent Garnier <[email protected]>
@lolodomo lolodomo requested a review from a team as a code owner April 8, 2023 18:44
@lolodomo lolodomo added enhancement New feature or request basic ui Basic UI labels Apr 8, 2023
@openhab openhab deleted a comment from dilyanpalauzov Apr 9, 2023
@openhab openhab deleted a comment from dilyanpalauzov Apr 9, 2023
@lolodomo
Copy link
Contributor Author

I have to update this PR now that PR #1834 has been merged.

@lolodomo
Copy link
Contributor Author

PR is again ready for review.

Copy link
Member

@kaikreuzer kaikreuzer left a comment

Choose a reason for hiding this comment

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

Thanks - just a very minor change request.

Copy link
Member

@kaikreuzer kaikreuzer left a comment

Choose a reason for hiding this comment

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

Thanks!

@kaikreuzer kaikreuzer merged commit 1e0f1a1 into openhab:main Apr 13, 2023
@kaikreuzer kaikreuzer added this to the 4.0 milestone Apr 13, 2023
@lolodomo lolodomo deleted the basicui_system_theme branch April 14, 2023 00:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
basic ui Basic UI enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BasicUI] Offer theme following system
2 participants