Universal Applets

add time remaining on battery to sysmonitor

Bug #260945 reported by termite on 2008-08-24

8

This bug affects 1 person

Affects		Status	Importance	Assigned to	Milestone
	Universal Applets	Fix Committed	Undecided	Przemysław Firszt

Bug Description

Currently, Sysmonitor doesn't display time remaining on battery. The patches to sensors.py and SysmonitorScreelet.py attached remedy this by reading a little extra data about the battery and displaying the time remaining if the user is currently on battery power.

Revision history for this message

termite (rb6) wrote on 2008-08-24:

#1

patch for sensors.py to add current rate of power drainage Edit (733 bytes, text/plain)

Revision history for this message

termite (rb6) wrote on 2008-08-24:

#2

patch for SysmonitorScreenlet.py to display time remaining if user is on battery Edit (1.7 KiB, text/plain)

Revision history for this message

termite (rb6) wrote on 2008-08-25:

#3

actual working patch to do the same thing :) Edit (1.7 KiB, text/plain)

Please ignore last SysmonitorScreenlet.py patch. New one attached that actually works...

Revision history for this message

Natan Yellin (aantny) wrote on 2008-08-29:

#4

@Pefi:
What's the status on this? Are you still reviewing the patch?

Revision history for this message

Natan Yellin (aantny) wrote on 2008-09-29:

#5

*Bump*

Revision history for this message

Przemysław Firszt (pefi) wrote on 2008-09-29:

#6

Hmm, looks like I missed this one. I'm very sorry.

Revision history for this message

Przemysław Firszt (pefi) wrote on 2008-09-29:

#7

I cannot force it to work. Termite: it works for you? I've applied the second patch, fixed an indent error, just to get another one 'tuple index out of range'. :-(

Przemysław Firszt (pefi) on 2008-10-02

Changed in universal-applets:
status:	New → Incomplete

Revision history for this message

termite (rb6) wrote on 2008-10-09:

#8

patch-sensors-add-battery-rate Edit (733 bytes, text/plain)

Attaching two patches, one to SysmonitorScreenlet.py and one to sensors.py. These currently apply cleanly against screenlets, not tested against Universal Applets.

Revision history for this message

termite (rb6) wrote on 2008-10-09:

#9

patch-sysmonitor-display-battery-time-remaining Edit (1.7 KiB, text/plain)

Revision history for this message

Przemysław Firszt (pefi) wrote on 2008-10-10:

#10

patches commited but needs to be tested .

Changed in universal-applets:
assignee:	nobody → pefi
status:	Incomplete → Fix Committed

Report a bug

This report contains Public information

Everyone can see this information.

You are

Subscribing...

Edit bug mail

Other bug subscribers

Related blueprints

Stabilize individual applets before stable release

Patches

Add patch

Bug attachments

Add attachment

Remote bug watches

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