Skip to content

Conversation

@balajis-qb
Copy link
Collaborator

@balajis-qb balajis-qb commented Sep 24, 2023

Here's a summary of the changes made

  • Previously, we were using a generic mouseup event to detect the click event
  • I've now refined it to specifically detect left mouse clicks, making our event handling more precise
  • This change ensures that our component responds correctly to left mouse clicks without interfering with other mouse actions.

Please review the code changes, and if everything looks good, consider merging this update.

Copy link

@SoyDiego SoyDiego left a comment

Choose a reason for hiding this comment

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

I tested and works perfectly but maybe for any reason the developers of PrimeNG are doing that @cetincakiroglu.

You are right @balajis-qb, also if you click with the mousewheel is opening the window to select a file too.

@balajis-qb
Copy link
Collaborator Author

balajis-qb commented Sep 24, 2023

Hi @SoyDiego ,

Thank you for approving my PR. Actually this PR is pointing to my forked repo's master. The correct PR I made was primefaces#13723

Sorry for the inconvenience. Could you please review my PR from this link (primefaces#13723)?

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