<table><tr><td style="">dkurz 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/D16013">View Revision</a></tr></table><br /><div><div><p>The margin helps to immediately grasp that the event starts/ends on that day, just like the radius. I think there should be a margin iff there is a radius.</p>

<p>In the current code, there is also no border drawn when an event continues in that direction (left for !end, right for !begin). Why did you change that? I think the border on the right side of your event on 21st makes the impression of a gap between the right border of the view and the event. My suggestion:</p>

<p>begin => radius, margin and border on the left side<br />
not begin => no radius, margin or border on the left side</p>

<p>end => radius, margin and border on the right side<br />
not end => no radius, margin or border on the right side</p>

<p>For short: Either all radius, margin and border, or none at all, per side</p>

<p>I hope I did not confuse what begin/end items are in the first place.</p></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>dkurz, kde-pim, maciejn, dvasin, rodsevich, winterz, vkrause, mlaurent, knauss, dvratil<br /></div>