Bug 456273 - 'how_to_install.txt' unexpectedly appears in the grid after installing Simp1e cursor theme
Summary: 'how_to_install.txt' unexpectedly appears in the grid after installing Simp1e...
Status: RESOLVED FIXED
Alias: None
Product: systemsettings
Classification: Applications
Component: kcm_cursortheme (other bugs)
Version First Reported In: master
Platform: Neon Linux
: NOR normal
Target Milestone: ---
Assignee: Alexander Lohnau
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-07-03 10:33 UTC by Patrick Silva
Modified: 2022-10-05 08:43 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed/Implemented In: 5.26
Sentry Crash Report:


Attachments
screen recording (2.02 MB, video/webm)
2022-07-03 10:33 UTC, Patrick Silva
Details
screenshot after installing 'Radioactive' cursor theme (197.93 KB, image/png)
2022-09-18 02:53 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-07-03 10:33:25 UTC
Created attachment 150360 [details]
screen recording

SUMMARY
'how_to_install.txt' unexpectedly appears in the grid after installing a cursor theme, and it disappears after reopening Cursors KCM. Please watch the attached screen recording.

EXPECTED RESULT
only installed cursor themes should appear in the grid

Operating System: KDE neon Unstable Edition
KDE Plasma Version: 5.25.80
KDE Frameworks Version: 5.96.0
Qt Version: 5.15.5
Graphics Platform: Wayland
Comment 1 Nate Graham 2022-07-05 17:45:01 UTC
Can reproduce with the Simp1e cursor theme. I suspect that this cursor theme is malformed, but we should be more robust about handling it, since this is user-generated content and who knows what bizarre stuff people will stick in there...
Comment 2 Patrick Silva 2022-09-18 02:53:07 UTC
Created attachment 152177 [details]
screenshot after installing 'Radioactive' cursor theme

Possibly related to this bug: after installing the cursor theme called Radioactive from KDE Store, two new entries called 'cursors'
and 'index.theme' seen in the screenshot attached to this comment appear in the grid, but them disappear after reopening Cursors KCM. 

Operating System: KDE neon Unstable Edition
KDE Plasma Version: 5.26.80
KDE Frameworks Version: 5.98.0
Qt Version: 5.15.6
Graphics Platform: Wayland
Comment 3 Alexander Lohnau 2022-09-18 06:52:30 UTC
>Possibly related to this bug: after installing the cursor theme called Radioactive from KDE Store, two new entries called 'cursors'
>and 'index.theme' seen in the screenshot attached to this comment appear in the grid, but them disappear after reopening Cursors KCM. 

It appears that they are extracted to the root of the icons folder and can consequently not be loaded.
Comment 4 Bug Janitor Service 2022-09-18 08:10:24 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/2137
Comment 5 Ben Cooksley 2022-09-30 18:59:06 UTC
Git commit 63df5f0f815cf0db3d754f826062cf923568365a by Ben Cooksley, on behalf of Alexander Lohnau.
Committed on 30/09/2022 at 18:40.
Pushed by bcooksley into branch 'master'.

kcms/cursortheme: Make code for new KNS entries more simple & robust

We always know this has to be a dir, otherwise we have regressions in KNS.

M  +9    -13   kcms/cursortheme/kcmcursortheme.cpp

https://invent.kde.org/plasma/plasma-workspace/commit/63df5f0f815cf0db3d754f826062cf923568365a
Comment 6 Ben Cooksley 2022-09-30 18:59:14 UTC
Git commit 3c5457124a7a854e10f07ceed6a34f9ab05a094a by Ben Cooksley, on behalf of Alexander Lohnau.
Committed on 30/09/2022 at 18:40.
Pushed by bcooksley into branch 'master'.

xcursor: Fix some cursor themes from KDE Store not being installed in subdir

M  +1    -1    kcms/cursortheme/xcursor/xcursor.knsrc

https://invent.kde.org/plasma/plasma-workspace/commit/3c5457124a7a854e10f07ceed6a34f9ab05a094a
Comment 7 Nate Graham 2022-10-05 07:02:49 UTC
Git commit 476029acbc433e3415f49c3a322a8fe619619de5 by Nate Graham, on behalf of Alexander Lohnau.
Committed on 05/10/2022 at 07:02.
Pushed by ngraham into branch 'Plasma/5.26'.

kcms/cursortheme: Make code for new KNS entries more simple & robust

We always know this has to be a dir, otherwise we have regressions in KNS.


(cherry picked from commit 63df5f0f815cf0db3d754f826062cf923568365a)

M  +9    -13   kcms/cursortheme/kcmcursortheme.cpp

https://invent.kde.org/plasma/plasma-workspace/commit/476029acbc433e3415f49c3a322a8fe619619de5
Comment 8 Ben Cooksley 2022-10-05 08:43:13 UTC
Git commit 76f53b026087b94d41f6f1478080d0770db55e17 by Ben Cooksley, on behalf of Alexander Lohnau.
Committed on 05/10/2022 at 08:43.
Pushed by bcooksley into branch 'Plasma/5.26'.

xcursor: Fix some cursor themes from KDE Store not being installed in subdir
(cherry picked from commit 3c5457124a7a854e10f07ceed6a34f9ab05a094a)

M  +1    -1    kcms/cursortheme/xcursor/xcursor.knsrc

https://invent.kde.org/plasma/plasma-workspace/commit/76f53b026087b94d41f6f1478080d0770db55e17