Propagate mouseup events when not dragging toolbar

djsutho · djsutho · commit decf92eb045a · 2014-02-25T14:55:23.000+10:00
In Chrome (this does not affect Firefox) onchange events on multi-selects are not fired all of the time e.g. with the following template:

&lt;body&gt;
    &lt;select multiple="multiple" onchange="console.log('clicked');"&gt;
        &lt;option  value="1"&gt;1&lt;/option&gt;
    &lt;/select&gt;
&lt;/body&gt;

Ctrl click on the option and view the js log. many of the events will not register. Therefore stop propagation and prevent default action only when handle is dragged.
diff --git a/debug_toolbar/static/debug_toolbar/js/toolbar.js b/debug_toolbar/static/debug_toolbar/js/toolbar.js
@@ -160,8 +160,8 @@
                     setTimeout(function () {
                         djdt.handleDragged = false;
                     }, 10);
+                    return false;
                 }
-                return false;
             });
             $(document).bind('close.djDebug', function() {
                 // If a sub-panel is open, close that
