> 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 > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
