Bug 433706

Summary: Option to move widgets within a page does not work
Product: [Applications] plasma-systemmonitor Reporter: Kishore Gopalakrishnan <kishore96>
Component: generalAssignee: KSysGuard Developers <ksysguard-bugs>
Status: RESOLVED FIXED    
Severity: normal CC: ahiemstra, nate, plasma-bugs
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: Other   
OS: Other   
Latest Commit: Version Fixed In: 5.21.3
Attachments: What happens when I try to move a widget within a page

Description Kishore Gopalakrishnan 2021-02-27 15:57:32 UTC
Created attachment 136226 [details]
What happens when I try to move a widget within a page

SUMMARY
The option to move a row of widgets in a system monitor page does not appear to have any effect.

STEPS TO REPRODUCE
1. Click on 'Edit page'
2. Click on the outermost box that appears on a widget
3. Click on the button that says 'move', and without releasing the mouse button, attempt to drag the widget up/down within the same page

OBSERVED RESULT
The widget moves along with the cursor, but it goes back to its initial position when you release the cursor (see attached video)

EXPECTED RESULT
The widget should be moveable using the 'move' button.

SOFTWARE/OS VERSIONS
Operating System: Arch Linux
KDE Plasma Version: 5.21.1
KDE Frameworks Version: 5.79.0
Qt Version: 5.15.2
Kernel Version: 5.10.17-1-lts

ADDITIONAL INFORMATION
Comment 1 Arjen Hiemstra 2021-03-10 12:41:36 UTC
Git commit 339675bf543160cc7e8aa516fc1c9dec9492ebe7 by Arjen Hiemstra.
Committed on 10/03/2021 at 12:41.
Pushed by ahiemstra into branch 'master'.

Do not call a non-existing function

Apparently the function was renamed but the calling code was not, which
broke moving items on a page.

M  +1    -1    src/page/MoveButton.qml

https://invent.kde.org/plasma/plasma-systemmonitor/commit/339675bf543160cc7e8aa516fc1c9dec9492ebe7
Comment 2 Arjen Hiemstra 2021-03-10 12:42:18 UTC
Git commit bc994c071ed8826893b51664ab026f8ee7b2e038 by Arjen Hiemstra.
Committed on 10/03/2021 at 12:42.
Pushed by ahiemstra into branch 'Plasma/5.21'.

Do not call a non-existing function

Apparently the function was renamed but the calling code was not, which
broke moving items on a page.


(cherry picked from commit 339675bf543160cc7e8aa516fc1c9dec9492ebe7)

M  +1    -1    src/page/MoveButton.qml

https://invent.kde.org/plasma/plasma-systemmonitor/commit/bc994c071ed8826893b51664ab026f8ee7b2e038