Bug 453226 - Impossible to remove a cursor theme installed from KDE Store
Summary: Impossible to remove a cursor theme installed from KDE Store
Status: RESOLVED FIXED
Alias: None
Product: systemsettings
Classification: Applications
Component: kcm_cursortheme (show other bugs)
Version: 5.24.90
Platform: Neon Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords: regression
Depends on:
Blocks:
 
Reported: 2022-04-30 11:36 UTC by Patrick Silva
Modified: 2022-05-25 20:28 UTC (History)
4 users (show)

See Also:
Latest Commit:
Version Fixed In: 5.25
Sentry Crash Report:


Attachments
screen recording (975.87 KB, video/webm)
2022-04-30 11:36 UTC, Patrick Silva
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Patrick Silva 2022-04-30 11:36:56 UTC
Created attachment 148472 [details]
screen recording

SUMMARY
After click on inline 'Remove' button of  a cursor theme installed from KDE Store, apparently the cursor theme is removed. But the cursor theme will still be available for selection next time I open Cursors KCM.
Please watch the attached screen recording.

EXPECTED RESULT
Cursors KCM is able to remove cursor themes

SOFTWARE/OS VERSIONS
Operating System: KDE neon Unstable Edition
KDE Plasma Version: 5.24.80
KDE Frameworks Version: 5.94.0
Qt Version: 5.15.3
Graphics Platform: X11

ADDITIONAL INFORMATION
also it's impossible to remove a cursor theme via 'Get new cursors' window
Comment 1 Nate Graham 2022-05-03 14:53:18 UTC
Can reproduce.
Comment 2 Alexander Lohnau 2022-05-03 16:27:51 UTC
On the console I get:
file:///home/user/kde/usr/share/kpackage/kcms/kcm_cursortheme/contents/ui/main.qml: Writing to "pendingDeletion" broke the binding to the underlying model


Reverting f3293bc59e1d4998579ad64a908482a54f2ad5ae fixes the issue.
Comment 3 Bug Janitor Service 2022-05-21 08:46:21 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/1753
Comment 4 Alexander Lohnau 2022-05-21 14:24:33 UTC
Git commit 90d36545eaa4deab38f70d127740af5373152006 by Alexander Lohnau.
Committed on 21/05/2022 at 14:15.
Pushed by alex into branch 'master'.

Revert "[kcms/cursor] Port to required properties"

We can not use the required properties, because it breaks the property binding.
https://github.com/qt/qtdeclarative/commit/ec79af7396fd695b25f8f7dba66e8048d54b953b apparently fixes that, however
it is not released as part of Qt6 or is part of the KDE patch collection.

This reverts commit f3293bc59e1d4998579ad64a908482a54f2ad5ae.

M  +9    -15   kcms/cursortheme/package/contents/ui/Delegate.qml

https://invent.kde.org/plasma/plasma-workspace/commit/90d36545eaa4deab38f70d127740af5373152006
Comment 5 Alexander Lohnau 2022-05-21 14:39:36 UTC
Git commit 628f7798395efcbd3382008addce036f4bbc4c19 by Alexander Lohnau.
Committed on 21/05/2022 at 14:39.
Pushed by alex into branch 'Plasma/5.25'.

Revert "[kcms/cursor] Port to required properties"

We can not use the required properties, because it breaks the property binding.
https://github.com/qt/qtdeclarative/commit/ec79af7396fd695b25f8f7dba66e8048d54b953b apparently fixes that, however
it is not released as part of Qt6 or is part of the KDE patch collection.

This reverts commit f3293bc59e1d4998579ad64a908482a54f2ad5ae.


(cherry picked from commit 90d36545eaa4deab38f70d127740af5373152006)

M  +9    -15   kcms/cursortheme/package/contents/ui/Delegate.qml

https://invent.kde.org/plasma/plasma-workspace/commit/628f7798395efcbd3382008addce036f4bbc4c19