Commit 2d066f00 authored by David Edmundson's avatar David Edmundson
Browse files

[sddm-theme] Fix initial focus after SDDM QQC2 Port

Summary:
QQC2 StackView has a major behavioural difference

QQuickStackViewPrivate::setCurrentItem
        item->setFocus(true);

This as a general rule makes sense, especially if you're pushing a whole
focus scope onto the stackView, it would automatically select the right
element. However as our base was not a focus scope the
SessionManagementScreen instance itself got focus, taking it away from
the child password box.

LockScreenUi has to be changed so that it sets the initial focus the
same even though the stackview is on QQC1.

Test Plan:
qmlscene in the sddm-theme directory with the dummydata setup
locked the screen regularly

Lots of gammaray

Reviewers: #plasma, ngraham

Reviewed By: ngraham

Subscribers: davidre, plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D25261
parent 4775c330
......@@ -25,7 +25,7 @@ import QtQuick.Controls 1.1
import org.kde.plasma.core 2.0 as PlasmaCore
import org.kde.plasma.components 2.0 as PlasmaComponents
Item {
FocusScope {
id: root
/*
......
......@@ -222,6 +222,7 @@ PlasmaCore.ColorScope {
showUserList: userList.y + mainStack.y > 0
focus: true // Drop this and hack below when porting this stackview to QQC2
Stack.onStatusChanged: {
// prepare for presenting again to the user
if (Stack.status == Stack.Activating) {
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment