Skip to content

Conversation

@pdanpdan
Copy link
Contributor

@pdanpdan pdanpdan commented Nov 7, 2025

feat: unify styles for btn-outline and btn-dash

  • from my tests tailwind accepts this and creates classes modified with variants as required !!! if you think this is not a good ideea please let me know !!!

feat: respect input:checked for btn-soft, cleanup
feat: respect input:checked for btn-ghost
feat: cleanup btn-dash
feat: cleanup btn-outline
feat: use color defined by modifier for btn-ghost
feat: remove rules for :hover on disabled button because it already has pointer-events-none

  • disabled btn cannot have hover because of pointer-events-none
  • disabled btn cannot have hover because of pointer-events-none
  • disabled btn cannot have hover because of pointer-events-none
  • disabled btn cannot have hover because of pointer-events-none

feat: use color defined by modifier for btn-link, do not color disabled links, do not remove underline

  • I think removing underline on mobile on hover was introduced by mistake - there is nothing else to suggest that underline should be removed

feat: do not remove underline from btn-link when it is in prose not-prose

example: https://play.tailwindcss.com/Gk9Jv0Vs6v?file=css

…ed links, do not remove underline

- I think removing underline on mobile on hover was introduced by mistake - there is nothing else to suggest that underline should be removed
- disabled btn cannot have hover because of pointer-events-none
- disabled buttons do not have hover
- disabled buttons do not have hover
disabled buttons do not have hover
- from my tests tailwind accepts this and creates classes modified with variants as required
@pdanpdan pdanpdan changed the title Button improvements and cleanup feat: button improvements and cleanup Nov 7, 2025
@saadeghi saadeghi self-assigned this Nov 7, 2025
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