Skip to content

fix(Input): engage when clicking over icons #4270

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Open
wants to merge 1 commit into
base: v3
Choose a base branch
from

Conversation

murshex
Copy link

@murshex murshex commented Jun 1, 2025

πŸ”— Linked issue

❓ Type of change

  • πŸ“– Documentation (updates to the documentation or readme)
  • 🐞 Bug fix (a non-breaking change that fixes an issue)
  • πŸ‘Œ Enhancement (improving an existing functionality)
  • ✨ New feature (a non-breaking change that adds functionality)
  • 🧹 Chore (updates to the build process or auxiliary tools and libraries)
  • ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

πŸ“š Description

engage input when clicking over icons

πŸ“ Checklist

  • I have linked an issue or discussion.
  • I have updated the documentation accordingly.

@murshex murshex requested a review from benjamincanac as a code owner June 1, 2025 15:08
@benjamincanac
Copy link
Member

This change from v2 was actually intended because it was required to set pointer-events-auto every time we used a clickable element in there like: https://ui.nuxt.com/components/input#with-clear-button

@murshex
Copy link
Author

murshex commented Jun 3, 2025

This change from v2 was actually intended because it was required to set pointer-events-auto every time we used a clickable element in there like: https://ui.nuxt.com/components/input#with-clear-button

Well to be fair, now its required to set 'pointer-events-none' every time we don't use a clickable element in there, which I would say is probably the more common use case.

@benjamincanac benjamincanac changed the title fix(Button): engage input when clicking over icons fix(Input): engage when clicking over icons Jun 12, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants