Don't anchor AbstractApplicationHeader blindly
ClosedPublic

Authored by apol on Jan 9 2020, 1:46 AM.

Details

Summary

If we do it, then QtQuick.Layouts complains when it's placed inside because
Layouts.fillWidth should be used instead.

Test Plan

Now I don't get the following warning:
QML AbstractApplicationHeader: Detected anchors on an item that is managed by a layout. This is undefined behavior; use Layout.alignment instead.

Diff Detail

Repository
R169 Kirigami
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.Jan 9 2020, 1:46 AM
Restricted Application added a project: Kirigami. · View Herald TranscriptJan 9 2020, 1:46 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
apol requested review of this revision.Jan 9 2020, 1:46 AM
mart accepted this revision.Jan 10 2020, 11:46 AM
This revision is now accepted and ready to land.Jan 10 2020, 11:46 AM
This revision was automatically updated to reflect the committed changes.