D8454: Make Saved Search feature discoverable
Elvis Angelaccio
noreply at phabricator.kde.org
Wed Nov 1 14:41:13 GMT 2017
elvisangelaccio added inline comments.
INLINE COMMENTS
> ngraham wrote in dolphincontextmenu.cpp:377
> We actually don't since we're no longer using `KIO::iconNameForUrl()` to get the icon for saved searches. Instead, as @anthonyfieroni suggested, we always hardcode the "folder-saved-search-symbolic" icon for saved searches, because even if `KIO::iconNameForUrl()` were fixed, it would still return a different icon (probably "file-search-symbolic") which may be inappropriate in certain icon themes.
Then we should remove the FIXME comment below? (dolphinsearchbox.cpp line 313)
REPOSITORY
R318 Dolphin
REVISION DETAIL
https://phabricator.kde.org/D8454
To: ngraham, #dolphin, #kde_applications, broulik, dfaure, markg, emateli, elvisangelaccio
Cc: anthonyfieroni, markg, emateli, elvisangelaccio, cfeck, #dolphin, spoorun, navarromorales, firef, ngraham, andrebarros, emmanuelp
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20171101/0fb911d2/attachment.htm>
More information about the kfm-devel
mailing list