Reorderable should not allow dragging with the right mouse button

[mstange]

If you right-click on a thread name and drag down, the context menu opens and at the same time you're dragging the thread.
You shouldn't be dragging the thread.

[julienw]

If a contributor would like to try this, the fix is adding a check for event.button in https://github.com/devtools-html/perf.html/blob/master/src/content/components/Reorderable.js#L42

[morrme]

Would the check be a part of the current conditional? and would it be for event.button == 1 ?

[julienw]

I'd do a if (event.button !== 0) { return } (with proper indentations :) ), separarely from the existing conditional for the sake of clarity.

[magopian]

I believe this can now be closed as it was fixed in 70f3cd3

[julienw]

true :)