From 4d0e81260c8a3aeaf2ad7be2c32c4e6447fa8bdd Mon Sep 17 00:00:00 2001 From: Nate Graham Date: Fri, 22 Oct 2021 13:54:23 -0600 Subject: [PATCH] kcms/users: Polish fingerprint UI on details page Now the button begins with a verb and ends with an ellipsis in conformance with the HIG, and there's also a little message that tells the user what works and what doesn't. This should help to manage expectations a bit. --- .../package/contents/ui/UserDetailsPage.qml | 36 +++++++++++++------ 1 file changed, 25 insertions(+), 11 deletions(-) diff --git a/kcms/users/package/contents/ui/UserDetailsPage.qml b/kcms/users/package/contents/ui/UserDetailsPage.qml index 8b26e42d2..0df3aeb31 100644 --- a/kcms/users/package/contents/ui/UserDetailsPage.qml +++ b/kcms/users/package/contents/ui/UserDetailsPage.qml @@ -172,17 +172,6 @@ SimpleKCM { } } - QQC2.Button { - flat: false - visible: kcm.fingerprintModel.deviceFound - text: i18n("Fingerprints") - icon.name: "fingerprint-gui" - onClicked: { - fingerprintDialog.account = user; - fingerprintDialog.openAndClear(); - } - } - Item { Layout.preferredHeight: deleteUser.height } @@ -219,6 +208,31 @@ SimpleKCM { onClicked: deleteMenu.open() } } + + QQC2.Button { + Layout.topMargin: deleteUser.height + Layout.alignment: Qt.AlignHCenter + flat: false + visible: kcm.fingerprintModel.deviceFound + text: i18n("Configure Fingerprint Authentication…") + icon.name: "fingerprint-gui" + onClicked: { + fingerprintDialog.account = user; + fingerprintDialog.openAndClear(); + } + } + QQC2.Label { + Layout.fillWidth: true + Layout.leftMargin: Kirigami.Units.largeSpacing * 2 + Layout.rightMargin: Kirigami.Units.largeSpacing * 2 + + visible: kcm.fingerprintModel.deviceFound + + text: xi18nc("@info", "Fingerprints can be used in place of a password when unlocking the screen and providing administrator permissions to applications and command-line programs that request them.Logging into the system with your fingerprint is not yet supported.") + + font: Kirigami.Theme.smallFont + wrapMode: Text.Wrap + } } Kirigami.OverlaySheet {