Bug 433706 - Option to move widgets within a page does not work
Summary: Option to move widgets within a page does not work
Status: RESOLVED FIXED
Alias: None
Product: plasma-systemmonitor
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Other Other
: NOR normal
Target Milestone: ---
Assignee: KSysGuard Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-02-27 15:57 UTC by Kishore Gopalakrishnan
Modified: 2021-03-10 16:03 UTC (History)
3 users (show)

See Also:
Latest Commit: https://invent.kde.org/plasma/plasma-systemmonitor/commit/bc994c071ed8826893b51664ab026f8ee7b2e038
Version Fixed In: 5.21.3

Attachments
What happens when I try to move a widget within a page (856.09 KB, video/x-matroska)
2021-02-27 15:57 UTC, Kishore Gopalakrishnan 		Details
View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
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