Changes between Initial Version and Version 1 of Ticket #18556, comment 15
- Timestamp:
- 08/22/2016 04:16:28 PM (8 years ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Ticket #18556, comment 15
initial v1 1 1 I've attached [https://core.trac.wordpress.org/attachment/ticket/18556/18556.5.diff another attempt] to fix this. With a single selector I'm targeting all current draggables and sortables, so there is no need to add code to all event handlers. The patch also disables popup menus on the admin menu in addition to the toolbar, which is less of an issue but still. 2 2 3 I'm using toggleClass on dragstart and dragstop instead of explicitly adding a class and removing it. This might cause issues state on some browsers if the events don't always fire in a predictable order, but it did work flawlessly on the ones I tested it with.3 I'm using toggleClass on dragstart and dragstop instead of explicitly adding a class and removing it. This might cause issues with state on some browsers if the events don't always fire in a predictable order, but it did work flawlessly on the ones I tested it with.