Fix Bug 299565 - Unnecessary repainting on mouse over when directory content is changing
Review Request #111146 - Created June 20, 2013 and submitted
Avoid a unnecessary resorting when items are changed. Only resort the items when the sorting role changes.
Works for me
Good debugging! Just two small inline suggestions, if you think that they make sense, you can include them in your commit. Thanks!
I think that you can move this assignment out of the loop, right? I don't know if the compiler is clever enough to see that.
Now that you've determined the changedRoles, why not use them here? This might even save some work in other places (like when only the size and date of a file change when it's updated, but these are not shown at all in the view -> saves a relayouting).