Review Request 119797: Show warning in lockscreen when capslock is enabled

[Kai Uwe Broulik]

> On Aug. 18, 2014, 8:55 vorm., Martin Klapetek wrote:
> > lookandfeel/contents/lockscreen/LockScreen.qml, line 162
> > <https://git.reviewboard.kde.org/r/119797/diff/2/?file=305636#file305636line162>
> >
> >     Thinking about the label...
> >     
> >     What if we'd remove the "Warning: " part and either just leave it like that or make the font red? My reasoning being that a text suddenly appearing is enough to notice and therefore the "warning" part is not necessary. If the purpose of the "Warning" is to catch user's attention, this could be done as well by the red color with the advantage of less words needed to read and red color generally meaning "warning/attention".
> >     
> >     Oh and the "!" at the end should go, somewhere there is a guideline saying "don't make screaming interfaces" ;)

Yes! Those "Hint:", "Warning:", "Note:" etc always bugged me.


- Kai Uwe


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


On Aug. 14, 2014, 10:25 nachm., Dan Vrátil wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/119797/
> -----------------------------------------------------------
> 
> (Updated Aug. 14, 2014, 10:25 nachm.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> -------
> 
> This will show a warning when capslock is enabled, like it used to in KDE 4 screen locker.
> 
> 
> Diffs
> -----
> 
>   lookandfeel/contents/lockscreen/LockScreen.qml ba95cb9 
>   lookandfeel/contents/components/UserSelect.qml 7b605b1 
> 
> Diff: https://git.reviewboard.kde.org/r/119797/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Dan Vrátil
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20140818/d85db38e/attachment.html>