| Summary: | Fifteenpieces About dialog displays a KDE3 branded image | ||
|---|---|---|---|
| Product: | TDE | Reporter: | Darrell <darrella> |
| Component: | tdetoys | Assignee: | Timothy Pearson <kb9vqf> |
| Status: | RESOLVED FIXED | ||
| Severity: | minor | CC: | bugwatch, darrella |
| Priority: | P1 | ||
| Version: | R14.0.0 [Trinity] | ||
| Hardware: | Other | ||
| OS: | Other | ||
| Compiler Version: | TDE Version String: | ||
| Application Version: | Application Name: | fifteenpieces | |
|
Description
Darrell
2012-06-18 02:32:47 CDT
Turns out this was an interesting bug. The image is actually updated. The bug appears only in a Trinity-KDE3 system. There, the KDE image is used rather than the Trinity image. In my case, I have KDE3 and Trinity concurrently installed --- the former for reference and comparison testing. I stumbled across the mysterious KDE3 icon while filing bug report 1081 about the menu search field crashing. While troubleshooting that bug, I needed to restart kicker. Opening the minicli (Alt-F2), and typing 'kicker' revealed the same icon as being used by fifteenpieces. That got me suspicious. Starting a pristine Trinity system without KDE3 revealed the icon for both the fifteenpieces About dialog and kicker were correct. I then knew something in Trinity was grabbing KDE3 icons rather than Trinity icons and the problem was not a Trinity icon that had not been updated for branding. The cause turned out to be the same as those patches resolving bug report 892 related to hard-coding. That is, something is looking in */share/applications/kde/*desktop rather than looking in */share/applications/tde/*desktop to find respective icons. I pushed a patch to resolve the problem (commit 5f382dd7). The patch fixes both the original report with fifteenpieces and with kicker in the minicli dialog. Although technically that patch fixed the fifteenpieces About dialog, I pushed a second patch so fifteenpieces uses its own icon for its About dialog rather than the kicker icon (commit 394fb8b8). As the About dialog in kweather uses similar code as fifteenpieces, I copied the example from kweather to create the fifteenpieces patch. Having KDE3 installed concurrently, even if only used occasionally for reference and comparison testing, actually helped expose a weird corner case bug. I don't believe the same thing could not happen with KDE4 because KDE4 uses */share/applications/kde4/ rather than */share/applications/kde/. Nonetheless, the patch for tdebase/kicker makes for a better system and is the right thing to do. Getting fifteenpieces to use its own icon is proverbial icing on the cake. :-) Resolved! |