<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/D25561">View Revision</a></tr></table><br /><div><div><p>Looks correct from deprecation-markup POV. Did you do a test build with EXCLUDE_DEPRECATED_BEFORE_AND_AT=5.65.0, to check if any tests or examples still use the deprecated signals and need some additions of #if KWIDGETSADDONS_ENABLE_DEPRECATED_SINCE(5, 65) in places?</p>

<p>When it comes to deprecated signals, seems those overloads have been present since introduction of KUrlLabel. Not sure about the usefulness of passing the url as argument with any signals, given that the url is a static property of the label, only set via the setter setUrl().<br />
Have you checked how the respective signals are used? Are there similar classes, what signals do they have?</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/D25561">https://phabricator.kde.org/D25561</a></div></div><br /><div><strong>To: </strong>mlaurent, dfaure, kossebau<br /><strong>Cc: </strong>cfeck, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns<br /></div>