fix: remove pointer-events: none, because it causes cursor: not-allow…

[vladmonday]

Removing the pointer-events: none for disabled button, because with it the cursor: not-allowed doesn't really work and all disabled button have default cursor instead of the not allowed one.

Updating existing component

Basic

• PR has description
• Changes to the component are backward compatible (including selectors structure). If not - add to the title of the PR "BREAKABLE_CHANGE""
• All changes to the component are reflected in the ReadMe
• If component is old and was not compliant with the latest guidelines - it was fixed (optional)

Style

• CSS selectors are named using BEM convention
• Design is compatible with Monday Design System

Storybook

• All the changes to the component should be reflected in the Storybook.
• Component passed Accessibility Plugin checks

Tests

• All current tests are passing
• New functionality is covered with tests
• Tests are compliant with TESTING_README.md instructions

[orrgottlieb]

Hi @vladmonday ,
The reason that we have pointer-events none is - facebook/react#4251
I'm checking to see if it is fixed in 16.13.1 or we might need to upgrade to 17 in order for it to be supported well.
it affects our tooltip when we are wrapping a component with it

[orrgottlieb]

Hi @vladmonday,
https://codepen.io/orrgottl/pen/jOMywxL this is actually how React works (attaching the event to the root element), so this is the reason it doesn't work.
they have several open bugs on the issue.
I rather leave the current issue as it is, We can have a fix for it using native event (which is a much bigger PR)
Let's discuss this
Orr