I added a few fixes here so that #3440 is still fixed after this request is merged.

Reassigned to @redmunds

@TomMalbran Sorry it's taken so long to get to this one. I started to review it, but there's a merge conflict. Can you merge the latest code? Thanks.

Sure, I'll fix the merge issue. Thanks for looking into this.

@redmunds Merged with master and fixed the conflict issue.

Done with initial review.

@redmunds Fixed the comment issues. Check on my replies about the event to see what we can do next. Thanks.

Done with second review.

I updated a bit the comment on the workingSetSort event and added a new workingSetDisableAutoSorting called every time 2 files are swapped when dragging and dropping them. This added a new event, but works better than before by not unnecessarily redrawing the working set after a drag/drop action. There might be a smarter way of figuring out when doing an automatic sort if it was previously resorted via a drag/drop action and avoid having the new event, but it is easier to do it like this.

Thanks! Looks good. Merging.

Note: I've reopened #2076 for more discussion since this doesn't appear to address the main confusion/pitfall I was concerned about (and in some sense the event name change may have made it worse).