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 514619 - A button on gnome-power-manager prefs is not translatable
A button on gnome-power-manager prefs is not translatable
Status: RESOLVED OBSOLETE
Product: gnome-power-manager
Classification: Deprecated
Component: gnome-power-preferences
2.21.x
Other All
: Normal minor
: ---
Assigned To: GNOME Power Manager Maintainer(s)
GNOME Power Manager Maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2008-02-05 21:34 UTC by Milo Casagrande
Modified: 2008-08-05 06:35 UTC
See Also:
GNOME target: ---
GNOME version: 2.21/2.22


Attachments
This patch should resolve the issue (607 bytes, patch)
2008-02-05 21:35 UTC, Milo Casagrande
rejected Details | Review

Description Milo Casagrande 2008-02-05 21:34:08 UTC
Please describe the problem:
There's a button on the gnome-power-manager prefs window that is not translatable. The button is "Reset to defaults".

Steps to reproduce:
1. 
2. 
3. 


Actual results:


Expected results:


Does this happen every time?


Other information:
Comment 1 Milo Casagrande 2008-02-05 21:35:10 UTC
Created attachment 104510 [details] [review]
This patch should resolve the issue

Simple patch to add as translatable the missing button.
Comment 2 Richard Hughes 2008-02-05 22:14:45 UTC
Please apply to 2-20 and trunk please. Thanks.
Comment 3 Milo Casagrande 2008-02-05 22:44:30 UTC
Hmm... sorry.. should I do that?
Comment 4 Diego Escalante Urrelo (not reading bugmail) 2008-07-26 06:22:27 UTC
Yey Milo, do you have an SVN account? Richard already said the patch is ok, so feel free to commit :).
If you don't have an SVN account, then please let us know so Richard, I, or any other guy with an account can do it for you.

Thanks for you help!

(note: updating patch status)
Comment 5 Milo Casagrande 2008-07-26 10:06:36 UTC
(In reply to comment #4)
> Yey Milo, do you have an SVN account? Richard already said the patch is ok, so
> feel free to commit :).
> If you don't have an SVN account, then please let us know so Richard, I, or any
> other guy with an account can do it for you.

Sorry, I don't have an SVN account... please commit it for me.

> Thanks for you help!

Thank you!


Comment 6 Diego Escalante Urrelo (not reading bugmail) 2008-07-29 05:44:38 UTC
Uhm, no longer valid for trunk?

Richard?
Comment 7 Ted Gould 2008-08-05 05:18:03 UTC
Running an rgrep on trunk:

ted@shi:~/Development/gnome-power-manager/trunk$ rgrep efaults * | grep eset
src/gpm-phone.c:	/* reset to defaults until we get BatteryStateChanged */
ted@shi:~/Development/gnome-power-manager/trunk$ 

I don't think it's valid any longer.  I'm marking it as "fixed", which is probably a bit presumptuous of me, but that way it'll drop off the list of patches needing review unless someone disagrees with my decision.
Comment 8 Diego Escalante Urrelo (not reading bugmail) 2008-08-05 06:35:01 UTC
Rejected and obsolete are more precise. And yes, I agree, no longer valid.