Differential D24011 Diff 66351 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 QtControls
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​
33​ Kirigami.FormLayout {
34​ Kirigami.Heading {
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.FormData.isSection: true
36​ Layout.fillWidth: true
37​ wrapMode: Text.WordWrap
38​ text: i18n("Global")
39​ }
40​ QtControls.CheckBox {
41​ Kirigami.FormData.label: i18n("Send usage statistics")
42​ checked: kcm.feedbackEnabled
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​ onToggled: kcm.feedbackEnabled = checked
44​ }
45​ QtControls.Label {
46​ text: i18n("This will only affect KDE software")
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​ }
ognarbUnsubmitted
Done

strange </p> tag in the text

ognarb: strange </p> tag in the text
48​
49​ Kirigami.Heading {
50​ Kirigami.FormData.isSection: true
51​ enabled: kcm.feedbackEnabled
52​ Layout.fillWidth: true
53​ wrapMode: Text.WordWrap
54​ text: i18n("Plasma")
55​ }
56​
57​ QtControls.Frame {
58​ Layout.preferredWidth: parent.width * 0.75
ngrahamUnsubmitted
Not Done

Ew gross, frames

ngraham: Ew gross, frames
59​ enabled: kcm.feedbackEnabled
60​ ColumnLayout {
61​ anchors.fill: parent
62​ QtControls.Label {
63​ Layout.fillWidth: true
64​ wrapMode: Text.WordWrap
65​ text: i18n("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.")
66​ }
67​ QtControls.Label {
68​ Layout.fillWidth: true
69​ wrapMode: Text.WordWrap
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​ text: i18n("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.")
71​ }
72​ QtControls.Label {
73​ Layout.fillWidth: true
74​ wrapMode: Text.WordWrap
75​ text: i18n("You can read more about our policy in the following link.")
76​ }
77​
78​ Kirigami.UrlButton {
79​ url: "https://kde.org/privacypolicy-apps.php"
80​ }
81​
82​ QtControls.ComboBox {
83​ id: statisticsModeCombo
84​ enabled: kcm.feedbackEnabled
85​ Layout.fillWidth: true
86​ currentIndex: 2
87​ textRole: "text"
88​ model: ListModel { id: modeOptions }
89​
90​ Component.onCompleted: {
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​ modeOptions.append({text: i18n("Basic System Information"), value: UserFeedback.Provider.BasicSystemInformation})
92​ modeOptions.append({text: i18n("Basic Usage Information"), value: UserFeedback.Provider.BasicUsageStatistics})
93​ modeOptions.append({text: i18n("Detailed System Information"), value: UserFeedback.Provider.DetailedSystemInformation})
94​ modeOptions.append({text: i18n("Detailed Usage Information"), value: UserFeedback.Provider.DetailedUsageStatistics})
95​
96​ for(var i = 0, c=modeOptions.count; i<c; ++i) {
97​ if (modeOptions.get(i).value === kcm.plasmaFeedbackLevel) {
98​ currentIndex = i;
99​ break;
100​ }
101​ }
102​ }
103​
104​ onActivated: {
105​ kcm.plasmaFeedbackLevel = modeOptions.get(index).value;
106​ }
107​ }
108​ }
109​ }
110​ }
111​}
112​