search

madrobby / scripty2

scripty2: for a more delicious web
http://scripty2.com/
Other
516 stars 57 forks source link

S2.UI.disableTextSelection() prevents focus #8

Closed tvandervossen closed 14 years ago

tvandervossen commented 14 years ago

I’m not sure if this is intentional, but S2.UI.disableTextSelection() prevents elements from getting focus because event handling is stopped in _textSelectionHandler for mousedown events.

Please see http://stuff.vandervossen.net/temporary/focustest/ for a test page and http://stuff.vandervossen.net/temporary/focustest/focus.mov for a demonstration of this issue.

savetheclocktower commented 14 years ago

We should just get rid of the mousedown handler altogether. I think that was left over from an era before -webkit-user-select and -moz-user-select.

(Thanks for the awesome test case!)

savetheclocktower commented 14 years ago

Fixed.