Code review comment for lp:~unity-team/unity8/dash-as-app

+++ qml/Stages/PhoneStage.qml 2014-07-27 01:55:40 +0000
@@ -116,6 +117,8 @@ Item {
         contentWidth: spreadRow.width - shift
         contentX: -shift

+ readonly property bool isActive: shiftedContentX > 0 || spreadDragArea.dragging
+
         // The flickable needs to fill the screen in order to get touch events all over.
         // However, we don't want to the user to be able to scroll back all the way. For
         // that, the beginning of the gesture starts with a negative value for contentX

In QML (unlike in C++) boolean properties are named simply "foo", not "isFoo". I would choose a less overloaded/generic term than "active" if possible. What's the difference between being "active" and being "interactive". I would add a comment explaining what it means to be "active" there.