| Summary: | *.desktop files, moved from FolderView widget, has incorrect icons | ||
|---|---|---|---|
| Product: | [Plasma] plasmashell | Reporter: | opensuse.lietuviu.kalba |
| Component: | Desktop icons & Folder View widget | Assignee: | Eike Hein <hein> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | bshah, kde, kde, mhruscak, plasma-bugs-null |
| Priority: | NOR | ||
| Version First Reported In: | 5.5.0 | ||
| Target Milestone: | 1.0 | ||
| Platform: | Other | ||
| OS: | Linux | ||
| Latest Commit: | http://commits.kde.org/plasma-workspace/7a54ad3dafc5cb689168b6d3dc00477e30f32d9b | Version Fixed/Implemented In: | |
| Sentry Crash Report: | |||
| Attachments: | Video | ||
|
Description
opensuse.lietuviu.kalba
2015-12-12 11:17:21 UTC
Created attachment 96007 [details]
Video
thanks When I open desktop:/ from Dolphin it works, so it might be FolderView doing something wrong/differently here? Do you mean live at runtime? It works fine in FV too if you reload or restart. Live when the applet is created. Works both ways with Dolphin. I'll take closer look tomorrow, thx *** Bug 360258 has been marked as a duplicate of this bug. *** Git commit 98de56e5ef14959d8e2bdc07bd69396e06a37e31 by Eike Hein. Committed on 09/03/2016 at 10:30. Pushed by hein into branch 'master'. Watch for and notify about XDG_DESKTOP_DIR changes. Will make KIO clients listing desktop:/ re-list at runtime. Makes the desktop not break when changing the desktop location in the user paths KCM. M +23 -0 kioslave/desktop/desktopnotifier.cpp M +4 -0 kioslave/desktop/desktopnotifier.h http://commits.kde.org/plasma-workspace/98de56e5ef14959d8e2bdc07bd69396e06a37e31 Git commit 7a54ad3dafc5cb689168b6d3dc00477e30f32d9b by Eike Hein. Committed on 09/03/2016 at 18:01. Pushed by hein into branch 'master'. Handle KIO URLs which proxy for local URLs in Icon applet. Dragging an app link .desktop file from Folder View showing desktop:/ from Folder View (but not Dolphin) to desktop/panel, spawning an Icon, was broken because: - KDirModel::metaData adds the UDS_LOCAL_PATH (local file) url as text/uri-list to the drag MIME data, but also the original desktop:/ URL as a second special MIME format for KDE URLs, using KUrlMimeData - Dolphin only does the first part, using a partial code copy from KDirModel - ContainmentInterface::processMimeData uses KUrlMimeData with default options to decode the drag MIME data - the default option is PreferKdeURLs, so it retrieves the desktop:/ URL from the FV drag but not the Dolphin drag (since it lacks it) - Thus the Icon applet is handed a desktop:/ URL in the FV case, which the code - not using KIO - isn't written to handle This patch adds an async KIO::StatJob to the Icon code when QURL::isLocalFile is fast to try and resolve an URL to a more local one the rest of the code can handle. Rewriting more of the Icon code to use KIO /would/ have been nice, however it ultimately ends up using KDesktopFile which can't handle non-local files, so at some point resolving to the proxied-for path would happen anyway. M +18 -0 applets/icon/plugin/icon_p.cpp M +2 -0 applets/icon/plugin/icon_p.h http://commits.kde.org/plasma-workspace/7a54ad3dafc5cb689168b6d3dc00477e30f32d9b This patch will afect KDE Plasma 5.6? Git commit 48f260df95dfb0115d44ab73d0731119bea901e9 by Eike Hein. Committed on 09/03/2016 at 19:00. Pushed by hein into branch 'Plasma/5.6'. Watch for and notify about XDG_DESKTOP_DIR changes. Will make KIO clients listing desktop:/ re-list at runtime. Makes the desktop not break when changing the desktop location in the user paths KCM. M +23 -0 kioslave/desktop/desktopnotifier.cpp M +4 -0 kioslave/desktop/desktopnotifier.h http://commits.kde.org/plasma-workspace/48f260df95dfb0115d44ab73d0731119bea901e9 Git commit 7d60594111ebc075bbd60d3e8d1fcdcdb0f3c139 by Eike Hein. Committed on 09/03/2016 at 19:01. Pushed by hein into branch 'Plasma/5.6'. Handle KIO URLs which proxy for local URLs in Icon applet. Dragging an app link .desktop file from Folder View showing desktop:/ from Folder View (but not Dolphin) to desktop/panel, spawning an Icon, was broken because: - KDirModel::metaData adds the UDS_LOCAL_PATH (local file) url as text/uri-list to the drag MIME data, but also the original desktop:/ URL as a second special MIME format for KDE URLs, using KUrlMimeData - Dolphin only does the first part, using a partial code copy from KDirModel - ContainmentInterface::processMimeData uses KUrlMimeData with default options to decode the drag MIME data - the default option is PreferKdeURLs, so it retrieves the desktop:/ URL from the FV drag but not the Dolphin drag (since it lacks it) - Thus the Icon applet is handed a desktop:/ URL in the FV case, which the code - not using KIO - isn't written to handle This patch adds an async KIO::StatJob to the Icon code when QURL::isLocalFile is fast to try and resolve an URL to a more local one the rest of the code can handle. Rewriting more of the Icon code to use KIO /would/ have been nice, however it ultimately ends up using KDesktopFile which can't handle non-local files, so at some point resolving to the proxied-for path would happen anyway. M +18 -0 applets/icon/plugin/icon_p.cpp M +2 -0 applets/icon/plugin/icon_p.h http://commits.kde.org/plasma-workspace/7d60594111ebc075bbd60d3e8d1fcdcdb0f3c139 > This patch will afect KDE Plasma 5.6?
Yes.
|