> On May 8, 2015, 9:14 p.m., Kai Uwe Broulik wrote:
> > lookandfeel/contents/loginmanager/Main.qml, line 97
> > <https://git.reviewboard.kde.org/r/123688/diff/1/?file=367729#file367729line97>
> >
> >     Why would you put that into a property? Just tive the Timer an id

Yes, done.
Would you have time to explain the difference to me (e.g. next week)? I'm 
always struggling with this stuff and the qt.io wiki doesn't help :-/


- David


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


On May 8, 2015, 6:48 p.m., David Kahles wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/123688/
> -----------------------------------------------------------
> 
> (Updated May 8, 2015, 6:48 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> -------
> 
> Disables the button and the passwordfield while the password check is
> running and hides the "Login Failed" message after 3 seconds.
> 
> 
> Diffs
> -----
> 
>   lookandfeel/contents/loginmanager/Main.qml 
> 528f9b4e43ee6b8636ca1c816d5bad556bede871 
> 
> Diff: https://git.reviewboard.kde.org/r/123688/diff/
> 
> 
> Testing
> -------
> 
> Login and failed login are working as expected.
> Tested in qmlscene and on two computers running Archlinux.
> 
> 
> Thanks,
> 
> David Kahles
> 
>

_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Reply via email to