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

feat(theming): Allow shipped backgrounds to have a dark variant #46453

Merged
merged 1 commit into from
Jul 12, 2024

Conversation

susnux
Copy link
Contributor

@susnux susnux commented Jul 11, 2024

Summary

Add the new default backgrounds with dark and light variants depending on color schema.
This also allows future shipped backgrounds to define a dark variant (and with minor adjustments probably we can also support custom ones).

vokoscreenNG-2024-07-11_15-00-30.webm

Checklist

@susnux susnux added this to the Nextcloud 30 milestone Jul 11, 2024
@susnux susnux self-assigned this Jul 11, 2024
@susnux susnux requested a review from sorbaugh July 11, 2024 14:28
Copy link
Member

@AndyScherzinger AndyScherzinger left a comment

Choose a reason for hiding this comment

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

code-wise and video look good to me!

@AndyScherzinger
Copy link
Member

Nice work @susnux 👍

@susnux susnux force-pushed the feat/background-with-dark-variant branch from cd54f0c to b21ec91 Compare July 11, 2024 14:35
Copy link
Member

@jancborchardt jancborchardt left a comment

Choose a reason for hiding this comment

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

Really nice! :)

@susnux susnux force-pushed the feat/background-with-dark-variant branch from b21ec91 to c3edc53 Compare July 11, 2024 18:05
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@susnux susnux force-pushed the feat/background-with-dark-variant branch from c3edc53 to d5dc6b2 Compare July 12, 2024 11:00
@susnux susnux merged commit e13b59b into master Jul 12, 2024
166 checks passed
@susnux susnux deleted the feat/background-with-dark-variant branch July 12, 2024 13:04
@susnux susnux mentioned this pull request Jul 16, 2024
5 tasks
@blizzz blizzz mentioned this pull request Jul 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants