Touch drag-and-drop support for toolbox#439
Touch drag-and-drop support for toolbox#439cubiclesoft wants to merge 3 commits intoGetmeUK:masterfrom
Conversation
|
This PR may resolve closed issue #382. |
indykoning
left a comment
There was a problem hiding this comment.
line 357 - 362, src/scripts/ui/toolbox.coffee:
I don't know coffeeScript but in normal javascript this might be a oneliner solution:
(ev.type == 'touchmove' ? ev.touches[0].clientX : ev.clientX)
It's a shorthand version of an if. This would remove the need for the @_domElement.style.left =
I do not know if it works the same way in coffeescript.
Other than that great work, can't wait for this to be merged!
|
As far as I know, CoffeeScript doesn't and won't ever support the ternary operator. It's got https://stackoverflow.com/questions/10146080/ternary-operation-in-coffeescript Also, I attempted to stick with the coding style of the surrounding code in this repository. |
A small set of changes that adds drag-and-drop support for touch devices on the floating toolbox.
Demo:
https://cubiclesoft.com/Unrelated/ContentToolsDragTest/sandbox/