291747 2012-01-17 09:06:45 +0000 Moving of multiple widgets 2014-01-16 07:23:03 +0000 1 1 2 Applications umbrello general unspecified Unlisted Binaries All RESOLVED FIXED NOR wishlist 2.12 (KDE 4.12) 1 ralf.habacker kde floeschie greatbunzinni kde ralf.habacker http://commits.kde.org/umbrello/87bb17d173927d590389fbef19b7edb1876d20d5 4.12.0 0 oldest_to_newest 1215435 0 ralf.habacker 2012-01-17 09:06:45 +0000 Version: unspecified (using Devel) OS: All Recent umbrello code allows multiple widgets to be selected, to be copied and pasted. Moving of multiple widgets is not supported. Reproducible: Didn't try Steps to Reproduce: Add some widgets, select them and try to move Actual Results: Only one widget will be selected and movable. Expected Results: All selected widgets should be movable. 1215487 1 ralf.habacker 2012-01-17 11:26:27 +0000 This feature seems to be implemented in the umbrello-soc branch, although it is not clear if this is a QGraphicsScene feature or simply added to main umbrello code. 1395688 2 82207 kde 2013-09-07 14:17:03 +0000 Created attachment 82207 Fix moving and undo-moving a selection of widgets 1395689 3 kde 2013-09-07 14:18:28 +0000 The current status of this issue in Umbrello 2.11.0 is almost right: it works as expected but undo works only for one of the selected items. On master though, there seems to be some regression: moving 5 items 200 pixels will cause only 1 item to move to move - by 5*200 pixels! Above patch fixes both issues. 1395694 4 andi.fischer 2013-09-07 15:18:00 +0000 *** Bug 298650 has been marked as a duplicate of this bug. *** 1395705 5 andi.fischer 2013-09-07 16:26:12 +0000 Git commit 87bb17d173927d590389fbef19b7edb1876d20d5 by Andi Fischer, on behalf of Joris Steyn. Committed on 07/09/2013 at 16:23. Pushed by fischer into branch 'master'. Fix moving and undo-moving a selection of widgets. The current status of this issue in Umbrello 2.11.0 is almost right: it works as expected but undo works only for one of the selected items. On master though, there seems to be some regression: moving 5 items 200 pixels will cause only 1 item to move to move - by 5*200 pixels! M +4 -17 umbrello/widgets/umlwidget.cpp http://commits.kde.org/umbrello/87bb17d173927d590389fbef19b7edb1876d20d5 1402498 6 kde 2013-10-09 20:08:34 +0000 *** Bug 279792 has been marked as a duplicate of this bug. *** 82207 2013-09-07 14:17:03 +0000 2013-09-07 14:17:03 +0000 Fix moving and undo-moving a selection of widgets umbrello-fix-move-selection-bug-291747.patch text/plain 1936 kde ZGlmZiAtLWdpdCBhL3VtYnJlbGxvL3dpZGdldHMvdW1sd2lkZ2V0LmNwcCBiL3VtYnJlbGxvL3dp ZGdldHMvdW1sd2lkZ2V0LmNwcAppbmRleCA0ZjhlYzQ5Li5lMDdjZmVmIDEwMDY0NAotLS0gYS91 bWJyZWxsby93aWRnZXRzL3VtbHdpZGdldC5jcHAKKysrIGIvdW1icmVsbG8vd2lkZ2V0cy91bWx3 aWRnZXQuY3BwCkBAIC0zNjIsMTcgKzM2Miw3IEBAIHZvaWQgVU1MV2lkZ2V0Ojptb3VzZU1vdmVF dmVudChRR3JhcGhpY3NTY2VuZU1vdXNlRXZlbnQqIG1lKQogICAgICAgICBkaWZmWSA9IDA7CiAg ICAgfQogCi0gICAgLyogQ29tbWFuZHMKLSAgICAgICAgVU1MRG9jKiBkb2MgPSBVTUxBcHA6OmFw cCgpLT5kb2N1bWVudCgpOwotICAgICAgICBkb2MtPmV4ZWN1dGVDb21tYW5kKG5ldyBDbWRNb3Zl V2lkZ2V0KHRoaXMsZGlmZlgsZGlmZlkpKTsqLwotCi0gICAgLy8gbW92ZVdpZGdldChkaWZmWCxk aWZmWSk7Ci0KLSAgICAvLyBERUJVRyhEQkdfU1JDKSA8PCAiYmVmb3JlIGNvbnN0cmFpbk1vdmVt ZW50Rm9yQWxsV2lkZ2V0czoiCi0gICAgLy8gICAgIDw8ICIgZGlmZlg9IiA8PCBkaWZmWCA8PCAi LCBkaWZmWT0iIDw8IGRpZmZZOwogICAgIGNvbnN0cmFpbk1vdmVtZW50Rm9yQWxsV2lkZ2V0cyhk aWZmWCwgZGlmZlkpOwotICAgIC8vIERFQlVHKERCR19TUkMpIDw8ICJhZnRlciBjb25zdHJhaW5N b3ZlbWVudEZvckFsbFdpZGdldHM6IgotICAgIC8vICAgICA8PCAiIGRpZmZYPSIgPDwgZGlmZlgg PDwgIiwgZGlmZlk9IiA8PCBkaWZmWTsKIAogICAgIC8vIG5vdGhpbmcgdG8gbW92ZQogICAgIGlm IChkaWZmWCA9PSAwICYmIGRpZmZZID09IDApIHsKQEAgLTM4NCwxMiArMzc0LDcgQEAgdm9pZCBV TUxXaWRnZXQ6Om1vdXNlTW92ZUV2ZW50KFFHcmFwaGljc1NjZW5lTW91c2VFdmVudCogbWUpCiAK ICAgICBERUJVRyhEQkdfU1JDKSA8PCAiZGlmZlg9IiA8PCBkaWZmWCA8PCAiIC8gZGlmZlk9IiA8 PCBkaWZmWTsKICAgICBmb3JlYWNoKFVNTFdpZGdldCogd2lkZ2V0LCBtX3NlbGVjdGVkV2lkZ2V0 c0xpc3QpIHsKLSAgICAgICAgUV9VTlVTRUQod2lkZ2V0KQotICAgICAgICAvL1VNTERvYyogbV9k b2MgPSBVTUxBcHA6OmFwcCgpLT5kb2N1bWVudCgpOwotICAgICAgICAvL0NtZE1vdmVXaWRnZXRC eSogY21kID0gbmV3IENtZE1vdmVXaWRnZXRCeSh3aWRnZXQsZGlmZlgsZGlmZlkpOwotICAgICAg ICAvL21fZG9jLT5leGVjdXRlQ29tbWFuZChjbWQpOwotICAgICAgICAvL21fZG9jLT5leGVjdXRl Q29tbWFuZChuZXcgQ21kTW92ZVdpZGdldEJ5KHdpZGdldCxkaWZmWCxkaWZmWSkpOwotICAgICAg ICBtb3ZlV2lkZ2V0QnkoZGlmZlgsIGRpZmZZKTsKKyAgICAgICAgd2lkZ2V0LT5tb3ZlV2lkZ2V0 QnkoZGlmZlgsIGRpZmZZKTsKICAgICB9CiAKICAgICAvLyBNb3ZlIGFueSBzZWxlY3RlZCBhc3Nv Y2lhdGlvbnMuCkBAIC01MjMsNyArNTA4LDkgQEAgdm9pZCBVTUxXaWRnZXQ6Om1vdXNlUmVsZWFz ZUV2ZW50KFFHcmFwaGljc1NjZW5lTW91c2VFdmVudCAqbWUpCiAgICAgfSBlbHNlIHsKICAgICAg ICAgLy8gQ29tbWFuZHMKICAgICAgICAgaWYgKG1fbW92ZWQpIHsKLSAgICAgICAgICAgIFVNTEFw cDo6YXBwKCktPmV4ZWN1dGVDb21tYW5kKG5ldyBVbWw6OkNtZE1vdmVXaWRnZXQodGhpcykpOwor ICAgICAgICAgICAgZm9yZWFjaChVTUxXaWRnZXQqIHdpZGdldCwgbV9zZWxlY3RlZFdpZGdldHNM aXN0KSB7CisgICAgICAgICAgICAgICAgVU1MQXBwOjphcHAoKS0+ZXhlY3V0ZUNvbW1hbmQobmV3 IFVtbDo6Q21kTW92ZVdpZGdldCh3aWRnZXQpKTsKKyAgICAgICAgICAgIH0KICAgICAgICAgICAg IG1fbW92ZWQgPSBmYWxzZTsKICAgICAgICAgfSBlbHNlIHsKICAgICAgICAgICAgIFVNTEFwcDo6 YXBwKCktPmV4ZWN1dGVDb21tYW5kKG5ldyBVbWw6OkNtZFJlc2l6ZVdpZGdldCh0aGlzKSk7Cg==