After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 537679 - Notification Icon Opens to Wrong Size in Vertical Area
Notification Icon Opens to Wrong Size in Vertical Area
Status: RESOLVED FIXED
Product: banshee
Classification: Other
Component: User Interface
1.0.0
Other Linux
: Normal normal
: 1.2
Assigned To: Banshee Maintainers
Banshee Maintainers
: 537886 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2008-06-10 20:49 UTC by Max Battcher
Modified: 2008-06-17 21:42 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Get the ParentWindow for the EventBox, not the Plug, when checking orientation. (981 bytes, patch)
2008-06-10 21:01 UTC, Sandy Armstrong
committed Details | Review

Description Max Battcher 2008-06-10 20:49:50 UTC
With a vertical notification area every time you start Banshee the notification icon gets set to its largest size, taking up too much relative space in the area.  Moving the area's panel to a horizontal position and back resets the size until the next time Banshee is run.
Comment 1 Sandy Armstrong 2008-06-10 20:57:28 UTC
This is due to some oversights in the use of the Tomboy icon sizing code.  I'll attach a patch.
Comment 2 Sandy Armstrong 2008-06-10 21:01:52 UTC
Created attachment 112510 [details] [review]
Get the ParentWindow for the EventBox, not the Plug, when checking orientation.

In the original Tomboy code "this" was the EventBox, whereas in the Banshee code "this" is the Plug containing the EventBox, so ParentWindow was always coming back null and the orientation wasn't actually being checked.

I noticed that with this patch the overly-large size is allocated in the notification area before the icon appears, and then after a second or so the proper-sized icon appears correctly in the notification area as you would expect.  I'll look into it.
Comment 3 Sandy Armstrong 2008-06-10 21:13:36 UTC
(In reply to comment #2)
> I noticed that with this patch the overly-large size is allocated in the
> notification area before the icon appears, and then after a second or so the
> proper-sized icon appears correctly in the notification area as you would
> expect.  I'll look into it.

Actually, I guess it always had that problem...or at least it seems to have that problem even without my patch applied.  Never noticed it before.  Anyway, since it's not a regression I'm less concerned about fixing it.
Comment 4 Bertrand Lorentz 2008-06-17 19:54:27 UTC
*** Bug 537886 has been marked as a duplicate of this bug. ***
Comment 5 Sandy Armstrong 2008-06-17 21:39:51 UTC
Fixed in r4159.