<table><tr><td style="">knauss accepted this revision.<br />knauss added inline comments.<br />This revision is now accepted and ready to land.
</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/D8087" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D8087#inline-33773" rel="noreferrer">View Inline</a><span style="color: #4b4d51; font-weight: bold;">vkrause</span> wrote in <span style="color: #4b4d51; font-weight: bold;">defaultrenderer.cpp:955</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">Factoring out the whole attachment block seems more useful indeed. But that's IMHO out of scope for this commit. Let's revisit cleaning up the render implementation once we have the API sorted out.</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">fair enough.</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R94 PIM: Message Library</div></div></div><br /><div><strong>BRANCH</strong><div><div>top</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D8087" rel="noreferrer">https://phabricator.kde.org/D8087</a></div></div><br /><div><strong>To: </strong>vkrause, knauss<br /><strong>Cc: </strong>mlaurent, KDE PIM, dvasin, winterz, vkrause, knauss, dvratil<br /></div>