<table><tr><td style="">whiting 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/D7968" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>Hey all, this change breaks Kompare <a href="https://bugs.kde.org/show_bug.cgi?id=390024" class="remarkup-link" target="_blank" rel="noreferrer">https://bugs.kde.org/show_bug.cgi?id=390024</a> which watches a KUrlRequester's textChanged signal to update a button's enabled state. We are not seeing the textChanged signal on the KUrlRequester anymore when programatically setting the url with the KUrlComboBox we give to the KUrlRequester via it's setUrl method. Looking at KUrlComboBox::setUrl it's blocking signals in the body of setUrl and since we are not connecting to the QLineEdit anymore we never get that signal.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R241 KIO</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D7968" rel="noreferrer">https://phabricator.kde.org/D7968</a></div></div><br /><div><strong>To: </strong>mwolff, apol, dfaure<br /><strong>Cc: </strong>whiting, Frameworks, michaelh<br /></div>