Version: 1.9 (using KDE 3.5.0 Level "a" , SUSE 9.3 UNSUPPORTED) Compiler: gcc version 3.3.5 20050117 (prerelease) (SUSE Linux) OS: Linux (i686) release 2.6.11.4-21.9-default After upgrading to KDE 3.5 icons for toolbar buttons "Filter Classify as spam" and "Filter Classify as NOT spam" are missing.
Please check the settings for these filters (in the second tab page).
So? Yes, I can select there icon (actually I didn't find those icons yet) but bugreport was about that those icons were lost during update KDE 3.4.? -> 3.5 Why? When I was activating those filters then I didn't select icons manually - they were already there. So they should stay like that after update also??
*** Bug 117439 has been marked as a duplicate of this bug. ***
This appears to be happening because the icons got renamed from mark_as_spam.png to mail_spam.png, and the same for _ham. Probably you should create an upgrade script to rename the filter icons in user configurations automatically.
They are simply missing in the distribution, both in 3.5-rc1 and 3.5. I copied them from a previous distribution, restarted kmail. Easy enough workaround.
Small but annoying and very notable bug :-( Names of the icons changed and nothing took care of this :-/ I had to manually edit ~/.kde/share/config/kmailrc to replace "mark_as_spam" with "mail_spam" and "mark_as_ham" with "mail_ham" to get the icons back.
Created attachment 13990 [details] KMail filter icons auto-update patch Here's the patch I created for the Debian kdepim packages, implementing an auto-update of user configuration files for the new icon names.
Could Daniel's patch please be included into cvs!? It is not distro specific and breaking user configuration should be fixed.
Sorry for being insistant, but could you please include this before the freeze? The patch is fine, but I don't like too keep custom patches, which should be fixed upstream, forever. It's not like KDE has a good track record with updating user configuration data...
KDE4 now uses the icon name specification from freedesktop.org, so this patch is no longer correct.
So this is out of date.