https://bugs.kde.org/show_bug.cgi?id=356548

Eike Hein <h...@kde.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|http://commits.kde.org/plas |http://commits.kde.org/plas
                   |ma-workspace/7a54ad3dafc5cb |ma-workspace/7d60594111ebc0
                   |689168b6d3dc00477e30f32d9b  |75bbd60d3e8d1fcdcdb0f3c139

--- Comment #11 from Eike Hein <h...@kde.org> ---
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

--- Comment #12 from Eike Hein <h...@kde.org> ---
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

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to