GNOME Bugzilla – Bug 120823
temperature indicator moves under clouds icon and is not visible
Last modified: 2004-12-22 21:47:04 UTC
Package: gnome-applets Severity: major Version: 2.3.6.2 Synopsis: temperature indicator moves under clouds icon and is not visible Bugzilla-Product: gnome-applets Bugzilla-Component: gweather Description: Description of Problem: When I have panel with size of 24 pix then weather applet has this look: [clouds][temperature] But when I change panel property - size to something more ie. 25...26 pix then applet looks like this: [clouds] [temperature] and temperature is then partially visible. I think that this should be customizable - I mean g weather applet layout. User should have ability to decide if he likes: [clouds][temperature] [clouds] (only) or [clouds] [temperature] (when there is space on panel for this) Steps to reproduce the problem: 1. place applet on panel 24pix size 2. resize panel height to for example 25 pix... 3. see layout changes Actual Results: layout changes Expected Results: should be user defined and/or connected to panel settings How often does this happen? always Additional Information: ------- Bug moved to this database by unknown@bugzilla.gnome.org 2003-08-27 05:38 ------- Reassigning to the default owner of the component, gnome-applets-maint@bugzilla.gnome.org.
Confirming this with 2.4.x. There's an area between 24 and 33 pixels that make the applet misbehave. I guess it should change layout at 33 pixels instead of 25.
*** Bug 123795 has been marked as a duplicate of this bug. ***
*** Bug 126791 has been marked as a duplicate of this bug. ***
Created attachment 23078 [details] [review] A patch that makes things a bit better
Created attachment 23079 [details] [review] An even better patch.
Please ignore the first patch. :) I changed 25 to 37. I picked 37 because 36 or smaller didn't seem to change the situation. Don't ask me why. 37 is a good value though because my biggish (Arial 12) fonts now fit under the icon. This is not an elegant fix but the situation is better with my patch than without it: now I can at least read the temperature when using a small panel.
Who could review and possibly check in this trivial one-liner UI PATCH?
Isn't this a duplicate of bug 135205 ? I'd mark this as a duplicate but I don't have the correct permission.
Yah, duplicate. The other bug has more correct fixes. We don't want to hardcode the sizes. We should check for the font size as well as the image size before determining the layout *** This bug has been marked as a duplicate of 135205 ***