Sync QML module padding to reflect system setting pages
ClosedPublic

Authored by davidedmundson on Feb 5 2019, 12:00 AM.

Details

Summary

In systemsettings 6a9df821ecfaa008206a260b20f595136e254821 it was
determined that the QML KCMs should do their own padding for some
reason.

QML KCMs now do this to match what's expected in system settings.

This change was not mirrored in KMultiDialog (and hence kcmshell5)

Populating d->modules had to move in order to make sure this was
available when the first page was added.

Test Plan

Ran kcmshell5 autostart (a qwidgets module) main frame still matched
where the buttons are
Ran kcmsheel5 lookandfeel (a qtquick module) main frame now matches
where the buttons are

Diff Detail

Repository
R295 KCMUtils
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 8024
Build 8042: arc lint + arc unit
davidedmundson created this revision.Feb 5 2019, 12:00 AM
Restricted Application added a project: Frameworks. · View Herald TranscriptFeb 5 2019, 12:00 AM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
davidedmundson requested review of this revision.Feb 5 2019, 12:00 AM
davidedmundson edited the test plan for this revision. (Show Details)Feb 5 2019, 12:18 AM
vpilo added a subscriber: vpilo.EditedFeb 7 2019, 2:16 PM

Works for me on KCMShell and SystemSettings.

broulik added a subscriber: broulik.Feb 8 2019, 9:26 AM

If you open multiple KCMs switching between QML- and Widget-ones leads to awkward outside margin changes with this patch.

kcmshell5 icons style

Switch between the two and observe how the sidebar dances (this is likely the same issue as when doing the same in System Settings in Icons mode rather than Sidebar)

vpilo added a comment.Feb 8 2019, 10:22 AM

If you open multiple KCMs switching between QML- and Widget-ones leads to awkward outside margin changes with this patch.

kcmshell5 icons style

Switch between the two and observe how the sidebar dances (this is likely the same issue as when doing the same in System Settings in Icons mode rather than Sidebar)

I didn't realize you could open multiple KCMs. With that, it's much easier for me to see that there's multiple differences between QML- and Widget- based KCMs. Disabling window decoration borders, it's even more visible.

davidedmundson edited the summary of this revision. (Show Details)
davidedmundson edited the test plan for this revision. (Show Details)

update

remove hardcoded value

Running kcmshell5 icons style was a good suggestion.

We have to change an internal layout and only one of the edges - which leads to this monstrosity :/

broulik accepted this revision.Feb 8 2019, 12:12 PM
This revision is now accepted and ready to land.Feb 8 2019, 12:12 PM
zzag added a subscriber: zzag.Feb 8 2019, 12:19 PM
zzag added inline comments.
src/kcmultidialog.cpp
40–41

It looks like includes are sorted. Maybe keep it that way?

This revision was automatically updated to reflect the committed changes.