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

Support theme meta w3c standard #32231

Merged
merged 1 commit into from
Feb 7, 2024
Merged

Support theme meta w3c standard #32231

merged 1 commit into from
Feb 7, 2024

Conversation

skjnldsv
Copy link
Member

@skjnldsv skjnldsv commented May 1, 2022

@skjnldsv skjnldsv added this to the Nextcloud 25 milestone May 1, 2022
@skjnldsv skjnldsv requested a review from a team May 1, 2022 10:14
@skjnldsv skjnldsv self-assigned this May 1, 2022
@skjnldsv skjnldsv requested review from PVince81, artonge, vanpertsch and szaimen and removed request for a team May 1, 2022 10:14
@szaimen

This comment was marked as resolved.

@skjnldsv

This comment was marked as resolved.

@szaimen
Copy link
Contributor

szaimen commented May 1, 2022

That's not related to this pr, is it?

Not really, no. Could be done in a follow-up I suppose...
image
image

@skjnldsv
Copy link
Member Author

skjnldsv commented May 1, 2022

That's not related to this pr, is it?

Not really, no. Could be done in a follow-up I suppose... image image

Ah! But it's also kinda related to the feature, sorry I misunderstood :)
I'll see that after the weekend 🙈

Copy link
Member

@CarlSchwan CarlSchwan left a comment

Choose a reason for hiding this comment

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

Nice feature :)

apps/theming/lib/ITheme.php Outdated Show resolved Hide resolved
This was referenced Aug 12, 2022
This was referenced Aug 24, 2022
This was referenced Sep 6, 2022
@skjnldsv skjnldsv mentioned this pull request Sep 15, 2022
This was referenced Sep 20, 2022
@blizzz blizzz modified the milestones: Nextcloud 25, Nextcloud 26 Sep 22, 2022
@blizzz blizzz mentioned this pull request Feb 1, 2023
@skjnldsv skjnldsv mentioned this pull request Feb 23, 2023
@szaimen szaimen requested review from a team and Pytal and removed request for vanpertsch, a team and szaimen April 13, 2023 10:44
@szaimen szaimen added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Apr 13, 2023
@skjnldsv skjnldsv mentioned this pull request May 3, 2023
@skjnldsv skjnldsv modified the milestones: Nextcloud 27, Nextcloud 28 May 3, 2023
@szaimen szaimen added 2. developing Work in progress and removed 3. to review Waiting for reviews labels May 16, 2023
@skjnldsv skjnldsv removed this from the Nextcloud 28 milestone Sep 2, 2023
@skjnldsv skjnldsv force-pushed the feat/theming-scheme-meta branch 3 times, most recently from d0cf244 to 4e2b616 Compare February 7, 2024 11:29
@skjnldsv skjnldsv added the 4. to release Ready to be released and/or waiting for tests to finish label Feb 7, 2024
@skjnldsv skjnldsv added this to the Nextcloud 29 milestone Feb 7, 2024
@skjnldsv skjnldsv removed the 2. developing Work in progress label Feb 7, 2024
apps/theming/lib/ITheme.php Fixed Show fixed Hide fixed
Signed-off-by: John Molakvoæ <skjnldsv@protonmail.com>
Signed-off-by: Simon L <szaimen@e.mail.de>
@skjnldsv skjnldsv merged commit 566586c into master Feb 7, 2024
140 checks passed
@skjnldsv skjnldsv deleted the feat/theming-scheme-meta branch February 7, 2024 17:20
@blizzz blizzz mentioned this pull request Mar 5, 2024
@ShGKme
Copy link
Contributor

ShGKme commented Mar 7, 2024

This PR changes focus-visible styles for me.

image

I have a dark system theme and a light Nextcloud theme. With this PR, I have

<meta name="color-scheme" content="light dark">

Is light dark an expected value? Why dark if I have the light theme?

With light - outline is the same as it was before this PR:

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4. to release Ready to be released and/or waiting for tests to finish enhancement feature: theming
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants