- User Since
- Apr 27 2015, 12:34 PM (229 w, 3 d)
Iterated with Nate's feedback
Wed, Sep 18
The patch looks good to me, is there a reason why it wasn't accepted?
Tue, Sep 17
Include a provider for plasmashell
Don't use "telemetry"
Use KDE's server
Why not? it could be useful. The alternative is darkness over what happened there.
Hey @ngraham, I introduced many of your points to the proposal. Here's some thoughts on the rest:
Shouldn't these be in Kirigami, then?
Improve how we display plasma settings
Include the policy, KUserfeedback decides the setting default, polishing.
Screenshot, I thought I'd uploaded it already.
Mon, Sep 16
Remove the dialog and read the settings globally for Plasma.
Moved the feedback settings to the About page.
Also make the property lowercase
Having 3 identical labels with slightly different conditions feels odd, maybe we should just take it into account if when assigning the property?
Thank you very much, Frederik!
Looks good :)
Sun, Sep 15
I'd write ON, uppercase, it's how it's usually represented.
Then maybe Qt shouldn't offer it? It's supposed to be writtable after all...
I'm interested in participating, could help organising but don't really have an easy venue.
Fri, Sep 13
Thu, Sep 12
Wed, Sep 11
Sun, Sep 8
Landing it because I need it for something I'm working on.
Other than that, LGTM +1
Sat, Sep 7
Thu, Sep 5
add missing files