Differential D24011 Diff 67192 kcms/feedback/package/contents/ui/main.qml

Changeset View

Standalone View

View Options

kcms/feedback/package/contents/ui/main.qml

  • This file was added.
1​/*
2​ * Copyright (C) 2019 David Edmundson <davidedmundson@kde.org>
3​ * Copyright (C) 2019 Aleix Pol Gonzalez <aleixpol@kde.org>
4​ *
5​ * This library is free software; you can redistribute it and/or
6​ * modify it under the terms of the GNU Library General Public
7​ * License as published by the Free Software Foundation; either
8​ * version 2 of the License, or (at your option) any later version.
9​ *
10​ * This library is distributed in the hope that it will be useful,
11​ * but WITHOUT ANY WARRANTY; without even the implied warranty of
12​ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
13​ * Library General Public License for more details.
14​ *
15​ * You should have received a copy of the GNU Library General Public License
16​ * along with this library; see the file COPYING.LIB. If not, write to
17​ * the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
18​ * Boston, MA 02110-1301, USA.
19​*/
20​
21​import QtQuick 2.1
22​import QtQuick.Layouts 1.1
23​import QtQuick.Controls 2.3 as QQC2
24​import org.kde.kirigami 2.6 as Kirigami
ngrahamUnsubmitted
Not Done

"as QQC2"

ngraham: "as QQC2"
25​import org.kde.userfeedback 1.0 as UserFeedback
26​import org.kde.kcm 1.2
27​
28​SimpleKCM {
29​ id: root
30​
31​ ConfigModule.buttons: ConfigModule.Defaults | ConfigModule.Apply
32​ leftPadding: width * 0.1
33​ rightPadding: leftPadding
34​
davidedmundsonUnsubmitted
Not Done

Generally we only want a label if it's deliberately spanning and not associated with a control.

Otherwise, we want:

Kirigami.FormData.label: i18n("Blah blah:")
davidedmundson: Generally we only want a label if it's deliberately spanning and not associated with a control.
35​ Kirigami.FormLayout {
36​ QQC2.CheckBox {
37​ Kirigami.FormData.label: i18n("Feedback:")
38​ checked: kcm.feedbackEnabled
39​ onToggled: kcm.feedbackEnabled = checked
40​ text: i18n("Allow KDE software to collect anomymous usage information")
41​ }
42​
ngrahamUnsubmitted
Done
  1. Since this is no longer just about Plasma, maybe the first sentence should say, "We make software for you".
  1. Use kuit formatting for newlines: xi18nc("@info", "Line 1<nl/>Line 2<nl/>Line 3<nl/>etc.");
ngraham: 1. Since this is no longer just about Plasma, maybe the first sentence should say, "We make…
ngrahamUnsubmitted
Not Done

#1 isn't done yet (it still talks about Plasma, not KDE software more generally)

ngraham: #1 isn't done yet (it still talks about Plasma, not KDE software more generally)
apolAuthorUnsubmitted
Not Done

But it's not true, only Plasma components will follow the setting. Only the checkbox will be used elsewhere. (hence the old ugly frame).

apol: But it's not true, only Plasma components will follow the setting. Only the checkbox will be…
43​ Kirigami.Separator {
44​ Layout.fillWidth: true
45​ }
46​
ngrahamUnsubmitted
Not Done

This is aligned in a weird way. Let's come up with a better way to communicate this information. Maybe we could add it to the text block.

ngraham: This is aligned in a weird way. Let's come up with a better way to communicate this information.
47​ QQC2.Label {
ognarbUnsubmitted
Done

strange </p> tag in the text

ognarb: strange </p> tag in the text
48​ Kirigami.FormData.label: i18n("Plasma:")
49​ Layout.maximumWidth: root.width * 0.5
50​ Layout.alignment: Qt.AlignHCenter
51​ Layout.fillWidth: true
52​ wrapMode: Text.WordWrap
53​ text: xi18nc("@info", "We make Plasma for you. You can help us improve it by contributing information on how you use it. This allows us to focus on things that matter to you.<nl/><nl/>Contributing usage information is optional and entirely anonymous. It will not associate the data with any kind of unique identifier, and will never track the documents you open, the websites you visit, or any other kind of personal information.<nl/><nl/>You can read more about our policy in the following link.")
54​ }
55​
56​ Kirigami.UrlButton {
57​ Layout.leftMargin: Kirigami.Units.gridUnit
58​ Layout.rightMargin: Kirigami.Units.gridUnit
ngrahamUnsubmitted
Not Done

Ew gross, frames

ngraham: Ew gross, frames
59​ url: "https://kde.org/privacypolicy-apps.php"
60​ }
61​
62​ QQC2.ComboBox {
63​ id: statisticsModeCombo
64​ enabled: kcm.feedbackEnabled
65​ Layout.fillWidth: true
66​ textRole: "text"
67​ model: ListModel { id: modeOptions }
68​
69​ Component.onCompleted: {
ngrahamUnsubmitted
Not Done

These strings still need to be re-worded, and possibly collapsed, and possible the whole UI needs to be re-thought. What if I want to submit detailed system information but only basic usage information? Maybe we need two combo boxes, one for system info and one for usage info.

ngraham: These strings still need to be re-worded, and possibly collapsed, and possible the whole UI…
apolAuthorUnsubmitted
Not Done

That's not how it's implemented in KUserfeedback.

enum TelemetryMode {
        NoTelemetry, ///< Transmit no data at all.
        BasicSystemInformation = 0x10, ///< Transmit basic information about the system.
        BasicUsageStatistics = 0x20, ///< Transmit basic usage statistics.
        DetailedSystemInformation = 0x30, ///< Transmit detailed system information.
        DetailedUsageStatistics = 0x40, ///< Transmit detailed usage statistics.
    };

DetailedSystemInformation = BasicUsageStatistics+BasicSystemInformation

And I don't think it's wrong. It's about making it easy for users to choose to help us learn about their behaviour.

apol: That's not how it's implemented in KUserfeedback. ``` enum TelemetryMode { NoTelemetry…
70​ modeOptions.append({text: i18n("Basic System Information"), value: UserFeedback.Provider.BasicSystemInformation})
71​ modeOptions.append({text: i18n("Basic Usage Information"), value: UserFeedback.Provider.BasicUsageStatistics})
72​ modeOptions.append({text: i18n("Detailed System Information"), value: UserFeedback.Provider.DetailedSystemInformation})
73​ modeOptions.append({text: i18n("Detailed Usage Information"), value: UserFeedback.Provider.DetailedUsageStatistics})
74​
75​ for(var i = 0, c=modeOptions.count; i<c; ++i) {
76​ if (modeOptions.get(i).value === kcm.plasmaFeedbackLevel) {
77​ currentIndex = i;
78​ break;
79​ }
80​ }
81​ if (currentIndex < 0)
82​ currentIndex = 2
83​ }
84​
85​ onActivated: {
86​ kcm.plasmaFeedbackLevel = modeOptions.get(index).value;
87​ }
88​ }
89​
90​ UserFeedback.FeedbackConfigUiController {
ngrahamUnsubmitted
Not Done

This label never shows up for me, no matter which entry is selected in the combo box.

ngraham: This label never shows up for me, no matter which entry is selected in the combo box.
91​ id: feedbackController
92​ }
93​
94​ QQC2.Label {
95​ Layout.maximumWidth: root.width * 0.5
96​ wrapMode: Text.WordWrap
97​ text: feedbackController.telemetryDescription(modeOptions.get(statisticsModeCombo.currentIndex).value)
98​ }
99​ }
100​}
101​