Opened 2 years ago

Closed 2 years ago

Last modified 2 years 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, 2 years 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, 2 years ago

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

comment:3 by nephele, 2 years 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, 2 years 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, 2 years 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, 2 years ago

Milestone: UnscheduledR1/beta4
Resolution: fixed
Status: newclosed

Fix merged in hrev56654

comment:7 by waddlesplash, 2 years ago

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