Skip to content

Replace currentColor with currentcolor (lowercase)#17510

Merged
philipp-spiess merged 2 commits intotailwindlabs:mainfrom
teddybradford:main
Apr 3, 2025
Merged

Replace currentColor with currentcolor (lowercase)#17510
philipp-spiess merged 2 commits intotailwindlabs:mainfrom
teddybradford:main

Conversation

@teddybradford
Copy link
Contributor

Replaces currentColor with currentcolor (lowercase) to match what's defined in CSS Color Module Level 4 and MDN (see: mdn/content#16592).

@teddybradford teddybradford requested a review from a team as a code owner April 2, 2025 14:21
Copy link
Contributor

@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.

Thanks! I think this is a nice change. Note that I had to revert a bunch of changed places since some snapshot is including CSS that is transformed via lightningcss which always changes currentcolor to currentColor.

@philipp-spiess philipp-spiess merged commit 3e41e9f into tailwindlabs:main Apr 3, 2025
7 checks passed
tongrow pushed a commit to tongrow/tailwind-normalize that referenced this pull request Jun 2, 2025
Replaces `currentColor` with `currentcolor` (lowercase) to match what's
defined in [CSS Color Module Level
4](https://www.w3.org/TR/css-color-4/#currentcolor-color) and
[MDN](https://developer.mozilla.org/en-US/docs/Web/CSS/color_value#currentcolor_keyword)
(see: mdn/content#16592).

---------

Co-authored-by: Philipp Spiess <hello@philippspiess.com>
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.

2 participants