<table><tr><td style="">dvratil requested changes to this revision.<br />dvratil added a comment.<br />This revision now requires changes to proceed.
</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/D16013">View Revision</a></tr></table><br /><div><div><p>There seems to be a regression compared to the previous version when drawing events that span multiple weeks - in the original version the event is not rounded on week end and week start, while on your screenshot it appears the event has round corners on Sunday and Monday. Can you look into it?</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/D16013#inline-87096">View Inline</a><span style="color: #4b4d51; font-weight: bold;">monthgraphicsitems.cpp:239</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 style="color: #aa4000">const</span> <span class="n">QColor</span> <span class="n">textColor</span> <span style="color: #aa2211">=</span> <span class="n">EventViews</span><span style="color: #aa2211">::</span><span class="n">getTextColor</span><span class="p">(</span><span class="n">bgColor</span><span class="p">);</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">    <span style="color: #aa4000">int</span> <span class="n">textMargin</span> <span style="color: #aa2211">=</span> <span style="color: #601200">7</span><span class="p">;</span>
</div><div style="padding: 0 8px; margin: 0 4px; ">    <span class="n">p</span><span style="color: #aa2211">-></span><span class="n">setPen</span><span class="p">(</span><span class="n">textColor</span><span class="p">);</span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;"><tt style="background: #ebebeb; font-size: 13px;">const</tt></p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R76 PIM: Event Views</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D16013">https://phabricator.kde.org/D16013</a></div></div><br /><div><strong>To: </strong>ognarb, KDE PIM: Junior Jobs, KDE PIM, mlaurent, dvratil<br /><strong>Cc: </strong>kde-pim, maciejn, dvasin, rodsevich, winterz, vkrause, mlaurent, knauss, dvratil<br /></div>