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