Bug 291747

Summary: Moving of multiple widgets
Product: [Applications] umbrello Reporter: Ralf Habacker <ralf.habacker>
Component: generalAssignee: Joris Steyn <kde>
Status: RESOLVED FIXED    
Severity: wishlist CC: floeschie, greatbunzinni, kde, ralf.habacker
Priority: NOR    
Version: unspecified   
Target Milestone: 2.12 (KDE 4.12)   
Platform: Unlisted Binaries   
OS: All   
Latest Commit: Version Fixed In: 4.12.0
Attachments: Fix moving and undo-moving a selection of widgets

Description Ralf Habacker 2012-01-17 09:06:45 UTC
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.
Comment 1 Ralf Habacker 2012-01-17 11:26:27 UTC
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.
Comment 2 Joris Steyn 2013-09-07 14:17:03 UTC
Created attachment 82207 [details]
Fix moving and undo-moving a selection of widgets
Comment 3 Joris Steyn 2013-09-07 14:18:28 UTC
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.
Comment 4 Andi Fischer 2013-09-07 15:18:00 UTC
*** Bug 298650 has been marked as a duplicate of this bug. ***
Comment 5 Andi Fischer 2013-09-07 16:26:12 UTC
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
Comment 6 Joris Steyn 2013-10-09 20:08:34 UTC
*** Bug 279792 has been marked as a duplicate of this bug. ***