feedback kcm: Use a horizontal slider like GammaRay does
ClosedPublic

Authored by apol on Mon, Oct 7, 1:56 PM.

Details

Summary

Instead of a combo box, which is a bit harder on the wording.
Also get more texts from kuserfeedback itself, no reason to translate twice.

Test Plan

Tested manually

Diff Detail

Repository
R120 Plasma Workspace
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
apol created this revision.Mon, Oct 7, 1:56 PM
Restricted Application added a project: Plasma. · View Herald TranscriptMon, Oct 7, 1:56 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
apol requested review of this revision.Mon, Oct 7, 1:56 PM
ngraham accepted this revision.Mon, Oct 7, 3:17 PM
This revision is now accepted and ready to land.Mon, Oct 7, 3:17 PM
broulik added a subscriber: broulik.Mon, Oct 7, 3:21 PM
broulik added inline comments.
kcms/feedback/package/contents/ui/main.qml
79

Use Array.findIndex

83–84

Use onMoved which is the slider equivalent to onActivated

apol marked 2 inline comments as done.Mon, Oct 7, 10:44 PM
apol updated this revision to Diff 67471.Mon, Oct 7, 10:46 PM

Address kai's comments

apol updated this revision to Diff 67472.Mon, Oct 7, 10:50 PM

ehm, typo

broulik added inline comments.Tue, Oct 8, 12:16 PM
kcms/feedback/package/contents/ui/main.qml
79

Unused

broulik accepted this revision.Tue, Oct 8, 12:16 PM
This revision was automatically updated to reflect the committed changes.