Differential D13422

[effects/slide] Use FormLayout in KCM
ClosedPublic
Actions

Authored by zzag on Jun 7 2018, 9:03 PM.

Details

Reviewers
mart
Group Reviewers
KWin
Commits
R108:1a1845b7d757: [effects/slide] Use FormLayout in KCM
Summary

HBoxLayout was a bad choice for label-spinbox pairs. Use appropriate
layout(FormLayout) for such things.

Test Plan

The KCM still looks the same.

Diff Detail

Repository
R108 KWin
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
zzag created this revision.Jun 7 2018, 9:03 PM
Restricted Application added a project: KWin. · View Herald TranscriptJun 7 2018, 9:03 PM
Restricted Application added a subscriber: kwin. · View Herald Transcript
zzag requested review of this revision.Jun 7 2018, 9:03 PM
mart accepted this revision.Jun 11 2018, 10:01 AM
This revision is now accepted and ready to land.Jun 11 2018, 10:01 AM
Closed by commit R108:1a1845b7d757: [effects/slide] Use FormLayout in KCM (authored by zzag). · Explain WhyJun 11 2018, 12:32 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Meffects/slide/slide_config.ui (156 lines)

Diff 36001

View Options

effects/slide/slide_config.ui

Loading...