Review Request 123169: Show percentage value beside brightness level slider

Siddhartha Sahu sh.siddhartha at gmail.com
Sun Mar 29 19:59:04 BST 2015



> On March 30, 2015, 12:04 a.m., Kai Uwe Broulik wrote:
> > Thank you for this patch!
> > 
> > However, we chose to remove the percentage from the brightness sliders in the battery monitor since you basically change the brightness to your liking and not to some odd percentage, so adding them back in the PowerDevil settings doesn't make sense.
> 
> Siddhartha Sahu wrote:
>     Hi Kai,
>     
>     Yes for the battery monitor it is not really required because we get immediate feedback.
>     
>     In the KCM, the sliders are for different profiles which come into play at different times. I do not like the brightness change that happens when I remove my AC adaptor for example, so I keep the brightness levels the same in all the profiles. Setting it the same is a tad tedious currently because I need to swtich tabs multiple times to check if both are in the same position.
>     
>     Well, I guess mine is a very special case, and would understand if this patch does not really make sense in the main repo. I can keep using it as a special patch on my end :)
> 
> Kai Uwe Broulik wrote:
>     What speaks against just unchecking the brightness action? Then it won't change it when you plug in or out the AC.
>     I hate when it messes with my brightness, so I just turned everything off and only ever manually change it :)

Ah. Why did I not think of that. Obvious in hindsight :D

Discarding the RR then. Thanks!


- Siddhartha


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/123169/#review78180
-----------------------------------------------------------


On March 29, 2015, 3:12 p.m., Siddhartha Sahu wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/123169/
> -----------------------------------------------------------
> 
> (Updated March 29, 2015, 3:12 p.m.)
> 
> 
> Review request for kde-workspace.
> 
> 
> Repository: powerdevil
> 
> 
> Description
> -------
> 
> I prefer to keep the same brightness level for all profiles. But its a bit difficult to set the same value in all tabs using just the slider. This patch adds a percentage value next to the slider. Screenshot included.
> 
> 
> Diffs
> -----
> 
>   daemon/actions/bundled/brightnesscontrolconfig.h 7ba29a7 
>   daemon/actions/bundled/brightnesscontrolconfig.cpp 3b5aaad 
> 
> Diff: https://git.reviewboard.kde.org/r/123169/diff/
> 
> 
> Testing
> -------
> 
> Compiles. KCM displays the percentage corresponding to slider value in all tabs.
> 
> 
> File Attachments
> ----------------
> 
> Slider with percentage
>   https://git.reviewboard.kde.org/media/uploaded/files/2015/03/29/359f2e65-c52c-4e4e-9c8a-c75c3a9a8576__kcm.png
> 
> 
> Thanks,
> 
> Siddhartha Sahu
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20150329/fe54e6c9/attachment.htm>


More information about the kde-core-devel mailing list