Launchpad has imported 70 comments from the remote bug at
https://bugs.kde.org/show_bug.cgi?id=155381.

If you reply to an imported comment from within Launchpad, your comment
will be sent to the remote bug automatically. Read more about
Launchpad's inter-bugtracker facilities at
https://help.launchpad.net/InterBugTracking.

------------------------------------------------------------------------
On 2008-01-10T09:02:43+00:00 Danielekde wrote:

Version:            (using KDE Devel)
Installed from:    Compiled sources
OS:                Linux

with gtk application (like pidgin or emesene) the icon appair bigger than the 
kde icons. I try to explain me better: The space used from the icon is the same 
but the gtk icon are painted bigger and than cut to stay in this space..
You can see my attachment to understand better.

I've notice that if you close any other application in the tray (also a
KDE application) all the gtk icons are re-painted correctly.

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/0

------------------------------------------------------------------------
On 2008-01-10T09:04:28+00:00 Danielekde wrote:

Created attachment 22930
Picture to explain this bug

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/1

------------------------------------------------------------------------
On 2008-01-10T14:00:58+00:00 Bill Egert wrote:

I don't know if this is a plasma problem.  I was seeing the same
behavior with pidgin in KDE 3.5 on Kubuntu.

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/2

------------------------------------------------------------------------
On 2008-01-10T20:53:17+00:00 Jonathan Thomas wrote:

Yeah, Pidgin did that with me in KDE 3.5.x too.

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/3

------------------------------------------------------------------------
On 2008-01-11T10:31:06+00:00 Danielekde wrote:

But the strange thing is that closing another icon, pidgin icon are
painted correctly!

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/4

------------------------------------------------------------------------
On 2008-01-12T19:03:16+00:00 Jasonbstubbs wrote:

The emesene icon looks fine. The icons can't be any bigger than 22x22
which is why the pidgin icon is being cut off. Are you sure this is not
a pidgin bug?

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/5

------------------------------------------------------------------------
On 2008-01-12T21:36:30+00:00 Crazy-4 wrote:

there are a lot more GTK ( basically no-kde apps because tcl apps have
problems also ) appplication with that problem .. like xchat , bmpx etc

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/6

------------------------------------------------------------------------
On 2008-01-13T05:31:59+00:00 Jasonbstubbs wrote:

Created attachment 22980
Possible fix

Looking at the screenshot, it appears that the GTK apps think their icon window
is bigger than it really is. I haven't been able to reproduce this yet, but I
wonder if this patch fixes it?

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/7

------------------------------------------------------------------------
On 2008-01-13T06:16:32+00:00 aseigo wrote:

yeah, that patch looks familiar: i had to do the same thing in kde3's
system tray ;)

the application sets the size of the item even though it has absolutely
NO WAY of knowing where it will appear.

one more reason the current system tray if friggin' brain damaged. no,
that's not kind to the brain damaged, because the system tray is far,
far, far more annoying and stupid....

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/8

------------------------------------------------------------------------
On 2008-01-13T15:55:31+00:00 Crazy-4 wrote:

I doesn't fix anything here. I'll attach an shot with some gtk apps and
a tcl one.

Just in case you wonder what that strange thing on the desktop is , it
is the amsn 'systray' icon =)

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/9

------------------------------------------------------------------------
On 2008-01-13T15:56:49+00:00 Crazy-4 wrote:

Created attachment 22997
systray shot with GTK and TCL apps

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/10

------------------------------------------------------------------------
On 2008-01-14T09:34:42+00:00 Danielekde wrote:

Jason,

I tried to apply your patch to the code but the situation is worst.
Without patch, if I close another icon in the tray, pidgin icon are painted 
correctly.
With the patch, if I close another icon in the tray, pidgin icon remains 
painted bad.


Reply at: 
https://bugs.launchpad.net/ubuntu/+source/kdebase-kde4/+bug/220085/comments/11

------------------------------------------------------------------------
On 2008-02-20T00:42:57+00:00 Sebastian Sauer wrote:

*** Bug 155529 has been marked as a duplicate of this bug. ***

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/12

------------------------------------------------------------------------
On 2008-02-25T19:28:29+00:00 Ginta wrote:

I can confirm this too with pidgin (I didn't have this error in kde
3.5.x )

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/13

------------------------------------------------------------------------
On 2008-02-25T20:43:28+00:00 Rex Dieter wrote:

joy, here too with some other gtk-based apps, including (fedora's)
sealert and puplet.  I think one (or both) of these are inducing a
plasma crash for me too (on session autostart), but I'll worry about
that later. :)

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/14

------------------------------------------------------------------------
On 2008-03-18T19:14:01+00:00 Alchark wrote:

I confirm this with pidgin. In KDE 3.5 Pidgin icon just used to fill the
whole height of the panel (thus taking up about 4x the space any other
icon used), but in my KDE4 SVN build this is cropped to a normal icon
size, with only the upper-left corner of the icon being visible as a
result. Probably, GTK apps just expect to be able to take up the whole
panel height and thus produce an icon larger than the tray widget
expects?

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/15

------------------------------------------------------------------------
On 2008-03-18T19:16:29+00:00 Ginta wrote:

I discovered that as a temporal workarround: if you close another
application in systray, GTK's icons will be normal size. For example:
open pidgin, open kmix, close kmix from tray, and see pidgin icon
correctly.

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/16

------------------------------------------------------------------------
On 2008-03-25T19:56:10+00:00 Gerald Cox wrote:

Created attachment 24050
Unidentified object between nm-applet and kpowersave

In this instance, believed to be sealert which hides itself, but apparently
having problems due to this issue.

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/17

------------------------------------------------------------------------
On 2008-03-25T19:57:04+00:00 Gerald Cox wrote:

Tried workaround mentioned in comment #16 - but doesn't work for me...
:-(

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/18

------------------------------------------------------------------------
On 2008-05-04T09:36:13+00:00 Gilboa Davara wrote:

#16 seems to work from time to time - but it's not consistent.
How can we help debug is problem?

- Gilboa

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/25

------------------------------------------------------------------------
On 2008-06-01T12:58:29+00:00 Mklapetek wrote:

I confirm this bug in Fedora 9 kde4 rawhide [development] packages,
version 4.0.80, all gtk icons are painted too big and then cropped just
like in comment 10's attachment and I also have this mysterious
unidentified object like in comment 17's attachment

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/26

------------------------------------------------------------------------
On 2008-06-06T12:11:56+00:00 Charis Kouzinopoulos wrote:

Just installed kde 4.1 in ubuntu, instead of the pidgin icon only a
white square appears.

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/27

------------------------------------------------------------------------
On 2008-06-06T15:44:38+00:00 L-lunak-5 wrote:

The Pidgin icon is not a KDE bug, Pidgin specified that systray icon
minimum size is 48x48, but our systray sets sizes to 22x22, and the
systray spec says apps are to cope with whatever size they get.

$ xwininfo -tree | grep pidgin
[click on Plasma now]
           0xc0001e "Pidgin": ("pidgin" "Pidgin")  48x48+0+0  +1238+936
$ xprop -id 0xc0001e | grep minimum
                program specified minimum size: 48 by 48

Unless somebody can confirm a case where the app does not specify such
size constraints, this is INVALID.


Reply at: 
https://bugs.launchpad.net/ubuntu/+source/kdebase-kde4/+bug/220085/comments/28

------------------------------------------------------------------------
On 2008-06-06T16:01:52+00:00 Kevin-kofler wrote:

How does KDE 3 deal with this? Does it give the apps what they request
and then downsize the result?

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/29

------------------------------------------------------------------------
On 2008-06-06T16:05:44+00:00 Kevin-kofler wrote:

Oh, and I see this happening in _all_ GTK+ applets, so maybe this is
inside GTK+'s own code. At least nm-applet (NetworkManager-gnome) and
gnome-packagekit are known to behave that way.

Moreover, all this doesn't explain the unidentified icon in comment #17
(I've seen more of these, usually there are some white rectangles in an
otherwise all-black icon, though I've also seen the blue noise as in
comment #17), is that actually a separate bug?

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/30

------------------------------------------------------------------------
On 2008-06-06T16:09:12+00:00 wolfger wrote:

>From comment #22: "systray spec says apps are to cope with whatever size
they get"

Well, in KDE3 these apps seem to cope with it just fine. The problem
only occurs in KDE4. So I'm not sure how anybody can be saying "oh, its
an app/gtk+ issue". It's a dis-improvement on KDE.

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/31

------------------------------------------------------------------------
On 2008-06-06T16:20:44+00:00 Rex Dieter wrote:

Thanks Lubos!

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/32

------------------------------------------------------------------------
On 2008-06-07T02:08:06+00:00 Jonathan Thomas wrote:

Actually, in KDE 3 the systray windows for GTk apps such as Pidgin were
just bigger than all the others.

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/33

------------------------------------------------------------------------
On 2008-06-07T02:13:38+00:00 Kevin-kofler wrote:

But it's constrained by the taskbar size (apparently shrunk to that
size, because it's definitely not displayed as 48×48 which doesn't fit
in my small taskbar) and not cut, so with a small enough taskbar, you
don't notice.

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/34

------------------------------------------------------------------------
On 2008-06-07T08:49:02+00:00 Sebastian Sauer wrote:

Created attachment 25179
test-patch

It seems GTK-apps try to be clever here since they scale there trayicon.

The patch above shows it. While e.g. klipper sticks with it's 22x22 size the
pidgin-trayicon always scales up/down to the maximal size. What a mess :-/

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/35

------------------------------------------------------------------------
On 2008-06-07T09:39:35+00:00 Sebastian Sauer wrote:

Created attachment 25180
patch for review

Attached patch fixes at least the pidgin-prob for me. Ok to commit?

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/36

------------------------------------------------------------------------
On 2008-06-07T09:41:55+00:00 Sebastian Sauer wrote:

Created attachment 25181
second try

ups, a small typo went in. Second try :)

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/37

------------------------------------------------------------------------
On 2008-06-07T21:42:27+00:00 Lukas-4 wrote:

Doesn't quite work - the icons are cropped at the first layout:
http://ktown.kde.org/~lukas/pics/gtkicons-kdetray4.png

However, after I start and quit some tray app, the icons relaid out and all is 
correct:
http://ktown.kde.org/~lukas/pics/gtkicons-kdetray4.png

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/38

------------------------------------------------------------------------
On 2008-06-07T22:27:39+00:00 Sebastian Sauer wrote:

Created attachment 25198
theird try

Thanks Lukas,

seems I am not able to reproduce the truncate :-/ Anyway, I did note that
MARGIN is used for the calculations but was never set on the layout what
would/could explain it.

Therefore; theird try/patch :)

It also contains 2 additional fixes;
1) according to some samples systemTrayClientId can be 0 and in that case we
wan't embed and
2) don't try to create a systray-icon if it was already created else we just
earn a black rect (reproducable by just trying to stark e.g. pidgin a second
time).

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/39

------------------------------------------------------------------------
On 2008-06-07T23:14:49+00:00 Lukas-4 wrote:

With the 3rd patch, things are now laid out correctly, but not on the
first run. Furthermore, the icons are somewhat smaller now (scaled down)
:/

On a side note, the systray gets completely busted when you position the
whole panel on the sides (vertically)

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/40

------------------------------------------------------------------------
On 2008-06-08T00:00:03+00:00 Sebastian Sauer wrote:

Created attachment 25199
testcase nr2

> but not on the first run

eh, I've no idea why that's the case now and can't reproduce it :-/
anyway, there is a small possibility that the "testcase nr2" attachment may fix
it (or make things even more worse). So, if you are motivated to give it a try
(last one I promise)?! :)

> the icons are somewhat smaller now (scaled down) :/

but at least they respect now the margin and therefore don't overlap with the
panel's border. Or does you mean they are not any longer 22x22 pixel like
before even if the panel's height is big enough (as in ~26pixel height)?! hmpf,
I can't reproduce this one too :-/

> completely busted when you position the whole panel on the sides

even more worse, if you try to move it back to a horizontal position it jumps
around here. But well, step by step ;)

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/41

------------------------------------------------------------------------
On 2008-06-08T00:47:52+00:00 Jasonbstubbs wrote:

I've fixed the problem with the first layout in a patch on dashboard but
haven't had time to get back to it yet. MARGIN and spacing are fine as
they are. The problem lays with the data used in calculations in
addWidgetToLayout or whatever that method is called.

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/42

------------------------------------------------------------------------
On 2008-06-08T00:56:25+00:00 Jasonbstubbs wrote:

Ok, I've committed the changes I've mentioned above, but the following
two changes aren't covered.

     Window ww = XCreateWindow(dpy, parentWidget() ? parentWidget()->winId() : 
DefaultRootWindow(dpy),
-            1, 1, 1, 1, 0, ga.depth, InputOutput, ga.visual,
+            0, 0, 22, 22, 0, ga.depth, InputOutput, ga.visual,

If this fixes this bug, then it should definitely be applied. ;)

+            if (systemTrayClientId == 0)
+                return true;
+
+            foreach(SystemTrayContainer* c, 
findChildren<SystemTrayContainer*>())
+                if (c->clientWinId() == systemTrayClientId)
+                    return true;

In a perfect world this shouldn't be needed but I guess it's not very
good practice to trust other applications. :)


Reply at: 
https://bugs.launchpad.net/ubuntu/+source/kdebase-kde4/+bug/220085/comments/43

------------------------------------------------------------------------
On 2008-06-08T00:58:25+00:00 Kevin-kofler wrote:

Especially GTK+ ones. ;-)

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/44

------------------------------------------------------------------------
On 2008-06-08T04:57:18+00:00 Sebastian Sauer wrote:

re;
-            1, 1, 1, 1, 0, ga.depth, InputOutput, ga.visual, 
+            0, 0, 22, 22, 0, ga.depth, InputOutput, ga.visual, 
probably your additional resize() does the same. No idea there.

re;
+            if (systemTrayClientId == 0) 
+                return true; 
better be safe then pay alimonies ;)

re;
+            foreach(SystemTrayContainer* c, 
findChildren<SystemTrayContainer*>()) 
+                if (c->clientWinId() == systemTrayClientId) 
+                    return true; 
This one fixes the case that unique-/single-instance apps seem to like to 
register a second time. iirc pidgin does so (or xchat?!) and it results in a 
SystemTrayContainer filled with a black rect.

re;
"patch on dashboard"
oh, man. Linus is right, we (KDE) are stupid and ugly. Sorry for that, but 
those "review-board" is just the wrong way imho.[/grant]


Reply at: 
https://bugs.launchpad.net/ubuntu/+source/kdebase-kde4/+bug/220085/comments/45

------------------------------------------------------------------------
On 2008-06-08T05:01:59+00:00 Kevin-kofler wrote:

To me, all 3 of those changes (quoted in comment #37 and comment #39)
look like they can't hurt, so I'd suggest committing them all.

Hopefully Lukáš can do some more testing with the latest patches.

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/46

------------------------------------------------------------------------
On 2008-06-08T16:38:59+00:00 Lukas-4 wrote:

Yes pls commit them so I can test, I just tested against current trunk
and therefore cannot apply your previous patches

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/47

------------------------------------------------------------------------
On 2008-06-09T03:21:24+00:00 Sebastian Sauer wrote:

fix committed with r818620 in trunk. It should now work as expected :)

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/48

------------------------------------------------------------------------
On 2008-06-09T19:44:43+00:00 Sebastian Sauer wrote:

*** Bug 163320 has been marked as a duplicate of this bug. ***

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/49

------------------------------------------------------------------------
On 2008-06-09T20:27:01+00:00 Lukas-4 wrote:

Even worse with current trunk :/

As you can see from the screenshot, gtk icons are not drawn at all
(blank space), KDE 3 app (konversation) icon is chopped and other
Qt4/KDE4 apps' icons are rendered mini small...

http://ktown.kde.org/~lukas/pics/gtkicons-kdetray4-badagain.png

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/50

------------------------------------------------------------------------
On 2008-06-09T21:50:42+00:00 Sebastian Sauer wrote:

hmmm, so we need to do the resize() then at a later time. r818824 should
fix that now.


Reply at: 
https://bugs.launchpad.net/ubuntu/+source/kdebase-kde4/+bug/220085/comments/51

------------------------------------------------------------------------
On 2008-06-10T00:19:44+00:00 Lukas-4 wrote:

Almost... now the GTK icons don't show at all; the blank space between
konversation and kwallet is nm-applet. It doesn't react to any mouse
event but it pops up balloon messages when I go online/offline :o)

http://ktown.kde.org/~lukas/pics/gtkicons-kdetray4-stillwrong.png

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/52

------------------------------------------------------------------------
On 2008-06-10T02:01:58+00:00 Sebastian Sauer wrote:

ok, it seems we got multiple new regressions since r818155 from comment
#37 :-/

see bug #163320 and bug #163565

@jason any idea?


Reply at: 
https://bugs.launchpad.net/ubuntu/+source/kdebase-kde4/+bug/220085/comments/53

------------------------------------------------------------------------
On 2008-06-11T00:40:02+00:00 Rex Dieter wrote:

Just for my own sanity if nothing else, are these genuine *kde* bugs, or
are we simply patching/working-around gtk bugs?  Or some middle ground?

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/54

------------------------------------------------------------------------
On 2008-06-11T10:33:15+00:00 René wrote:

I just want to confirm the same problem with Pidgin in KDE 4.0.81 Beta1
(OpenSUSE 11.0 packages), there is a bad zoom factor of it's icon or bad
colors. There are already problems in KDE3 with zooming the Pidgin icon,
it always appears to be a bit bigger than the KDE icons around it in the
taskbar. In KDE4, the initial Pidgin icon appears even bigger and after
changing for instance on incoming messages it shows only a white space
with a small piece of somewhat of the icon picture on the right downside
corner. GTK version 2.12.9.

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/55

------------------------------------------------------------------------
On 2008-06-11T10:38:51+00:00 René wrote:

Created attachment 25258
The white space to the right of the Klipper icon should be the Pidgin online 
state icon

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/56

------------------------------------------------------------------------
On 2008-06-12T02:13:18+00:00 Sebastian Sauer wrote:

Rex: imho neither of it, the X11-protocol for systrayicons just sucks :)

Rene: what revision is that? >=r818824?


Reply at: 
https://bugs.launchpad.net/ubuntu/+source/kdebase-kde4/+bug/220085/comments/57

------------------------------------------------------------------------
On 2008-06-12T09:09:39+00:00 René wrote:

Sebastian: I can't tell you the SVN revision reliably, since I use
precompiled binaries from the OpenSUSE factory. Help->About KDE tells
me: Version 4.00.81 (KDE 4.0.81 >= 20080527) "release 6.4"

rpm -qi --changelog kdebase4-workspace-4.0.81-5.1:
----------------------------------------------------------------
Install Date: Tue 10 Jun 2008 08:06:02 PM CEST      Build Host: build15
Group       : System/GUI/KDE                Source RPM: 
kdebase4-workspace-4.0.81-5.1.src.rpm
Size        : 18459986                         License: GPL v2 or later
Signature   : DSA/SHA1, Tue 10 Jun 2008 09:35:23 AM CEST, Key ID 
58d8ff412e1efa87
URL         : http://www.kde.org/
Summary     : The KDE Workspace Components
Description :
This package contains the basic packages for a K Desktop Environment
workspace.


Authors:
--------
    The KDE Team <[email protected]>
Distribution: KDE:KDE4:UNSTABLE:Desktop / openSUSE_Factory
* Wed May 28 2008 [email protected]
- obsolete kdebase4-workspace-plasmoids

* Thu Mar 13 2008 [email protected]
- update to 4.0.66
...
----------------------------------------------------------------

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/58

------------------------------------------------------------------------
On 2008-06-13T12:21:42+00:00 René wrote:

In the latest OpenSUSE packages versioned as 4.0.82 the systray problem
with Pidgin is solved, except that the icon background of all icons is
not transparent. Additionally, all icons are better aligned now to the
systray (vertically centered). As far as I can see, there is a progress
... ;-) Thanks

Name        : kdebase4-workspace           Relocations: (not relocatable)
Version     : 4.0.82                            Vendor: openSUSE Build Service
Release     : 18.1                          Build Date: Wed 11 Jun 2008 
11:04:22 PM CEST


Reply at: 
https://bugs.launchpad.net/ubuntu/+source/kdebase-kde4/+bug/220085/comments/59

------------------------------------------------------------------------
On 2008-06-13T12:36:10+00:00 René wrote:

Created attachment 25303
Systray icons now in 4.0.82 (OpenSUSE 11.0).

FYI: This is the way the systray icons look now in 4.0.82 (OpenSUSE
11.0).

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/60

------------------------------------------------------------------------
On 2008-06-13T18:38:14+00:00 Jasonbstubbs wrote:

After over six months this is good to hear. Good work by Sebastian. :)

The background color issue is known and covered in a separate bug, so
I'll close this one as fixed.

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/61

------------------------------------------------------------------------
On 2008-06-13T18:59:20+00:00 Alchark wrote:

Created attachment 25308
Gentoo with a recent SVN checkout

As for me, I still have problems with the systray. It looks fine as long as
there are few enough icons, but when more are added, they are displayed in two
rows, incorrectly resized (see the image). These are KMix, KNetworkmanager,
KBluetooth and KWallet. Space to the left is occupied by the digital clock,
which is displayed in black font for some reason. Thus, I believe that it is
not FIXED yet, unfortunately.

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/62

------------------------------------------------------------------------
On 2008-06-13T19:08:20+00:00 Jasonbstubbs wrote:

Do you have a revision number for the checkout? `svn info` in
kdebase/workspace/plasma/applets/systemtray would be great to confirm.

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/63

------------------------------------------------------------------------
On 2008-06-14T10:16:09+00:00 Alchark wrote:

Jason, as you requested, here is my `svn info`:

Path: .
URL: 
svn://anonsvn.kde.org/home/kde/trunk/KDE/kdebase/workspace/plasma/applets/systemtray
Repository Root: svn://anonsvn.kde.org/home/kde
Repository UUID: 283d02a7-25f6-0310-bc7c-ecb5cbfe19da
Revision: 820179
Node Kind: directory
Schedule: normal
Last Changed Author: sebsauer
Last Changed Rev: 818824
Last Changed Date: 2008-06-09 23:08:32 +0400 (Пнд, 09 Июн 2008)

The problem survives reboots.

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/64

------------------------------------------------------------------------
On 2008-06-14T11:06:08+00:00 Jasonbstubbs wrote:

That's the latest. Reopening, but I can't reproduce it here. :(

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/65

------------------------------------------------------------------------
On 2008-06-14T12:06:46+00:00 Lukas-4 wrote:

Same problems as in comment #56, the systray is basically unusable :{
nm-applet doesn't even show up although it is running

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/66

------------------------------------------------------------------------
On 2008-06-14T12:39:11+00:00 Alchark wrote:

Created attachment 25328
After removing/readding the applet

After removing the applet from the panel and adding it again, icon sizes are as
expected, but icons are drawn in a wrong place. This is the same SVN build.
Again, place to the left is not empty, but rather occupied by the clock.

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/67

------------------------------------------------------------------------
On 2008-06-15T02:20:15+00:00 Sebastian Sauer wrote:

Created attachment 25348
Patch by Mathias Kraus for system tray layouting

As posted on http://lists.kde.org/?l=kde-panel-
devel&m=121345463825632&w=2

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/68

------------------------------------------------------------------------
On 2008-06-15T02:21:49+00:00 Sebastian Sauer wrote:

*** Bug 163565 has been marked as a duplicate of this bug. ***

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/69

------------------------------------------------------------------------
On 2008-06-15T02:28:39+00:00 Sebastian Sauer wrote:

Comment on attachment 25348
Patch by Mathias Kraus for system tray layouting

Index: systemtraywidget.cpp
===================================================================
--- systemtraywidget.cpp        (revision 820643)
+++ systemtraywidget.cpp        (working copy)
@@ -132,31 +132,31 @@

     // Figure out where the next widget should go
     if (m_orientation == Qt::Horizontal) {
+        // Calculate the items that fit into a column
+        m_maxCount = (maximumHeight() + m_mainLayout->spacing()) /
(widget->height() + m_mainLayout->spacing()) -1;
+
         setMinimumSize(QSize(22 * (m_nextColumn + 1) + m_mainLayout->spacing()
* m_nextColumn,
                              22 * (m_maxCount + 1) + m_mainLayout->spacing() *
m_maxCount));
+
         // Add down then across when horizontal
         m_nextRow++;
-        if ((m_nextRow == m_maxCount && m_nextColumn != 0) ||
-            minimumHeight() + widget->height() + m_mainLayout->spacing() >
maximumHeight()) {
+        if (m_nextRow > m_maxCount){
             m_nextColumn++;
             m_nextRow = 0;
         }
-        if (m_nextColumn == 0) {
-            m_maxCount = m_nextRow;
-        }
     } else {
+        // Calculate the items that fit into a row
+        m_maxCount = (maximumWidth() + m_mainLayout->spacing()) /
(widget->width() + m_mainLayout->spacing()) -1;
+
         setMinimumSize(QSize(22 * (m_maxCount + 1) + m_mainLayout->spacing() *
m_maxCount,
                              22 * (m_nextRow + 1) + m_mainLayout->spacing() *
m_nextRow));
+
         // Add across then down when vertical
         m_nextColumn++;
-        if ((m_nextColumn == m_maxCount && m_nextRow != 0) ||
-            minimumWidth() + widget->width() + m_mainLayout->spacing() >
maximumWidth()) {
+        if (m_nextColumn > m_maxCount) {
             m_nextRow++;
             m_nextColumn = 0;
         }
-        if (m_nextRow == 0) {
-            m_maxCount = m_nextColumn;
-        }
     }
 }

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/70

------------------------------------------------------------------------
On 2008-06-15T12:53:45+00:00 Alchark wrote:

Patch from comment #62 resolved all the issues for me, looking forward
for it to appear in trunk. Thanks to everybody involved!

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/71

------------------------------------------------------------------------
On 2008-06-16T02:12:01+00:00 Sebastian Sauer wrote:

patch committed with r820991 in trunk. another try to mark the bug
closed :)


Reply at: 
https://bugs.launchpad.net/ubuntu/+source/kdebase-kde4/+bug/220085/comments/72

------------------------------------------------------------------------
On 2008-06-16T05:19:29+00:00 Lukas-4 wrote:

Yay! Nice work, all seems to be working fine so far (except the black
non-transparent background, but that's another issue)

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/73

------------------------------------------------------------------------
On 2008-06-21T00:46:33+00:00 Sebastian Sauer wrote:

*** Bug 164567 has been marked as a duplicate of this bug. ***

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/74

------------------------------------------------------------------------
On 2018-09-04T19:17:38+00:00 Andrew-crouthamel wrote:

Hello! Plasma 4 was replaced by Plasma 5 four years ago by the KDE
community. In that time we have made great strides in stability and
functionality. We are closing all Plasma 4 bugs as most of them are no
longer applicable to the new frameworks Plasma 5 is built upon. If you
could, please re-test with the latest version of Plasma 5, and submit a
new bug to "plasmashell" if you continue to have an issue. Thank you!

Reply at: https://bugs.launchpad.net/ubuntu/+source/kdebase-
kde4/+bug/220085/comments/76


** Changed in: kde-baseapps
       Status: Fix Released => Unknown

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/220085

Title:
  version 2.4.1 icon does not scale correctly in kde4's systray

To manage notifications about this bug go to:
https://bugs.launchpad.net/kde-baseapps/+bug/220085/+subscriptions

-- 
ubuntu-bugs mailing list
[email protected]
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to