Summary: | Recently accessed items in Dolphin (KDE 4.9.0) uses OpenSuSE specific package_utility_time icon | ||
---|---|---|---|
Product: | [Applications] dolphin | Reporter: | Bastian Beischer <bastian.beischer> |
Component: | panels: places | Assignee: | Dolphin Bug Assignee <dolphin-bugs-null> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | adaptee, bastian.beischer, johu, kamikazow, kde, mail, ulilicht, wengxt |
Priority: | NOR | ||
Version: | 2.1.80 | ||
Target Milestone: | --- | ||
Platform: | Arch Linux | ||
OS: | Linux | ||
URL: | http://kde.org/announcements/announce-4.10-beta1.png | ||
Latest Commit: | http://commits.kde.org/kde-baseapps/d86ec02b684fad73f7aa55654681773efd851709 | Version Fixed In: | 4.9.2 |
Sentry Crash Report: | |||
Bug Depends on: | |||
Bug Blocks: | 303462 | ||
Attachments: |
Screenshot showing the correct Nepomuk places in Dolphin
Icons still missing in KDE 4.11 |
Description
Bastian Beischer
2012-07-31 10:24:01 UTC
This problem does also exist on Gentoo (and probably any other non-OpenSuSE distribution, too). The icon package_utility_time.png cannot be found in any KDE release and googling for it shows only OpenSuSE related matches, so it looks like the author wrongly picked an OpenSuSE specific icon of which he might have thought it would be an KDE icon. KDE provides the icon actions/chronometer which looks similar and could be used here. Thanks for the bug report! I agree that this should be changed. Git commit d7e7ca53bb95c7555bbf107d92b47ac25eda1918 by Frank Reininghaus. Committed on 09/08/2012 at 23:38. Pushed by freininghaus into branch 'KDE/4.9'. Use a better icon for recently accessed items in the Places Panel The problem with the icon "package_utility_time" that has been used before was that it is not available on all distros. The new icon "actions/chronometer" should be available on every KDE install. Thanks to Elias Probst for suggesting the new icon - I'm not that familiar with the icons provided by KDE myself. FIXED-IN: 4.9.1 M +1 -1 dolphin/src/panels/places/placesitem.cpp http://commits.kde.org/kde-baseapps/d7e7ca53bb95c7555bbf107d92b47ac25eda1918 Git commit a2deeadb7ad5b0428702e6fd434175d861b82944 by Frank Reininghaus. Committed on 09/08/2012 at 23:52. Pushed by freininghaus into branch 'KDE/4.9'. Follow-up to d7e7ca53bb95c7555bbf107d92b47ac25eda1918 - use correct icon I overlooked that "actions/" is not part of the icon name. Just rebuilding and re-running Dolphin did not reveal this issue, it seems that the old icon "package_utility_time" was still cached somewhere. I hope that it really works on all distros now. M +1 -1 dolphin/src/panels/places/placesitem.cpp http://commits.kde.org/kde-baseapps/a2deeadb7ad5b0428702e6fd434175d861b82944 Git commit a97f5d3a10483a3a71847d6459c96ef6dcc9332a by Frank Reininghaus. Committed on 09/08/2012 at 23:38. Pushed by freininghaus into branch 'master'. Use a better icon for recently accessed items in the Places Panel The problem with the icon "package_utility_time" that has been used before was that it is not available on all distros. The new icon "chronometer" should be available on every KDE install. Thanks to Elias Probst for suggesting the new icon - I'm not that familiar with the icons provided by KDE myself. (cherry picked from commit d7e7ca53bb95c7555bbf107d92b47ac25eda1918) M +1 -1 dolphin/src/panels/places/placesitem.cpp http://commits.kde.org/kde-baseapps/a97f5d3a10483a3a71847d6459c96ef6dcc9332a *** Bug 305216 has been marked as a duplicate of this bug. *** David found out (see bug 305216) that the incorrect icon name is used in more places than I thought. I think we should just replace also the remaining "package_utility_time" by "chronometer". Resetting assignee to default as per bug #305719 Git commit d86ec02b684fad73f7aa55654681773efd851709 by Frank Reininghaus. Committed on 11/09/2012 at 19:25. Pushed by freininghaus into branch 'KDE/4.9'. Use a better icon for recently accessed items, part 2 I had missed one places where the incorrect icon was used in my previous commit d7e7ca53bb95c7555bbf107d92b47ac25eda1918. FIXED-IN: 4.9.2 M +1 -2 dolphin/src/panels/places/placesitemmodel.cpp http://commits.kde.org/kde-baseapps/d86ec02b684fad73f7aa55654681773efd851709 Looking at the announcement screenshot of KDE 4.10 beta 1, it looks like this bug does still exist: http://kde.org/announcements/announce-4.10-beta1.png REOPENing this bug… grep shows it's not using this. Also it works fine as a new user. I think if you had 4.9 then upgrade, you'll still have the broken icons. In KDE 4.10, we use different icons for the different 'timeline' places: https://git.reviewboard.kde.org/r/106874/ AFAIK, they are in oxygen-icons. I can't test it at the moment, so I don't know what's going wrong. Do we actually have any evidence that this bug is reproducible except for that screenshot? (In reply to comment #12) > Do we actually have any evidence that this bug is reproducible except for that screenshot? I'm just building 4.9.80/4.10 beta1 which might take ~3 hours, then I'll be able to confirm. The problem persists with 4.9.80 using my already existing account. Cannot reproduce using a new account, as the corresponding items don't appear in Dolphin's places panel at all, although Nepomuk is enabled, running, finished indexing… I even re-started the KDE session. Does anyone know why? OK, I can confirm. If the 'old' icons are in kfileplaces/bookmarks.xml, the 'new' icons are not applied. You can work around the issue manually by removing the "timeline:" entries from that file, but I see that this should better be fixed inside Dolphin. In principle, it would be better if we opened a new report about that, because this is unrelated to the issue that this report was about. Recycling of old reports for new problems makes bug reports unreadable in the long term. About the problem you see when you create a new user: no idea, sorry. Thinking about it again, I think that there is no good solution for the problem. If we change the icons in kfileplaces/bookmarks.xml, we also overwrite any custum icons that the user has chosen for the "timeline" entries. This is obviously not an option. (In reply to comment #14) > Cannot reproduce using a new account, as the corresponding items don't > appear in Dolphin's places panel at all, although Nepomuk is enabled, > running, finished indexing… I even re-started the KDE session. > > Does anyone know why? Try to disable and then re-enable indexing, see https://git.reviewboard.kde.org/r/107464/. A fix for that issue has already been provided and will be committed soon. Created attachment 75500 [details] Screenshot showing the correct Nepomuk places in Dolphin (In reply to comment #17) > Try to disable and then re-enable indexing, see > https://git.reviewboard.kde.org/r/107464/. A fix for that issue has already > been provided and will be committed soon. Using this patch now. After restarting Dolphin I could see the correct icons (see attached screenshot). So this issue only remains for existing KDE profiles… Thanks. Does that mean that you get the correct 'Places' icons for a) the new user, b) the existing user, or both? (In reply to comment #19) > Thanks. Does that mean that you get the correct 'Places' icons for > > a) the new user, > b) the existing user, > > or both? Sorry, I wasn't clear enough: # for a new user: → Start KDE → Start Dolphin → No timeline icons/entries are shown at all → Close Dolphin → Wait until Nepomuk starts indexing → Start Dolphin again → *Correct* timeline entries/icons are shown # for an existing user → Start KDE → Start Dolphin → *Wrong* timeline icons are shown for the timeline entries → Close Dolphin → Remove 'timeline' entries from ~/.kde4/share/apps/kfileplaces/bookmarks.xml → Start Dolphin again → *Correct* timeline entries/icons are shown The question is, whether it is possible to make use of kconf_update to remove the old timeline entries which make use of the wrong icon…?! (In reply to comment #20) > The question is, whether it is possible to make use of kconf_update to > remove the old timeline entries which make use of the wrong icon…?! The problem is that this would also overwrite any custom icon choices that the user made for these entries, so I think that we cannot do that. Closing this report again because the issue that has originally been reported here is fixed. (In reply to comment #16) > Thinking about it again, I think that there is no good solution for the > problem. If we change the icons in kfileplaces/bookmarks.xml, we also > overwrite any custum icons that the user has chosen for the "timeline" > entries. This is obviously not an option. And why can't the chronometer icon just be used for migrated accounts? Was the icon removed from Oxygen or what? If yes: Poke the Oxygen team to re-add it. (In reply to comment #22) > And why can't the chronometer icon just be used for migrated accounts? If I'm not mistaken, the chronometer icon is being used for migrated accounts. If an earlier version stored that icon name in kfileplaces/bookmarks.xml, then it should also be used by a newer version. Created attachment 81779 [details]
Icons still missing in KDE 4.11
Should this be resolved now? I am using KDE 4.11 fromt the Kubuntu Backports PPA, the Icons are still missing. (see attachment) Please note: On the screenshot, you can see that i am using KFaenza Icon Theme. The Problem also exists if i switch to oxygen and restart dolphin. Cheers, Uli (In reply to comment #25) > Should this be resolved now? I am using KDE 4.11 fromt the Kubuntu Backports > PPA, the Icons are still missing. (see attachment) Please see the earlier comments 15 and 16 in this report. The incorrect icons are stored in a file in your home directory. Upgrading to a newer KDE version does not change anything about that - you have to edit the file. Sorry for the inconvenience. Thanks for your kind answer, that helped! I deleted all <bookmark...> Entries from the HOME/.kde/share/apps/kfileplaces/bookmarks.xml file (In reply to comment #26) > (In reply to comment #25) > > Should this be resolved now? I am using KDE 4.11 fromt the Kubuntu Backports > > PPA, the Icons are still missing. (see attachment) > > Please see the earlier comments 15 and 16 in this report. The incorrect > icons are stored in a file in your home directory. Upgrading to a newer KDE > version does not change anything about that - you have to edit the file. > Sorry for the inconvenience. SRY, i meant <bookmark href="timeline:...."> ;-) It is important that dolphin is closed while doing this. (In reply to comment #27) > Thanks for your kind answer, that helped! I deleted all <bookmark...> > Entries from the > HOME/.kde/share/apps/kfileplaces/bookmarks.xml > file > > > (In reply to comment #26) > > (In reply to comment #25) > > > Should this be resolved now? I am using KDE 4.11 fromt the Kubuntu Backports > > > PPA, the Icons are still missing. (see attachment) > > > > Please see the earlier comments 15 and 16 in this report. The incorrect > > icons are stored in a file in your home directory. Upgrading to a newer KDE > > version does not change anything about that - you have to edit the file. > > Sorry for the inconvenience. |