<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>Another thing I just noticed: You use border width 1 unconditionally, when it was 2 unconditionally before.</p>

<p>I think that border width 1 is better if the border does not carry information, so +1 in case of CategoryOnly or ResourceOnly. However, if the border DOES cary information (ResourceInsideCategoryOutside, CategoryInsideResourceOutside), that information might be harder to grasp. I experimented a little bit and found this most useful:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">{Category,Resource}Only => border width 1</li>
<li class="remarkup-list-item">Otherwise => border width 2</li>
</ul>

<p>However, we should not change too much in a single commit, so I suggest to leave it at "ft" for now and then improve border width in another Diff. If you still choose to ignore "ft" completely, you should remove the static const that is then unused.</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>