Review Request: Make web shortcut icons look better in krunner

Michael Pyne mpyne at purinchu.net
Mon Feb 9 03:34:45 CET 2009


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/47/
-----------------------------------------------------------

Review request for Plasma.


Summary
-------

Right now the icons on some web shortcut searches look poor, (for instance, our love kde: shortcut gives me a question mark icon).  I see that there is code to give a sane default for icons with no favicon, but it doesn't seem to work here.  Using the kdelibs KMimeType instead of direct DBus calls seems to fix that.

In addition since most favicons are small and look poor scaled up by the runner, I added the ability to obey the Icon= setting of the .desktop files defining a web shortcut.  (For instance, try adding Icon=kde to the kde.desktop file in kdebase/runtime/kurifilter-plugins/ikws/searchproviders or installed at `kde4-config --install services`/searchproviders/kde.desktop.  Now you get the very nice glassy KDE logo when you type kde:KFoo in krunner).


Diffs
-----

  kdebase/workspace/plasma/runners/webshortcuts/webshortcutrunner.cpp 923467 

Diff: http://reviewboard.kde.org/r/47/diff


Testing
-------

I've tested gg: (which has a favicon), kde: (whose Icon= overrides) and imdb: (no favicon, nice default used instead of the question mark).


Thanks,

Michael



More information about the Plasma-devel mailing list