BUG: 402574
When we're moving a bunch of icons to the area which is already busy, we didn't check whether we're moving icons to different places.
Now we check additionally if we're moving icon to the place already taken by previously moved icons.
hein |
Plasma |
BUG: 402574
When we're moving a bunch of icons to the area which is already busy, we didn't check whether we're moving icons to different places.
Now we check additionally if we're moving icon to the place already taken by previously moved icons.
Lint Skipped |
Unit Tests Skipped |
Looks sensible to me. Good stuff.
Wait a week to allow Eike a chance to comment, otherwise consider this a ship it!
containments/desktop/plugins/folder/positioner.cpp | ||
---|---|---|
418 | AFAIK, the first part of condition is "check if to is blank and if it's not, check whether we're not moving icon to itself, and if we do, it's alright" |