Buttons "return" and "home" stay lit when phone is locked

[GoogleCodeExporter]

Buttons "return" and "home" stay lit when phone is locked,so that's discharging 
the battery. 

PS : Sometimes when I lock the phone, the buttons act normally

Original issue reported on code.google.com by yafou...@gmail.com on 19 Apr 2011 at 9:57

[GoogleCodeExporter]

It's not a bug, it's a feature.
They act as notification light. So If you have a notification, they lights up. 
Clear the notifications, they shuts down. Gerard Majaxxxx!

Original comment by m.pluvin...@gmail.com on 19 Apr 2011 at 10:00

[GoogleCodeExporter]

[deleted comment]

[GoogleCodeExporter]

Thank you for your response, I will try to change that in the config of my 
phone.
Sorry for the inconvenience, I thought that was a bug ..

Original comment by yafou...@gmail.com on 19 Apr 2011 at 10:05

[GoogleCodeExporter]

Original comment by wkp...@gmail.com on 19 Apr 2011 at 1:55

• Added labels: Priority-Low
• Removed labels: Priority-Medium

[GoogleCodeExporter]

IMO it is a bug. The battery drains quicker when the buttons are lightened and 
if you sleep next to your phone and it happens to receive a text message or any 
other notification, the whole room would light up.

It's a feature if you could enable or disable it at your pleasure...

Original comment by savalent...@gmail.com on 19 Apr 2011 at 3:40

[GoogleCodeExporter]

hmm true. I personally like this feature but there should be an option to 
enable or disable it at will

Original comment by amaury.l...@gmail.com on 19 Apr 2011 at 6:38

[GoogleCodeExporter]

Is there a way to disable this, Editing some config file?

Original comment by cmgonza...@gmail.com on 19 Apr 2011 at 10:29

[GoogleCodeExporter]

There is a setting for LED notification:

Settings => Sound settings => Notification => ...

but it is not working for now.

Original comment by wkp...@gmail.com on 20 Apr 2011 at 6:36

[GoogleCodeExporter]

Original comment by wkp...@gmail.com on 20 Apr 2011 at 7:17

• Changed state: Fixed
• Added labels: Priority-Medium, Milestone-Beta
• Removed labels: Priority-Low

[GoogleCodeExporter]

framework/base
--- a/services/java/com/android/server/NotificationManagerService.java
+++ b/services/java/com/android/server/NotificationManagerService.java
@@ -440,7 +440,7 @@ public class NotificationManagerService extends 
INotificationManager.Stub
             } else if (action.equals(Intent.ACTION_SCREEN_ON)) {
                 mScreenOn = true;
                 if (mAmberGreenLight) {
-                    if (!mNotificationAlwaysOnEnabled) {
+                    if (mNotificationAlwaysOnEnabled) {
                         updateGreenLight();
                     }
                 } else {
@@ -452,7 +452,7 @@ public class NotificationManagerService extends 
INotificationManager.Stub
             } else if (action.equals(Intent.ACTION_SCREEN_OFF)) {
                 mScreenOn = false;
                 if (mAmberGreenLight) {
-                    if (!mNotificationAlwaysOnEnabled) {
+                    if (mNotificationAlwaysOnEnabled) {
                         updateGreenLight();
                     }
                 } else {

Original comment by wkp...@gmail.com on 20 Apr 2011 at 7:27

[GoogleCodeExporter]

Issue 37 has been merged into this issue.

Original comment by wkp...@gmail.com on 29 Apr 2011 at 2:09

[GoogleCodeExporter]

Original comment by wkp...@gmail.com on 29 May 2011 at 12:58

• Changed state: Verified

[GoogleCodeExporter]

Issue 217 has been merged into this issue.

Original comment by wkp...@gmail.com on 10 Jan 2012 at 3:47