Revise title and description texts for the different modes
ClosedPublic

Authored by ngraham on Jan 9 2020, 7:29 PM.

Details

Summary

Currently, the text in the user feedback KCM is quite intimidating and scary to me for
various reasons:

  • The titles do not accurately describe what will actually be sent
  • The description text of "don't send anything" is inappropriately a description of everything
  • The same slightly awkward "here's what we don't send" text is releated in every description

Overall it does not give me confidence and make me want to submit information.

This patch (hopefully) improves the text by making the titles more descriptive, making the
description text simpler and more user-friendly, ensuring that every bit of description
text is actually describing its setting, and removing the "here's what we don't send" text,
which should be shown only once by the UI that presents this information (e.g. the Feedback
KCM).

Test Plan

How it looks now in the Feedback KCM (with D26552 applied):

How it looks in UserFeedbackConsole:

Diff Detail

Repository
R849 User Feedback Collection Framework
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
ngraham requested review of this revision.Jan 9 2020, 7:29 PM
ngraham created this revision.
ngraham retitled this revision from Revise title and description tests for the different modes to Revise title and description texts for the different modes.Jan 9 2020, 7:30 PM
ngraham edited the summary of this revision. (Show Details)
ngraham edited the test plan for this revision. (Show Details)
apol added a comment.Jan 9 2020, 9:29 PM

Maybe we should sell a bit the feature? We need people who care to enable it.

"Don't participate in any surveys" is rather dry. How about we talk about them as "usability surveys"?

src/provider/core/feedbackconfiguicontroller.cpp
180–181

Can we mention why it's not ideal?

ngraham added a comment.EditedJan 10 2020, 12:56 AM

The "Don't share anything" and "Don't participate in any surveys" text is just the explanation for the disabled status; it doesn't feel quite right to me to try to sell the feature there. If we want people to turn this feature on, making them visit the KCM or settings window will ensure that approximately 0% of people will do it no matter what the text says where, because what kind of weirdo explores System Settings for the fun of it? People like us do this, but regular users don't. What we should probably do is show a persistent notification or something after some number of days of usage following each major version update. Or maybe we could ask in the installer or the first-run wizard that everyone hates the idea of. ;)

That's probably a separate conversation though. :)

ngraham updated this revision to Diff 73168.Jan 10 2020, 12:57 AM

Call them "usability surveys"

I think the stand-alone app case (as seen in the second video) now misses the motivational explanation for why we do this, in the Plasma KCM this is added independently. Maybe just add that sentence to the top of the default config dialog?

Yeah, the idea behind this change was that the parent UI presenting these strings would include the explanation.

Does the standalone dialog live in this repo?

Yeah, the idea behind this change was that the parent UI presenting these strings would include the explanation.

Does the standalone dialog live in this repo?

Yes, there's a default implementation in KUserFeedback::FeedbackConfigWidget, which is used in UserFeedbackConsole.

ngraham updated this revision to Diff 73220.Jan 10 2020, 6:26 PM

Add explanation string at the top of the default dialog

ngraham edited the test plan for this revision. (Show Details)Jan 10 2020, 6:27 PM

Thanks! Maybe "KDE" -> "us", to keep the vendor-neutral wording, as this is also used outside of KDE. I'm ok with the rest of the wording changes.

You have to first introduce a group for "us" to refer to, or else it's not clear. :) If "KDE" is no-go, maybe it could say, "You can help the developers"

ngraham updated this revision to Diff 73285.Jan 11 2020, 5:05 PM

Tweak the wording of the explanatory text

Thanks, that works for me. +1 on this then.

This revision was not accepted when it landed; it landed in state Needs Review.Jan 13 2020, 2:57 AM
This revision was automatically updated to reflect the committed changes.