Default to pointer-events: none on Sortable helper
#160
Conversation
src/SortableContainer/index.js
Outdated
| @@ -288,6 +288,7 @@ export default function sortableContainer(WrappedComponent, config = {withRef: f | |||
| this.helper.style.width = `${this.width}px`; | |||
| this.helper.style.height = `${this.height}px`; | |||
| this.helper.style.boxSizing = 'border-box'; | |||
| this.helper.style.pointerEvents = 'none'; | |||
There was a problem hiding this comment.
White space police! 🚓 Tab spotted!👮
There was a problem hiding this comment.
😂
The one time I'm lazy and decide to use the built-in github editor 🙄
There was a problem hiding this comment.
Never will I ever again.
There was a problem hiding this comment.
Prettier-bot would not have allowed this 🤖
There was a problem hiding this comment.
It may be time to step away from open source, Clauderic
clauderic
force-pushed
the
feature/helper_pointer_events
branch
from
fbb7c5b
to
e0c7f79
Compare
…r_events Default to `pointer-events: none` on Sortable helper
|
Why!! just why??!!? |
|
@clauderic Took me a while until I finally found why the cursor is not behaving as it is in the examples. Ahmedisam99 pointed it out already, in a rude manner though, that this change breaks the cursor styles. Would be nice if this was configurable. Also it's misleading that the Examples are not up-to-date where this change has not yet been made. |
Because... trackpad ninjas.
https://twitter.com/Vjeux/status/840264657424146436