D26538: Don't anchor AbstractApplicationHeader blindly

Aleix Pol Gonzalez noreply at phabricator.kde.org
Thu Jan 9 01:46:03 GMT 2020


apol created this revision.
Herald added a project: Kirigami.
Herald added a subscriber: plasma-devel.
apol requested review of this revision.

REVISION 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.

REPOSITORY
  R169 Kirigami

BRANCH
  master

REVISION DETAIL
  https://phabricator.kde.org/D26538

AFFECTED FILES
  src/controls/templates/AbstractApplicationHeader.qml

To: apol
Cc: plasma-devel, fbampaloukas, GB_2, domson, dkardarakos, ngraham, apol, ahiemstra, davidedmundson, mart, hein
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20200109/35c29871/attachment.html>


More information about the Plasma-devel mailing list