Opened 18 months ago

Closed 17 months ago

Last modified 17 months ago

#18113 closed bug (fixed)

PowerStatus doesn't abort cleanly on desktop

Reported by: SamuraiCrow Owned by: nobody
Priority: low Milestone: R1/beta5
Component: Applications/PowerStatus Version: R1/beta4
Keywords: Cc:
Blocked By: Blocking:
Platform: x86-64

Description

After notifying the user that there is no battery detected, it shoudn't immediately also ask whether it should run in a window or in the deskbar.

Change History (7)

comment:1 by nephele, 18 months ago

It probably should imo. I think the alert that there is no battery is out of place, instead the icon should reflect this. I have many laptops where you can just take out batteries at runtime if you want.

comment:2 by korli, 18 months ago

Actually there is another driver for this, acpi_ac, which could confirm ac power is connected.

comment:3 by nephele, 18 months ago

Maybe powerstatus can simply show an icon that it is connected via AC then, and remove the warning entirely?

in reply to:  3 comment:4 by korli, 17 months ago

Replying to nephele:

Maybe powerstatus can simply show an icon that it is connected via AC then, and remove the warning entirely?

Sounds good,

comment:5 by pulkomandy, 17 months ago

When there is no battery, the alert does not show up. It only shows if your system has no APM nor ACPI available, in which case there is no use at all for the app. That's why there is a warning, and the app should indeed immediately exit in that case.

comment:6 by waddlesplash, 17 months ago

Milestone: UnscheduledR1/beta4
Resolution: fixed
Status: newclosed

Fix merged in hrev56654

comment:7 by waddlesplash, 17 months ago

Milestone: R1/beta4R1/beta5
Note: See TracTickets for help on using tickets.