Skip to content

Conversation

@tassilogroeper
Copy link
Contributor

Q A
Bug fix? yes
New feature? no
Deprecations? no
Documentation? no
Issues
License MIT

Confusion may occur, if the badge shows a different style on mouse hover. As it is not a button, we need to remove it.

Additionally, the badge component on Shadcn does not hold hover styles, when interacted with a mouse. see https://ui.shadcn.com/docs/components/badge

@carsonbot carsonbot added Status: Needs Review Needs to be reviewed Bug Bug Fix Toolkit labels Nov 29, 2025
@Kocal Kocal force-pushed the toolkit-badge-remove-hover_style branch from a77470f to f14635b Compare November 29, 2025 14:03
Kocal added a commit to Kocal/symfony-ux that referenced this pull request Nov 29, 2025
…nguish from Button (Tassilo Gröper)

This PR was merged into the 2.x branch.

Discussion
----------

[Toolkit][Badge] Remove hover style to clearly distinguish from Button

| Q              | A
| -------------- | ---
| Bug fix?       | yes
| New feature?   | no
| Deprecations?  | no
| Documentation? | no
| Issues         |
| License        | MIT

Confusion may occur, if the badge shows a different style on mouse hover. As it is not a button, we need to remove it.

Additionally, the badge component on Shadcn does not hold hover styles, when interacted with a mouse. see  https://ui.shadcn.com/docs/components/badge

Commits
-------

f14635b [Toolkit][Badge] Remove hover style to clearly distinguish badge from button
@Kocal Kocal merged commit 0de38f6 into symfony:2.x Nov 29, 2025
26 of 31 checks passed
@Kocal
Copy link
Member

Kocal commented Nov 29, 2025

Thanks @tassilogroeper!

Kocal added a commit that referenced this pull request Nov 30, 2025
* 2.x:
  bug #3197 [Toolkit][Badge] Remove hover style to clearly distinguish from Button (Tassilo Gröper)
  [Toolkit][Kdb] Add new component kdb from shadcn ui
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug Bug Fix Status: Needs Review Needs to be reviewed Toolkit

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants