From kde-panel-devel Mon May 27 09:36:40 2013 From: Dario Freddi Date: Mon, 27 May 2013 09:36:40 +0000 To: kde-panel-devel Subject: Re: Battery Monitor revamp Message-Id: X-MARC-Message: https://marc.info/?l=kde-panel-devel&m=136964741305629 MIME-Version: 1 Content-Type: multipart/mixed; boundary="--===============0912731573277944757==" --===============0912731573277944757== Content-Type: multipart/alternative; boundary=e89a8ff25604c16d1004ddafe2f7 --e89a8ff25604c16d1004ddafe2f7 Content-Type: text/plain; charset=ISO-8859-1 2013/5/27 Kai Uwe Broulik > I could add it to the expandable battery details but it will show the > overall remaining time for all batteries in contrast to the other > properties which are directly related to the battery they emerge from. > Actually I don't care. I don't have a strong opinion on whether to show it > or not. > > Would that be an acceptable compromise for all? > I don't really like the compromise as it makes the estimation even worse (even if for most cases you only have 1 battery). I can have a look at the code over the next days and see if I can provide a patch. P.S.: Thanks for your work, the new plasmoid looks great > > Dario Freddi schrieb: > > > 2013/5/24 > >> What's missing: >> - Remaining time. There is no "global" category anymore and since we >> discourage and removed the option for displaying remaining time, I didn't >> see the need to implement it in solod so we can show it in eg. the battery >> advanced properties >> >> > I promised to never talk about this again, but I have to break in: as > co-maintainer of the battery plasmoid I won't accept this change at all. I > am not going to argue about usefulness, defaults and truthfulness of the > value as too much has been said already. I will just say that many > distributions and users are relying on this hidden feature, and we cannot > remove it ESPECIALLY when we won't be providing updates to 4.x anymore. > > Such a choice has only one possible outcome: distributions and users > patching the plasmoid for restoring the functionality, hidden or not, > resulting in harder bug triaging and higher chances for breakage. > > Again, the argument is not technical but completely focused on what some > users and distributions apparently want, and the obvious consequences of > the change. > > --e89a8ff25604c16d1004ddafe2f7 Content-Type: text/html; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable
2013/5/27 Kai Uwe Broulik <kde@privat.broulik.de&= gt;
I could add it to the expandable battery details but it will show the overa= ll remaining time for all batteries in contrast to the other properties whi= ch are directly related to the battery they emerge from.
Actually I don&= #39;t care. I don't have a strong opinion on whether to show it or not.=

Would that be an acceptable compromise for all?
I don't really like the compromise as it makes the e= stimation even worse (even if for most cases you only have 1 battery). I ca= n have a look at the code over the next days and see if I can provide a pat= ch.

P.S.: Thanks for your work, the new plasmoi= d looks great
=A0

Dario F= reddi <drf54321@= gmail.com> schrieb:


2013/5/24 <= ;kde@privat.brou= lik.de>
=
What's missing:
=A0- Remaining time. There is no "global" category anymore and si= nce we discourage and removed the option for displaying remaining time, I d= idn't see the need to implement it in solod so we can show it in eg. th= e battery advanced properties


I promised to never talk about this ag= ain, but I have to break in: as co-maintainer of the battery plasmoid I won= 't accept this change at all. I am not going to argue about usefulness,= defaults and truthfulness of the value as too much has been said already. = I will just say that many distributions and users are relying on this hidde= n feature, and we cannot remove it ESPECIALLY when we won't be providin= g updates to 4.x anymore.

Such a choice has only one possible outcome: distributi= ons and users patching the plasmoid for restoring the functionality, hidden= or not, resulting in harder bug triaging and higher chances for breakage.<= /div>

Again, the argument is not technical but completely foc= used on what some users and distributions apparently want, and the obvious = consequences of the change.


--e89a8ff25604c16d1004ddafe2f7-- --===============0912731573277944757== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel --===============0912731573277944757==--