Bug 242840 - Bluecurve KWin theme shows pixel noise for "sticky" icon
Summary: Bluecurve KWin theme shows pixel noise for "sticky" icon
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: redhat-artwork
Version: 7
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Than Ngo
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2007-06-06 03:40 UTC by Kevin Kofler
Modified: 2007-11-30 22:12 UTC (History)
0 users

Fixed In Version: 7.0.0-11.fc7
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2007-06-28 01:54:23 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
Patch to fix "show on all desktops" in Bluecurve KWin theme (2.92 KB, patch)
2007-06-08 01:14 UTC, Kevin Kofler
no flags Details | Diff
Corresponding specfile patch (1.16 KB, patch)
2007-06-08 01:15 UTC, Kevin Kofler
no flags Details | Diff

Description Kevin Kofler 2007-06-06 03:40:29 UTC
Description of problem:
The Bluecurve KWin theme's "sticky" ("show on all desktops") icon isn't 
displayed properly, instead some garbage pixels are being shown.

Version-Release number of selected component (if applicable):
redhat-artwork-kde-7.0.0-9.fc7

How reproducible:
Always

Steps to Reproduce:
1. Set the KWin window decoration theme to Bluecurve.
2. Enable the "sticky" icon. (AFAICT, this is the default with 
kde-settings-3.5-28.fc7.1.)
  
Actual results:
There's some random pixels not representing anything.

Expected results:
A recognizable icon should be displayed.

Additional info:
I'm willing to try to hunt this down in the source code if I'm the only one 
who still cares about Bluecurve. ;-)

Comment 1 Kevin Kofler 2007-06-08 00:51:14 UTC
So, given that I'd have to look into that for Quarticurve-kwin (the KDE 4 port) 
anyway, I looked at the problem and found it: someone must have changed 
BASE_BUTTON_SIZE from 16 to 17 at some point, but the pixel data for the "pin" 
icons is all in 16×16. I'm currently trying to figure out the best way to fix 
it. (I have the icon itself fixed, but the masking is still not right in my 
current patch.)

Comment 2 Kevin Kofler 2007-06-08 01:14:38 UTC
Created attachment 156535 [details]
Patch to fix "show on all desktops" in Bluecurve KWin theme

So this one is the good one. Masking now works too (I had to explicitly fill
the extra row and column there). Than, can you please apply this and push this
bugfix to F7 (maybe even FC6 too) ASAP?

Comment 3 Kevin Kofler 2007-06-08 01:15:45 UTC
Created attachment 156537 [details]
Corresponding specfile patch

And here's the corresponding specfile patch.

Comment 4 Than Ngo 2007-06-12 14:40:17 UTC
Kevin, i have taken a look at the patch, it looks fine. But i think it's 
better only change BASE_BUTTON_SIZE to 16.

I will ask David as package maintainer to push this bugfix to F7. Thanks

Comment 5 Kevin Kofler 2007-06-12 15:44:02 UTC
The reason I did my patch this way is that 1. I didn't know why and when 
BASE_BUTTON_SIZE was changed and 2. I didn't want to change the looks except 
for fixing that bug. BASE_BUTTON_SIZE is used in a few more places.

Comment 6 Kevin Kofler 2007-06-19 02:15:57 UTC
Now that you got the KDE 3.5.7 updates out to testing, may I kindly remind you 
of this bug?

Comment 7 Than Ngo 2007-06-19 15:46:40 UTC
It's now fixed in redhat-artwork-7_0_0-11_fc7. I will push it into F7 update 
today. Thanks

Comment 8 Fedora Update System 2007-06-19 21:36:24 UTC
redhat-artwork-7.0.0-11.fc7 has been pushed to the Fedora 7 testing repository.  If problems still persist, please make note of it in this bug report.

Comment 9 Kevin Kofler 2007-06-21 12:52:30 UTC
I marked my patches obsolete because they are not what went in.

I tested the package from testing and it works fine (in fact I'm running it now 
without any problems).

Comment 10 Fedora Update System 2007-06-28 01:54:20 UTC
redhat-artwork-7.0.0-11.fc7 has been pushed to the Fedora 7 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 11 Kevin Kofler 2007-08-13 16:48:15 UTC
Can you please apply the fix to devel too (now that we're probably going to 
still ship KWin v3 in F8)?


Note You need to log in before you can comment on or make changes to this bug.