D24011: Provide a telemetrics kcm module for Plasma

Aleix Pol Gonzalez noreply at phabricator.kde.org
Wed Oct 2 14:42:19 BST 2019


apol added inline comments.

INLINE COMMENTS

> ngraham wrote in main.qml:68
> 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.

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.

> ngraham wrote in main.qml:41
> #1 isn't done yet (it still talks about Plasma, not KDE software more generally)

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

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D24011

To: apol, #plasma, vkrause, ngraham
Cc: aspotashev, ngraham, bcooksley, davidedmundson, ognarb, broulik, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, ragreen, ZrenBot, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20191002/49a8ee56/attachment.html>


More information about the Plasma-devel mailing list