<table><tr><td style="">mpyne created this revision.<br />mpyne added reviewers: Frameworks, Plasma, mart.<br />Herald added a project: Kirigami.<br />Herald added a subscriber: plasma-devel.<br />mpyne requested review of this revision.
</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/D14984">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>I recently rebuilt my Qt, KF5, and Plasma 5 environments from scratch. I noticed that System Settings no longer started.</p>

<p>It took me a while to figure out the error message but the proximate issue ended up being:</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">file:///home/kde-svn/kde-5/lib64/qml/org/kde/kirigami.2/templates/private/ScrollView.qml:95:9: 
"Connections.enabled" is not available in QtQuick 2.5. 
             enabled: !Settings.tabletMode 
             ^</pre></div>

<p>I imagine that Qt 5 has stricter checking on the use of QML properties. In this case we ask for QtQuick as of Qt 5.5, but Connections.enabled is only supported in Qt 5.7 on.</p>

<p>I fixed this (and one other such usage) by changing the import statement to ask for QtQuick 5.7. We already require at least Qt 5.7 anyways so this isn't a new dependency for current KF5.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Installed the updated Kirigami, successfully launched System Settings. Additionally the Plasma desktop wallpaper started working again too, though that wasn't why I'd made this fix.</p>

<p>I verified by manual inspection of all uses of the 'Connection' QML item that there were no other uses of the "enabled" property from Qt 5.7.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R169 Kirigami</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D14984">https://phabricator.kde.org/D14984</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/controls/private/RefreshableScrollView.qml<br />
src/controls/templates/private/ScrollView.qml</div></div></div><br /><div><strong>To: </strong>mpyne, Frameworks, Plasma, mart<br /><strong>Cc: </strong>plasma-devel, apol, davidedmundson, mart, hein<br /></div>