Review Request 126930: sddm theme: Add keyboard layout button

David Rosca nowrep at gmail.com
Sat Jan 30 22:59:35 UTC 2016


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

(Updated Jan. 30, 2016, 10:59 p.m.)


Review request for Plasma.


Changes
-------

`visible: keyboard.layouts.length > 1` as in lock screen


Repository: plasma-workspace


Description
-------

Show keyboard layout button in sddm theme, same as in lock screen.


Diffs (updated)
-----

  lookandfeel/contents/loginmanager/Main.qml 1bbefec 
  lookandfeel/contents/loginmanager/dummydata/keyboard.qml PRE-CREATION 

Diff: https://git.reviewboard.kde.org/r/126930/diff/


Testing
-------

Looks & works exactly the same as in lock screen.

Btw, there is a comment saying that password input must be first child to get focus (automatically when login screen is shown?), but it is not true. It works fine (in all cases = qmlscene, sddm-greeter --test-mode and in actual sddm session) even with password input being second child. So, remove the comment?


Thanks,

David Rosca

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


More information about the Plasma-devel mailing list