<table><tr><td style="">dvratil 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/D14163">View Revision</a></tr></table><br /><div><div><p>This change improves the existing code. I agree that a better solution is needed, but until we have that, I don't see any reason to not go forward with this, so +1 from me.</p>

<p><a href="https://phabricator.kde.org/p/pino/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@pino</a>, if you have anything that could replace this at some point, please share it. Maybe it could be adapted and put into Frameworks at some point so this imperfect thing can be dropped completely.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R174 PIM: KContacts</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D14163">https://phabricator.kde.org/D14163</a></div></div><br /><div><strong>To: </strong>vkrause, mlaurent, pino<br /><strong>Cc: </strong>dvratil, huftis, pino, kde-pim, dvasin, rodsevich, winterz, vkrause, mlaurent, knauss<br /></div>