| Summary: | gnome-icon-theme icons do not appear correctly | ||
|---|---|---|---|
| Product: | TDE | Reporter: | Darrell <darrella> |
| Component: | tdebase | Assignee: | Timothy Pearson <kb9vqf> |
| Status: | NEW --- | ||
| Severity: | major | CC: | bugwatch, darrella, kb9vqf |
| Priority: | P5 | ||
| Version: | R14.0.x [Trinity] | ||
| Hardware: | Other | ||
| OS: | Linux | ||
| Compiler Version: | TDE Version String: | ||
| Application Version: | Application Name: | ||
| Bug Depends on: | 1313 | ||
| Bug Blocks: | |||
| Attachments: |
Screenshot of system tray icons with gnome-icon-theme icons
Screenshot of desktop device icons before suggested changes Screenshot of desktop device icons after suggested changes |
||
|
Description
Darrell
2014-12-15 16:25:54 CST
With a fresh profile, the following icons did not appear correctly: Desktop ------- My Documents Web Browser Unmounted NFS share Mounted removable medium System Tray ----------- krandr tdenetworkmanager tdepowersave klipper kalarm akregator kget kmix, kmail, and korganizer icons are correct but are the crystalsvg defaults. I expect icon sets like gnome-icon-theme not to have any respective app icons, but I also expect the default TDE icons to appear. Interestingly, I got busy with something else for several minutes and when I returned to the fresh desktop the Desktop icons had self-adjusted to TDE icons. The system tray icons were still generic. That the Desktop icons self-adjusted is great but I don't know why that did not happen when the desktop first initialized. Created attachment 2397 [details] Screenshot of system tray icons with gnome-icon-theme icons Screenshot attached. I don't know how to resolve these icon bugs except ensuring they all default to TDE icons. Digging deeper indicates the gnome-icon being displayed probably is mimetypes/text-x-generic.png, which probably is the equivalent of TDE's application-vnd.tde.misc.png. I am guessing the system tray icon problem is a code bug --- refer to the patch used in commit 6209a28 for bug 1313. Comparing the icon names used in MATE and TDE, the following fixes would be more portable and be more xdg compliant or at least non TDE friendly. These fixes would provide the expected icon when using a TDE icon set but transfer nicely to non TDE icon sets. Desktop ------- My Documents: Create a sym link in all TDE places icon directories: ln -s folder_home.png user-home.png. Change /opt/trinity/share/apps/kdesktop/Desktop/My_Documents Icon=user-home. I am guessing xdg compliant icon sets all contain user-home.png so the Icon= change would satisfy all icon sets including TDE. Web Browser: Create a sym link in all TDE apps icon directories: ln -s konqueror.png web-browser.png. Change /opt/trinity/share/apps/kdesktop/Desktop/Web_Browser Icon=web-browser. I am guessing xdg compliant icon sets all contain web-browser.png so the Icon= change would satisfy all icon sets including TDE. I successfully tested both of the above by manually creating the sym links and editing the desktop files. While I can create a patch to update the desktop files, I don't know how to create the sym links in the build process. Before and after screenshots attached. Created attachment 2398 [details]
Screenshot of desktop device icons before suggested changes
Created attachment 2399 [details]
Screenshot of desktop device icons after suggested changes
Regarding the generic icons for Unmounted NFS share and Mounted removable medium. TDE uses nfs_mount.png and nfs_unmount.png. The non TDE icon sets do not include such distinctions. Creating sym links and changing nfs_mounted.desktop or nfs_unmounted.desktop Icon= would only half work because non TDE icon sets are not that explicit. They use the same icon for mounted and unmounted. I believe the reason is the other desktops rely on the Places mechanism to recognize whether a device is (un)mounted. The best we could do is create one sym link, say 'ln -s nfs_mount.png drive-harddisk' and update the nfs_mounted.desktop Icon=drive-harddisk. Users would see a generic icon for unmounted devices. Or vice-versa. Likewise for removable medium. The non TDE icon sets do not distinguish between mounted and unmounted. One icon is used because unlike TDE, there is no little green triangle indicator that a device is mounted. Those indications are part of the Places mechanism, usually in the file managers. (In reply to Darrell from comment #3) > Comparing the icon names used in MATE and TDE, the following fixes would be > more portable and be more xdg compliant or at least non TDE friendly. These > fixes would provide the expected icon when using a TDE icon set but transfer > nicely to non TDE icon sets. > > Desktop > ------- > My Documents: > > Create a sym link in all TDE places icon directories: ln -s folder_home.png > user-home.png. Change /opt/trinity/share/apps/kdesktop/Desktop/My_Documents > Icon=user-home. I am guessing xdg compliant icon sets all contain > user-home.png so the Icon= change would satisfy all icon sets including TDE. > > Web Browser: > > Create a sym link in all TDE apps icon directories: ln -s konqueror.png > web-browser.png. Change /opt/trinity/share/apps/kdesktop/Desktop/Web_Browser > Icon=web-browser. I am guessing xdg compliant icon sets all contain > web-browser.png so the Icon= change would satisfy all icon sets including > TDE. > > I successfully tested both of the above by manually creating the sym links > and editing the desktop files. > > While I can create a patch to update the desktop files, I don't know how to > create the sym links in the build process. > > Before and after screenshots attached. When we finish the XDG update no symlinks will be needed. I just didn't want to make the rest of those (large!) changes before R14.0.0 release, potentially destabilizing our first release in over 2 years in the process. ;-) |