other (geekier) options for text label would be nice, especially "energy"

Bug #754283 reported by Tianon Gravi
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Battery Status
Confirmed
Wishlist
Ivan Zorin

Bug Description

This isn't a bug as much as a feature request. I love being able to choose either percentage or time as the text label by my icon, but I would really love to be able to select "geekier" values from the Battery Status window, especially "Current" within the Battery Charge section. For me, knowing that my battery is below 10 Wh tells me that I'm much closer to death than the 26% would have me believe, since my computer just hard kills around 6 Wh. (Yeah, my battery has lost of lot of its original design capacity...) So, it would be more useful for me to see the battery icon and (8.2 Wh). If I get around to looking further into it, I might be able to submit a patch for this. :P

Tianon Gravi (tianon)
summary: - other (geekier) options for text label would be nice, like current
- voltage
+ other (geekier) options for text label would be nice, like "energy"
summary: - other (geekier) options for text label would be nice, like "energy"
+ other (geekier) options for text label would be nice, especially
+ "energy"
Revision history for this message
Tianon Gravi (tianon) wrote :

That was surprisingly easy to add. Very nice code organization, and especially love that you've used tabs instead of spaces. Attached is my patch, and of course since it's not my project, feel free to change it however you'd like, or even reject it altogether. :)

Revision history for this message
Ivan Zorin (iaz) wrote :

Thank you for taking the time to report this bug and helping to make BatteryStatus better.
Frankly saying, I just don't think about this, but I'm very like this idea - in some cases displaying such information may be really useful.
I haven't any rejections, so I guess it will be available in the next version. Stay tuned and great thanks for the patch!

Changed in battery-status:
assignee: nobody → Ivan Zorin (iaz)
importance: Undecided → Wishlist
status: New → Confirmed
milestone: none → 0.1.2
Revision history for this message
Tianon Gravi (tianon) wrote :

Yeah, you bet! Thanks for making a better battery applet. :)

Revision history for this message
Tianon Gravi (tianon) wrote :

After using my patch successfully for the past 7 months, I noticed that I was inadvertently cutting off all decimals, and then displaying one (so it was always .0). Attached is a fix of my patch. The differentiating line is on line 2624, was rounding and then making it an integer, which is useless, so I just round and forget about the int(). You probably want to apply the same fix to self.battery_percent on the line above it (where I originally copied mine from :P).

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.