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 405421 - TFT head-up brightness appears in centre of screen, rather than 3/5 down
TFT head-up brightness appears in centre of screen, rather than 3/5 down
Status: RESOLVED FIXED
Product: gnome-power-manager
Classification: Deprecated
Component: gnome-power-manager
SVN TRUNK
Other Linux
: Normal enhancement
: ---
Assigned To: GNOME Power Manager Maintainer(s)
GNOME Power Manager Maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2007-02-07 16:36 UTC by Paul Sladen
Modified: 2007-07-26 12:57 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
patch to fix wrong feedback widget position (4.41 KB, patch)
2007-07-15 10:44 UTC, Cosimo Cecchi
none Details | Review

Description Paul Sladen 2007-02-07 16:36:38 UTC
Forwarded from https://launchpad.net/bugs/60902

When the brightness changes, a head-up dialogue appears for a couple of seconds. This is in the centre of the screen and covers up anything currently being worked on:

  http://www.paul.sladen.org/ubuntu/bugs/volume-brightness-location.png

The volume-control equivalent appears 3/4 of the way down the screen which is a much better location and does not get in the way. Ideally, the brightness should be the same.
Comment 1 Richard Hughes 2007-02-07 16:54:54 UTC
Sure, agree. Is it exactly 3/4 from the top of the screen? Is it measured to the center of the box or just the top? Thanks.
Comment 2 Paul Sladen 2007-02-07 21:02:48 UTC
When I was writing a dbus-controlled Head-Up daemon specially /just/ to display on-screen messages, I tried various placements, the 3/5 of the way down the screen looked better than the 3/4 that the audio daemon was using.  I presume that this was probably 3/5 for the centre of the box.

Eg. the position was probably:

  at((screen.width-box.width)*1/2, (screen.height-box.height)*3/5);

But /play/ first.  If this gets changed in g-p-m, it can easily be changed in whatever displays the volume and other status.
Comment 3 Richard Hughes 2007-02-14 15:30:28 UTC
Cool cheers. Could you knock up a patch please, as I'm currently unable to test this. If you are busy I can do it in a few weeks time, but it will miss 2-18.

Richard.
Comment 4 Cosimo Cecchi 2007-07-15 10:44:57 UTC
Created attachment 91804 [details] [review]
patch to fix wrong feedback widget position

I took care of this bug and wrote a patch that implements for the gpm-feedback-widget the same behaviour of the volume widget. Now they look the same in size and they should appear exactly in the same place.
Comment 5 Richard Hughes 2007-07-16 17:21:10 UTC
Cosimo, please commit. Many thanks.
Comment 6 Cosimo Cecchi 2007-07-19 18:05:52 UTC
How can I commit it myself?
Comment 7 Richard Hughes 2007-07-26 12:57:40 UTC
2007-07-26  Richard Hughes  <richard@hughsie.com>

	* data/gpm-feedback-widget.glade:
	* src/gpm-feedback-widget.c: (gpm_feedback_show),
	(gpm_feedback_display_value):
	Apply a patch from Cosimo Cecchi <anarki@lilik.it> to fix the position
	and the style of the brightness widget. Fixes #405421