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

Add ::-webkit-details-marker pseudo to marker variant #17362

Merged
merged 3 commits into from
Mar 25, 2025

Conversation

thecrypticace
Copy link
Contributor

See #17360

This PR updates the marker variant to also target the marker present in <summary> elements in WebKit browsers. Chromium uses ::marker and is therefore already covered.

@thecrypticace thecrypticace requested a review from a team as a code owner March 24, 2025 22:48

Verified

This commit was signed with the committer’s verified signature.
codebytere Shelley Vohr

Verified

This commit was signed with the committer’s verified signature.
codebytere Shelley Vohr
@thecrypticace thecrypticace force-pushed the feat/webkit-details-marker branch from dd9c8d6 to 6126a88 Compare March 24, 2025 22:49
Copy link
Member

@philipp-spiess philipp-spiess left a comment

Choose a reason for hiding this comment

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

Nice!

@@ -623,7 +623,12 @@ export function createVariants(theme: Theme): Variants {
staticVariant('first-line', ['&::first-line'])

// TODO: Remove alpha vars or no?
Copy link
Member

Choose a reason for hiding this comment

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

Is this still relevant? I don't even know what this comment is trying to say so we might wanna remove it lol

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah we should drop the comment

Verified

This commit was signed with the committer’s verified signature.
codebytere Shelley Vohr
@RobinMalfait RobinMalfait merged commit bd501e8 into main Mar 25, 2025
6 checks passed
@RobinMalfait RobinMalfait deleted the feat/webkit-details-marker branch March 25, 2025 12:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants