Review Request: PreviewJob supports webpages

Sebastian Kügler sebas at
Wed May 2 11:55:50 BST 2012

This is an automatically generated e-mail. To reply, visit:

(Updated May 2, 2012, 10:55 a.m.)

Review request for KDE Runtime and Dawit Alemayehu.


Update diff with fixes to Thomas' commment.


Make htmlcreator thumbnailer plugin support remote URLs

Requesting thumbnails for URLs has been added to kdelibs in 4.7. This patch makes the webcreator thumbnailer plugin support remote URLs as well.

We've been using a similar webcreator in Plasma Active, with this patch, our use cases would be covered by stock KDE.

Diffs (updated)

  kioslave/thumbnail/htmlcreator.cpp afbcea2 
  kioslave/thumbnail/htmlthumbnail.desktop 9cc4e02 



Used plugin through Plasma's preview engine for URLs, thumbnails of local html files also still work (tested through Dolphin).

I'm getting all-white images sometimes, the check for mainFrame()'s contentSize() mitigates this partly, but it still happens. As that's not a regression for local code, I'd like to get this patch merged. (When the error happens, loadFinished() still returns true, so it isn't caught by the m_loadedOk guard.) If anyone has suggestions how to handle these kinds of "silent failures", they would be most welcome :)


previews for bookmarks


Sebastian Kügler

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <>

More information about the kde-core-devel mailing list