[Panel-devel] Multiple batteries support in battery applet

[maor]

On Wednesday 19 December 2007 16:25:28 Sebastian Kuegler wrote:
> Hi Maor,
>
> On Wednesday 19 December 2007 06:18:39 maor wrote:
> > I'm new here ,well no actually new on the list just on the active way :-)
> > Having some free time  ,I've been trying to update the power management
> > engine and battery applet to support deices with multiple batteries. I've
> > attached the resulted patches. It's seems to work fine beside the tooltip
> > the i still haven't decided how it should be done. I'll appreciate your
> > feedback.
>
> Rocking!
>
> Can you quickly explain what the patch does? I assume (from a glance at the
> code) that it shows two batteries?
>
Yes, currently i choose to show all batteries even those that are not actually 
connected.
> The reason I'm asking is because I recall someone having issues with two
> bays and one battery. If the first bay found doesn't contain a battery, the
> applet assumes there is none, but the second bay isn't checked. It would be
> nice if your patch covered that as well.
>
beacause i choose to show all batteries bays it solve this issue also 
(actually i think i was that complain about it).
> I don't see any problems with the patch, and I'm inclined to commit it
> otherwise.
>
Great :)
> Thanks,

Thanks,
Maor