Changeset View
Changeset View
Standalone View
Standalone View
src/provider/core/feedbackconfiguicontroller.cpp
| Show First 20 Lines • Show All 131 Lines • ▼ Show 20 Line(s) | |||||
| 132 | QString FeedbackConfigUiController::telemetryName(KUserFeedback::Provider::TelemetryMode mode) const | 132 | QString FeedbackConfigUiController::telemetryName(KUserFeedback::Provider::TelemetryMode mode) const | ||
| 133 | { | 133 | { | ||
| 134 | switch (mode) { | 134 | switch (mode) { | ||
| 135 | case Provider::NoTelemetry: | 135 | case Provider::NoTelemetry: | ||
| 136 | return tr("Disabled"); | 136 | return tr("Disabled"); | ||
| 137 | case Provider::BasicSystemInformation: | 137 | case Provider::BasicSystemInformation: | ||
| 138 | return tr("Basic system information"); | 138 | return tr("Basic system information"); | ||
| 139 | case Provider::BasicUsageStatistics: | 139 | case Provider::BasicUsageStatistics: | ||
| 140 | return tr("Basic usage statistics"); | 140 | return tr("Basic system information and usage statistics"); | ||
| 141 | case Provider::DetailedSystemInformation: | 141 | case Provider::DetailedSystemInformation: | ||
| 142 | return tr("Detailed system information"); | 142 | return tr("Detailed system information and basic usage statistics"); | ||
| 143 | case Provider::DetailedUsageStatistics: | 143 | case Provider::DetailedUsageStatistics: | ||
| 144 | return tr("Detailed usage statistics"); | 144 | return tr("Detailed system information and usage statistics"); | ||
| 145 | } | 145 | } | ||
| 146 | 146 | | |||
| 147 | return {}; | 147 | return {}; | ||
| 148 | } | 148 | } | ||
| 149 | 149 | | |||
| 150 | QString FeedbackConfigUiController::telemetryDescription(KUserFeedback::Provider::TelemetryMode mode) const | 150 | QString FeedbackConfigUiController::telemetryDescription(KUserFeedback::Provider::TelemetryMode mode) const | ||
| 151 | { | 151 | { | ||
| 152 | const auto name = applicationName(); | 152 | const auto name = applicationName(); | ||
| 153 | if (name.isEmpty()) { | 153 | if (name.isEmpty()) { | ||
| 154 | switch (mode) { | 154 | switch (mode) { | ||
| 155 | case Provider::NoTelemetry: | 155 | case Provider::NoTelemetry: | ||
| 156 | return tr( | 156 | return tr( | ||
| 157 | "We make this application for you. You can help us improve it by contributing information on how you use it. " | 157 | "Don't share anything" | ||
| 158 | "This allows us to make sure we focus on things that matter to you.\n" | | |||
| 159 | "Contributing statistics is of course entirely anonymous, will not use any kind of unique identifier and " | | |||
| 160 | "will not cover any data you process with this application." | | |||
| 161 | ); | 158 | ); | ||
| 162 | case Provider::BasicSystemInformation: | 159 | case Provider::BasicSystemInformation: | ||
| 163 | return tr( | 160 | return tr( | ||
| 164 | "Share basic system information. " | 161 | "Share basic system information such as the version of the application and the operating system" | ||
| 165 | "No unique identification is included, nor data processed with the application." | | |||
| 166 | ); | 162 | ); | ||
| 167 | case Provider::BasicUsageStatistics: | 163 | case Provider::BasicUsageStatistics: | ||
| 168 | return tr( | 164 | return tr( | ||
| 169 | "Share basic system information and basic statistics on how often you use the application. " | 165 | "Share basic system information and basic statistics on how often you use the application" | ||
| 170 | "No unique identification is included, nor data processed with the application." | | |||
| 171 | ); | 166 | ); | ||
| 172 | case Provider::DetailedSystemInformation: | 167 | case Provider::DetailedSystemInformation: | ||
| 173 | return tr( | 168 | return tr( | ||
| 174 | "Share basic statistics on how often you use the application, as well as detailed information about your system. " | 169 | "Share basic statistics on how often you use the application, as well as more detailed information about your system" | ||
| 175 | "No unique identification is included, nor data processed with the application." | | |||
| 176 | ); | 170 | ); | ||
| 177 | case Provider::DetailedUsageStatistics: | 171 | case Provider::DetailedUsageStatistics: | ||
| 178 | return tr( | 172 | return tr( | ||
| 179 | "Share detailed system information and statistics on how often individual features of the application are used. " | 173 | "Share detailed system information and statistics on how often individual features of the application are used." | ||
| 180 | "No unique identification is included, nor data processed with the application." | | |||
| 181 | ); | 174 | ); | ||
| 182 | } | 175 | } | ||
| 183 | } else { | 176 | } else { | ||
| 184 | switch (mode) { | 177 | switch (mode) { | ||
| 185 | case Provider::NoTelemetry: | 178 | case Provider::NoTelemetry: | ||
| 186 | return tr( | 179 | return tr( | ||
| 187 | "We make %1 for you. You can help us improve it by contributing information on how you use it. " | 180 | "Don't share anything" | ||
| 188 | "This allows us to make sure we focus on things that matter to you.\n" | 181 | ); | ||
apol: Can we mention why it's not ideal? | |||||
| 189 | "Contributing statistics is of course entirely anonymous, will not use any kind of unique identifier and " | | |||
| 190 | "will not cover any data you process with %1." | | |||
| 191 | ).arg(name); | | |||
| 192 | case Provider::BasicSystemInformation: | 182 | case Provider::BasicSystemInformation: | ||
| 193 | return tr( | 183 | return tr( | ||
| 194 | "Share basic system information. " | 184 | "Share basic system information such as the version of %1 and and the operating system" | ||
| 195 | "No unique identification is included, nor data processed with %1." | | |||
| 196 | ).arg(name); | 185 | ).arg(name); | ||
| 197 | case Provider::BasicUsageStatistics: | 186 | case Provider::BasicUsageStatistics: | ||
| 198 | return tr( | 187 | return tr( | ||
| 199 | "Share basic system information and basic statistics on how often you use %1. " | 188 | "Share basic system information and basic statistics on how often you use %1" | ||
| 200 | "No unique identification is included, nor data processed with %1." | | |||
| 201 | ).arg(name); | 189 | ).arg(name); | ||
| 202 | case Provider::DetailedSystemInformation: | 190 | case Provider::DetailedSystemInformation: | ||
| 203 | return tr( | 191 | return tr( | ||
| 204 | "Share basic statistics on how often you use %1, as well as detailed information about your system. " | 192 | "Share basic statistics on how often you use %1, as well as more detailed information about your system" | ||
| 205 | "No unique identification is included, nor data processed with %1." | | |||
| 206 | ).arg(name); | 193 | ).arg(name); | ||
| 207 | case Provider::DetailedUsageStatistics: | 194 | case Provider::DetailedUsageStatistics: | ||
| 208 | return tr( | 195 | return tr( | ||
| 209 | "Share detailed system information and statistics on how often individual features of %1 are used. " | 196 | "Share detailed system information and statistics on how often individual features of %1 are used." | ||
| 210 | "No unique identification is included, nor data processed with %1." | | |||
| 211 | ).arg(name); | 197 | ).arg(name); | ||
| 212 | } | 198 | } | ||
| 213 | } | 199 | } | ||
| 214 | 200 | | |||
| 215 | return QString(); | 201 | return QString(); | ||
| 216 | } | 202 | } | ||
| 217 | 203 | | |||
| 218 | QString FeedbackConfigUiController::telemetryModeDetails(int telemetryIndex) const | 204 | QString FeedbackConfigUiController::telemetryModeDetails(int telemetryIndex) const | ||
| Show All 36 Lines | |||||
| 255 | 241 | | |||
| 256 | QString FeedbackConfigUiController::surveyModeDescription(int surveyIndex) const | 242 | QString FeedbackConfigUiController::surveyModeDescription(int surveyIndex) const | ||
| 257 | { | 243 | { | ||
| 258 | const auto name = applicationName(); | 244 | const auto name = applicationName(); | ||
| 259 | if (name.isEmpty()) { | 245 | if (name.isEmpty()) { | ||
| 260 | switch (surveyIndex) { | 246 | switch (surveyIndex) { | ||
| 261 | case 0: | 247 | case 0: | ||
| 262 | return tr( | 248 | return tr( | ||
| 263 | "We make this application for you. In order to ensure it actually does what you need it to do we " | 249 | "Don't participate in usability surveys" | ||
| 264 | "would like to ask you about your use cases and your feedback, in the form of a web survey." | | |||
| 265 | ); | 250 | ); | ||
| 266 | case 1: | 251 | case 1: | ||
| 267 | return tr( | 252 | return tr( | ||
| 268 | "I will occasionally participate in web surveys about the application, not more than four times a year though." | 253 | "Participate in surveys about the application not more than four times a year" | ||
| 269 | ); | 254 | ); | ||
| 270 | case 2: | 255 | case 2: | ||
| 271 | return tr( | 256 | return tr( | ||
| 272 | "I will participate in web surveys whenever one is available. Surveys can of course be deferred or skipped." | 257 | "Participate in surveys about the application whenever one is available (they can be deferred or skipped)" | ||
| 273 | ); | 258 | ); | ||
| 274 | } | 259 | } | ||
| 275 | } else { | 260 | } else { | ||
| 276 | switch (surveyIndex) { | 261 | switch (surveyIndex) { | ||
| 277 | case 0: | 262 | case 0: | ||
| 278 | return tr( | 263 | return tr( | ||
| 279 | "We make %1 for you. In order to ensure it actually does what you need it to do we " | 264 | "Don't participate in usability surveys" | ||
| 280 | "would like to ask you about your use cases and your feedback, in the form of a web survey." | | |||
| 281 | ).arg(name); | 265 | ).arg(name); | ||
| 282 | case 1: | 266 | case 1: | ||
| 283 | return tr( | 267 | return tr( | ||
| 284 | "I will occasionally participate in web surveys about %1, not more than four times a year though." | 268 | "Participate in surveys about %1 not more than four times a year" | ||
| 285 | ).arg(name); | 269 | ).arg(name); | ||
| 286 | case 2: | 270 | case 2: | ||
| 287 | return tr( | 271 | return tr( | ||
| 288 | "I will participate in web surveys about %1 whenever one is available. Surveys can of course be deferred or skipped." | 272 | "Participate in surveys about %1 whenever one is available (they can be deferred or skipped)" | ||
| 289 | ).arg(name); | 273 | ).arg(name); | ||
| 290 | } | 274 | } | ||
| 291 | } | 275 | } | ||
| 292 | 276 | | |||
| 293 | return QString(); | 277 | return QString(); | ||
| 294 | } | 278 | } | ||
| 295 | 279 | | |||
| 296 | QString FeedbackConfigUiController::applicationName() const | 280 | QString FeedbackConfigUiController::applicationName() const | ||
| Show All 12 Lines | |||||
Can we mention why it's not ideal?