Skip to content

[5.x] Refactor input component styles#79

Open
Roene-JustBetter wants to merge 1 commit into
masterfrom
Roene-JustBetter-patch-2
Open

[5.x] Refactor input component styles#79
Roene-JustBetter wants to merge 1 commit into
masterfrom
Roene-JustBetter-patch-2

Conversation

@Roene-JustBetter

Copy link
Copy Markdown
Member

We don't need importants here this will break the ring on the inputs.

Removed unnecessary exclamation marks from ring classes.
@jordythevulder

Copy link
Copy Markdown
Collaborator

I can remember that the important was necessary when you used tailwind in combination with the tailwind form plugin. That added custom css on inputs also targeting the rings. But maybe that has changed. Can you double check that?

@jordythevulder jordythevulder left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

See other comment

@Roene-JustBetter

Copy link
Copy Markdown
Member Author

I can remember that the important was necessary when you used tailwind in combination with the tailwind form plugin. That added custom css on inputs also targeting the rings. But maybe that has changed. Can you double check that?

Checked it with @tailwindcss/forms inside my project and without the !important it is working. With the !important it is not working.

@royduin

royduin commented Jun 16, 2026

Copy link
Copy Markdown
Member

Let's wait for RAP-1908 so we can see possible visual differences

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