<table><tr><td style="">vkrause added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D18731">View Revision</a></tr></table><br /><div><div><p>Tested Konqueror bookmarks. New bookmarks don't seem to get the favicon set until you explicitly trigger the "Update Favicon" action, afterwards they have it and it remains across restarts. So this seems to work as expected.<br />
Regarding the indirect dependencies, yes, this does not actually change much on its own. I run into this while debugging icon loading issues in Okular on Android, which seems related to KIconThemes interfering, so less places where I need to strip it out help.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R294 KBookmarks</div></div></div><br /><div><strong>BRANCH</strong><div><div>master</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D18731">https://phabricator.kde.org/D18731</a></div></div><br /><div><strong>To: </strong>vkrause, davidedmundson<br /><strong>Cc: </strong>broulik, cfeck, kde-frameworks-devel, michaelh, ngraham, bruns<br /></div>