Closed warnerandy closed 10 years ago
cool, i'll make the changes
@frankiefu i've made your changes let me know if there is anything else i need to do
@frankiefu fixed the tabbing and the missed camelCasing
lgtm
FWIW, you can achieve the same with preventDefault. See https://chromium.googlesource.com/chromium/src/+/master/ui/webui/resources/js/cr/ui/splitter.js#138
Sounds like this control should be built into the tracking gesture.
@azakus The preventDefault needs to be done in mousedown. Calling preventDefault in trackstart did not work. I'll have to look into how trackstart is implemented.
@arv I imagine that trackstart is probably too late, as it is sent asynchronously, and only after a hysteresis threshold.
down doesn't work too. e.g.
on-down="{{preventSelection}}"
...
preventSelection: function(e) {
e.preventDefault();
}
@frankiefu You need to use on-mousedown
. See #4. I manually tested it.
@azakus Maybe we should move this into PointerGestures repo. preventDefault in down should work, shouldn't it?
Thanks @arv for the insight. Also polymer now uses polymer-gestures instead of PointerGestures. I have filed couple issues on polymer-gestures regarding preventDefault in down doesn't work and have disable selection built into tracking gesture: https://github.com/Polymer/polymer-gestures/issues/20 https://github.com/Polymer/polymer-gestures/issues/21
Having text become selected while moving the splitter is unwanted behavior. So lets disable text selection while the splitter is moving.