> On Dec. 6, 2012, 3:17 p.m., Kai Uwe Broulik wrote: > > What I dislike about the 20% graphic is that it is red which means when I > > have 30% left, the icon will already be red indicating soon power outage
Yeah, sure, so change that bound to 20? 20 is KDE's default "low battery" threshold. So make it: 20-49 looks like 40 10-19 looks like 20? - Xuetian ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/107612/#review23073 ----------------------------------------------------------- On Dec. 6, 2012, 2:23 p.m., Xuetian Weng wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/107612/ > ----------------------------------------------------------- > > (Updated Dec. 6, 2012, 2:23 p.m.) > > > Review request for Plasma, Marco Martin and Viranch Mehta. > > > Description > ------- > > Current battery plasmoid icon is very confusing when showing the percentage. > > Currently, > 96-100 -> looks like 100 > 81-95->looks like 80 (looks werid) > 50-80->looks like 60 (looks werid) > 20->49->looks like 40 > 10->19->looks like 20 > > When people see the icon shows as 60, they will expect that whey have lost > nearly 40 percent, but actually 70-80 is also covered in this range. > > And range doesn't split evenly is also a problem. > This patch change it to > 90-100 looks like 100 > 70-89 looks like 80 > 50-69 looks like 60 > 30-49 looks like 40 > 10-29 looks like 20 > > Which IMHO is much better for visual. > > > Diffs > ----- > > plasma/generic/applets/batterymonitor/contents/ui/BatteryIcon.qml cfb50df > > Diff: http://git.reviewboard.kde.org/r/107612/diff/ > > > Testing > ------- > > No problem. > > > Thanks, > > Xuetian Weng > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
