[Differential] [Commented On] D4157: [Icon Applet] Use KIO::statJob to work with remote URLs

[David Faure]

dfaure added inline comments.

INLINE COMMENTS

> iconapplet.cpp:159
>  
> -            return;
> +        if (name.isEmpty() && url.scheme().startsWith(QLatin1String("http"))) {
> +            name = url.host();

Why only http?

This would be useful for FTP, SMB and many other things, no?

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D4157

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: broulik, #plasma, mart, dfaure
Cc: plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170119/6d0d9e0e/attachment-0001.html>