techmik

convo about "always on"

Sep 26th, 2011
94
0
Never
Not a member of Pastebin yet? Sign Up, it unlocks many cool features!
HTML 1.75 KB | None | 0 0
  1. <techmik> i want to watch a video on my phone in the dark, andnot have the led softkeys turn off... if the screen is lit, they are, period
  2. <techmik> oh, i think i see the question now
  3. <maniac103> then this is the completely wrong approach :)
  4. <techmik> that is what i was looking to you for
  5. <techmik> =)
  6. <maniac103> the right approach would make your liblights provide a keypad / button backlight light object
  7. <techmik> i know you know the leds inside out
  8. <maniac103> then everything just works(TM)
  9. <Kaik541> yay maniac103 agrees with what I said... hijacking the notification method was not a good idea xD
  10. <techmik> yes, but you didnt know the alternative =)P
  11. <Kaik541> I just told you it wasn't good and you should ask someone else
  12. <Kaik541> I was right :P
  13. <maniac103> techmik: something like that: https://github.com/CyanogenMod/android_device_motorola_jordan/blob/gingerbread/liblights/lights.c#L280
  14. <techmik> i wanted to figure it out =P
  15. <techmik> awesome
  16. <maniac103> then you can tweak the light levels in Settings -> CyanogenMod -> Display -> Automatic Brightness -> Edit levels and adjust the button brightness there (you probably want >0 for all levels)
  17. <Kaik541> yeah, ours are only on/off, no levels
  18. <maniac103> techmik: if you want the keypad light to also show notifications, you need to handle that case in liblights as well (see how my linked jordan liblights shares the LED between battery and notifications)
  19. <techmik> k
  20. <Kaik541> I think they already do
  21. <Kaik541> https://github.com/CyanogenMod/android_device_samsung_aries-common/blob/gingerbread/liblight/lights.c#L119
  22. <maniac103> Kaik541: there's only the backlight, not a keypad light
  23. <Kaik541> maniac103: yeah, I'm saying we'd have to add it there, but we already have the notification stuff in place
Advertisement
Add Comment
Please, Sign In to add comment