Workspace KCM Redesign Using Kirigami
ClosedPublic

Authored by furkantokac on May 18 2018, 10:31 PM.

Details

Summary

Workspace KCM is redesigned by using Kirigami. Functionality is same. Recommendations are welcome.

New design:

Old design:

Diff Detail

Repository
R119 Plasma Desktop
Branch
kcmWorkspaceFormatting
Lint
No Linters Available
Unit
No Unit Test Coverage
furkantokac created this revision.May 18 2018, 10:31 PM
Restricted Application added a project: Plasma. · View Herald TranscriptMay 18 2018, 10:31 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
furkantokac requested review of this revision.May 18 2018, 10:31 PM
furkantokac edited the summary of this revision. (Show Details)May 18 2018, 10:33 PM
furkantokac edited the summary of this revision. (Show Details)
furkantokac edited the summary of this revision. (Show Details)
furkantokac added reviewers: ngraham, romangg.
furkantokac edited the summary of this revision. (Show Details)May 18 2018, 10:46 PM
furkantokac edited the summary of this revision. (Show Details)

Workspace KCM is redesigned by using Kirigami.

furkantokac retitled this revision from Workspace KCM Redesign and Formatting Improvement to Workspace KCM Redesign Using Kirigami.May 18 2018, 11:16 PM
furkantokac edited the summary of this revision. (Show Details)
ngraham accepted this revision.May 19 2018, 1:50 AM

Nicely done! Code and visual changes look good to me. Tested it out and everything still works. Just two little comment formatting nitpicks, then this is good to go from my perspective! (wait for other reviews too before landing, please)

kcms/workspaceoptions/package/contents/ui/main.qml
32–39

"Visual behavior settings"

57–60

"Click behavior settings"

This revision is now accepted and ready to land.May 19 2018, 1:50 AM

Comment correction.

furkantokac edited the summary of this revision. (Show Details)May 21 2018, 7:46 AM
This revision was automatically updated to reflect the committed changes.

Looks gorgeous!