<table><tr><td style="">sredman created this revision.<br />Herald added a project: KDE Connect.<br />Herald added a subscriber: kdeconnect.<br />sredman requested review of this revision.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D15108">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Previously, incoming messages were appened to a QList. This list was accidentally sorted because that's how Android returned them, but new messages were appended to the wrong end of the list. This patch specifically and intentionally sorts messages so new ones become visible</p></div></div><br /><div><strong>TEST PLAN</strong><div><ul class="remarkup-list">
<li class="remarkup-list-item">Open SMS GUI, verify that the most-recent messages are shown</li>
<li class="remarkup-list-item">Either send or recieve an SMS</li>
<li class="remarkup-list-item">Wait about 5s (I do not know why this is necessary. Probably some Android weirdness)</li>
<li class="remarkup-list-item">De-select the current conversation, then re-select it<ul class="remarkup-list">
<li class="remarkup-list-item">TODO: Make the app automatically respond to new messages</li>
</ul></li>
<li class="remarkup-list-item">The newly sent or recieved message should be shown in the most-recent position</li>
</ul></div></div><br /><div><strong>REPOSITORY</strong><div><div>R224 KDE Connect</div></div></div><br /><div><strong>BRANCH</strong><div><div>sms-message-sorting</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D15108">https://phabricator.kde.org/D15108</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>plugins/telephony/conversationsdbusinterface.cpp<br />
plugins/telephony/conversationsdbusinterface.h</div></div></div><br /><div><strong>To: </strong>sredman<br /><strong>Cc: </strong>kdeconnect, rmenezes, julioc, Leptopoda, timothyc, jdvr, yannux, Danial0_0, johnq, Pitel, adeen-s, SemperPeritus, ndavis, daniel.z.tg, jeanv, seebauer, bugzy, MayeulC, menasshock, tctara, apol<br /></div>