Skip to content

Conversation

adamwathan
Copy link
Member

Early in v4 development we decided to give all form controls a transparent background, but in v3 we only did this for button elements.

This PR reverts that decision to make things consistent with v3, as we've noticed this is something that tends to break for people when upgrading from v3 to v4.

The default background color of form elements is the Field system color which automatically adapts to light and dark modes if you don't touch it, so it feels reasonable to keep this as the default. Changing it just makes upgrading harder and doesn't really make anything easier for anyone else.

@adamwathan adamwathan requested a review from a team as a code owner November 8, 2024 02:14
@adamwathan adamwathan enabled auto-merge (squash) November 8, 2024 15:14
@adamwathan adamwathan disabled auto-merge November 8, 2024 15:19
@adamwathan adamwathan merged commit 192109b into next Nov 8, 2024
1 check passed
@adamwathan adamwathan deleted the fix/dont-make-inputs-transparent branch November 8, 2024 15:19
tongrow pushed a commit to tongrow/tailwind-normalize that referenced this pull request Jun 2, 2025
…ndlabs#14913)

Early in v4 development we decided to give all form controls a
transparent background, but in v3 we only did this for button elements.

This PR reverts that decision to make things consistent with v3, as
we've noticed this is something that tends to break for people when
upgrading from v3 to v4.

The default background color of form elements is the [`Field` system
color](https://developer.mozilla.org/en-US/docs/Web/CSS/system-color#field)
which automatically adapts to light and dark modes if you don't touch
it, so it feels reasonable to keep this as the default. Changing it just
makes upgrading harder and doesn't really make anything easier for
anyone else.

---------

Co-authored-by: Adam Wathan <[email protected]>
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.

3 participants