#5723 closed bug (fixed)
disableSelection() doesn't work cross-browser
| Reported by: | andreinjquery | Owned by: | |
|---|---|---|---|
| Priority: | minor | Milestone: | 1.8.5 |
| Component: | ui.core | Version: | 1.8.2 |
| Keywords: | Cc: | ||
| Blocked by: | Blocking: |
Description
I have a drag & drop row reordering in a table, but in Chrome the disableSelection() does not seem to work and the table will be selected as the user drags the mouse to another row.
To resolve this I changed the following line in disableSelection()...
.css('MozUserSelect', 'none');
... to ...
.css({'MozUserSelect': 'none', 'WebkitUserSelect': 'none'});
This seemed to fix the problem.
Change History (9)
comment:1 Changed 9 years ago by
comment:2 Changed 9 years ago by
There is also another bug. Internet Explorer needs attribute also on all children elements, if any. For example: <p>foo bla <b>bar</b>bla bla<p> Setting attribute unselectable on paragraph is not enough.
comment:4 Changed 9 years ago by
| Milestone: | TBD → 1.9 |
|---|---|
| Summary: | disableSelection() also needs 'WebkitUserSelect' set to none → disableSelection() doesn't work cross-browser |
comment:7 Changed 9 years ago by
| Milestone: | 1.9 → 1.8.5 |
|---|
comment:8 Changed 9 years ago by
...and changing a behavior well established since many versions in the process as reported in #6105.
comment:9 Changed 9 years ago by
Core: Fixed .disableSelect() and .enableSelect() in all browsers. Fixes #5723 - disableSelection() doesn't work cross-browser.
Changeset: 16e93d5189a5aed10df23e85b5d40b14d126eede
enableSelection() needs to be changed as well to:
.css({ 'MozUserSelect': '', 'WebkitUserSelect': '' });