D5961: Adoption of KUserFeedback for Discover

[Aleix Pol Gonzalez]

apol added a comment.


  In D5961#540955 <https://phabricator.kde.org/D5961#540955>, @ngraham wrote:
  
  > So this patch adds a semi-hidden menu item on the About page that opens the User Feedback KCM in a KCMShell window. I think this needs a better UX:
  >
  > 1. We finally need a real Settings page/window for Discover if we're going to be adding more user-configurable settings like this one. The About page isn't an appropriate place for this. Maybe we should Change Sources to Settings and add it there.
  
  
  True, maybe we should re-rename Sources to settings again.
  
  > 2. Shouldn't the UI here be Discover-specific? The global KCM doesn't seem relevant since it talks about turning on or off telemetry for all KDE apps and allows configuring stuff for Plasma specifically. Strictly speaking, Discover isn't a part of Plasma. And from a user perspective, Discover is just an app among other apps.
  
  Discover is part of plasma and the information shared should be centrally configured.

REPOSITORY
  R134 Discover Software Store

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

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