D14984: Explicitly request Qt 5.7's QtQuick to make use of Connections.enabled

2018-08-22 Thread Michael Pyne
This revision was automatically updated to reflect the committed changes.
Closed by commit R169:b23e642e5ac4: Explicitly request Qt 5.7's QtQuick to 
make use of Connections.enabled (authored by mpyne).

REPOSITORY
  R169 Kirigami

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D14984?vs=40179&id=40265

REVISION DETAIL
  https://phabricator.kde.org/D14984

AFFECTED FILES
  src/controls/private/RefreshableScrollView.qml
  src/controls/templates/private/ScrollView.qml

To: mpyne, #frameworks, #plasma, mart, davidedmundson
Cc: plasma-devel, apol, davidedmundson, mart, hein


D14984: Explicitly request Qt 5.7's QtQuick to make use of Connections.enabled

2018-08-21 Thread Michael Pyne
mpyne created this revision.
mpyne added reviewers: Frameworks, Plasma, mart.
Herald added a project: Kirigami.
Herald added a subscriber: plasma-devel.
mpyne requested review of this revision.

REVISION SUMMARY
  I recently rebuilt my Qt, KF5, and Plasma 5 environments from scratch. I 
noticed that System Settings no longer started.
  
  It took me a while to figure out the error message but the proximate issue 
ended up being:
  

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 
 ^
  
  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.
  
  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.

TEST PLAN
  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.
  
  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.

REPOSITORY
  R169 Kirigami

REVISION DETAIL
  https://phabricator.kde.org/D14984

AFFECTED FILES
  src/controls/private/RefreshableScrollView.qml
  src/controls/templates/private/ScrollView.qml

To: mpyne, #frameworks, #plasma, mart
Cc: plasma-devel, apol, davidedmundson, mart, hein