Skip to content

feat: set global color-scheme#2379

Merged
JammingBen merged 1 commit intomainfrom
feat/global-css-color-scheme
Apr 16, 2026
Merged

feat: set global color-scheme#2379
JammingBen merged 1 commit intomainfrom
feat/global-css-color-scheme

Conversation

@JammingBen
Copy link
Copy Markdown
Member

Sets the current theme via the color-scheme CSS prop, ensuring that native browser elements like scroll bars have the proper color.

closes #2123

Comment thread index.html
<head>
<meta charset="utf-8" />
<meta name="viewport" content="initial-scale=1.0, minimum-scale=1.0" />
<meta name="theme-color" content="#375f7E" />
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

why the removal? I thought this does something shiny in Safari

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Ah okay, can you remember what? 🤔 I thought I remove it because it's not widely supported by browsers and #375f7e looks like a random color that's not related to any of our colors.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

ok 👍 no idea where it's even visible in Safari. Ok, kill it then.

@streaminganger
Copy link
Copy Markdown

i recall the date picker also had some colorScheme setting based on isDark. i think this pr will cover that too

@streaminganger
Copy link
Copy Markdown

:class="{ 'oc-date-picker-dark': isDark }"

.oc-date-picker-dark input {
color-scheme: dark;
}
.oc-date-picker-dark input::-webkit-calendar-picker-indicator {
filter: invert(0);
}

this is it. but im not sure what filter: invert(0) does and what happens if its removed?

Sets the current theme via the `color-scheme` CSS prop,
ensuring that native browser elements like scroll bars
have the proper color.
@JammingBen JammingBen force-pushed the feat/global-css-color-scheme branch from bce0842 to 84af22f Compare April 16, 2026 10:47
@JammingBen
Copy link
Copy Markdown
Member Author

Thanks for the hint! This is obsolete now.

@JammingBen JammingBen merged commit 168cd7a into main Apr 16, 2026
29 checks passed
@JammingBen JammingBen deleted the feat/global-css-color-scheme branch April 16, 2026 11:43
openclouders pushed a commit that referenced this pull request Apr 16, 2026
@openclouders openclouders mentioned this pull request Apr 16, 2026
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Dark mode scrollbar

4 participants