----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/121419/#review71679 -----------------------------------------------------------
+1 from me - Lukáš Tinkl On Pro. 9, 2014, 11:15 odp., Kai Uwe Broulik wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/121419/ > ----------------------------------------------------------- > > (Updated Pro. 9, 2014, 11:15 odp.) > > > Review request for Plasma and Daniel Vrátil. > > > Repository: plasma-workspace > > > Description > ------- > > Don't use a separate label which causes the layout to shift but just use the > notification label and prepend the warning to any actual message. > > > Diffs > ----- > > lookandfeel/contents/lockscreen/LockScreen.qml ca9beb2 > > Diff: https://git.reviewboard.kde.org/r/121419/diff/ > > > Testing > ------- > > Yup. Looks better imho. > > > File Attachments > ---------------- > > Just the warning > > https://git.reviewboard.kde.org/media/uploaded/files/2014/12/09/ff5b6404-f55a-4e72-afd3-1348f2e12258__capslockwarning1.png > Warning with failed unlock > > https://git.reviewboard.kde.org/media/uploaded/files/2014/12/09/f29104c3-3b76-4508-b9be-3fad2a3d7baa__capslockwarning2.png > > > Thanks, > > Kai Uwe Broulik > >
_______________________________________________ Plasma-devel mailing list [email protected] https://mail.kde.org/mailman/listinfo/plasma-devel
