[SMS App] Clean up runtime warning about undefined behavior
ClosedPublic

Authored by sredman on Oct 30 2018, 8:11 PM.

Details

Summary

Move ColumnLayout containing text elements inside of the graphical bounding rectangle

Test Plan

Pre patch:

  • Launch messaging app
  • Open conversation
  • Notice huge spew of warnings like: "qrc:/qml/ChatMessage.qml:84:5: QML ColumnLayout: Detected anchors on an item that is managed by a layout. This is undefined behavior; use Layout.alignment instead."

Post patch:

  • Launch messaging app
  • Open conversation
  • No warnings about undefined behavior

Diff Detail

Repository
R224 KDE Connect
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
sredman created this revision.Oct 30 2018, 8:11 PM
Restricted Application added a project: KDE Connect. · View Herald TranscriptOct 30 2018, 8:11 PM
Restricted Application added a subscriber: kdeconnect. · View Herald Transcript
sredman requested review of this revision.Oct 30 2018, 8:11 PM
apol accepted this revision.Oct 31 2018, 12:20 PM
This revision is now accepted and ready to land.Oct 31 2018, 12:20 PM
This revision was automatically updated to reflect the committed changes.