<table><tr><td style="">kossebau 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/D23927">View Revision</a></tr></table><br /><div><div><p>When going to adapt some code to follow the deprecation, I now look though at some inconsistency in the API:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">KTitleWidget has a property "pixmap"</li>
<li class="remarkup-list-item">the <tt style="background: #ebebeb; font-size: 13px;">setPixmap</tt> methods all set this pixmap, with overloads for convenience</li>
<li class="remarkup-list-item"><tt style="background: #ebebeb; font-size: 13px;">setIcon</tt> is without a getter and a documented property, the relation to the pixmap property is unclear just by the API names</li>
</ul>

<p>With this in mind, I fear I consider this change a regression in API quality. I'd propose to revert this, or go the full mile and make "icon" a real property, deprecating the pixmap one. What do you think?</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R236 KWidgetsAddons</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D23927">https://phabricator.kde.org/D23927</a></div></div><br /><div><strong>To: </strong>vkrause, dfaure<br /><strong>Cc: </strong>kossebau, dhaumann, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns<br /></div>