<table><tr><td style="">knauss added a comment.
</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/D8125" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>Also to do this with the root block would add a lot of global logic into the grantlee themes. Maybe we find a better solution for this.</p></div></div><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/D8125#inline-33968" rel="noreferrer">View Inline</a><span style="color: #4b4d51; font-weight: bold;">asiconpart.html:4</span></div>
<div style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; white-space: pre-wrap; clear: both; padding: 4px 0; margin: 0;"><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);"><span class="p"><</span><span style="color: #00702a">div</span> <span style="color: #354bb3">id</span><span style="color: #aa2211">=</span><span style="color: #766510">"attachmentDiv{{block.attachmentIndex}}"</span><span class="p">></span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">{% endif %}
</div><div style="padding: 0 8px; margin: 0 4px; "><span class="p"><</span><span style="color: #00702a">hr</span><span class="p">/></span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">not really happy with this change, becuase it makes it harder to read. and mire error prune to forget a clossing "</div>".<br />
Also maybe we want this logic to be done globally, to deside, ok this is an attachment, so we add this block to the part. IMO it doesn't makes sense to add this to every plugin. And to make the links usefull for kmail to jump to attachments we need these links.</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R94 PIM: Message Library</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D8125" rel="noreferrer">https://phabricator.kde.org/D8125</a></div></div><br /><div><strong>To: </strong>vkrause, knauss<br /><strong>Cc: </strong>KDE PIM, dvasin, winterz, vkrause, mlaurent, knauss, dvratil<br /></div>