Merge lp:~macslow/unity/phablet-notification-renderer into lp:unity/phablet

[Mirco Müller]

First basic NotificationRenderer (Frontend) for NotifyOSD NG. Ephemeral and 2-button snap-decision layout are support for a start. Using mocked up notification-data for the moment.

First basic NotificationRenderer (Frontend) for NotifyOSD NG. Ephemeral and 2-button snap-decision layout are support for a start. Using mocked up notification-data for the moment.

[Michał Sawicz (saviq)]

* the change to .bzrignore seems unrelated, please don't submit it
* I'd rename Bubble to Notification
* I don't see a reason to separate Bubble.qml into a separate dir - unless really difficult, there should only be a single Notification component that lays itself out according to the properties set
* "property bool snapDecision: actions != null" - this should depend on the type of the notification, not the presence of actions
* the whole bubble is 0.85 opaque, is that per-design? shouldn't just the background be transparent and the text fully opaque?
* please separate the components with newlines
* "visible: secondaryIconSource ? true : false" you can use just "source" here, secondaryIconSource is an alias anyway
* "left: parent.left; leftMargin: units.gu(8)" you should anchor to icon.right instead
* "fillMode: Image.PreserveAspectCrop" for secondaryIcon seems unnecessary, the icon should always be 2x2 GUs
* "leftMargin: secondaryIcon.visible ? units.gu(11) : units.gu(8)" anchor to icon or secondaryIcon instead
* "width: secondaryIcon.visible ? units.gu (25) : units.gu (28)" anchor to parent.right instead
* "font.family: "Ubuntu"" is default
* "opacity: 1.0" is default, and does _not_ mean that the text itself will be fully opaque, because the parent item is not opaque [1]
* "maximumLineCount: 1" is unnecessary when wrapping is disabled
* the above applies to bodyLabel, too, except that bodyLabel should anchor to summaryLabel's left and bottom
* you should elide the body, too, in case it overflows
* "width: units.gu (17.5)" is arbitrary, calculate the width based on parent.width
* "text: actions.get(1).action_label" you should almost never need Model.get() - use Views or Positioners with Repeaters [2] [3]
* the Loader is unnecessary, if you use a Repeater, it will not create anything if the model is empty
* use add/move/delete transitions

* in this case I'd rather see the layout to use Positioners [2] extensively for better readability and flexibility, like so:

Column {
    Row {
        UbuntuShape { id: icon }
        Image { id: secondaryIcon}
        Column {
            Label { id: summary }
            Label { id: body }
        }
    }
    Flow {
        Repeater { id: buttons }
    }
}

[1] http://qt-project.org/doc/qt-5.0/qtquick/qml-qtquick2-item.html#opacity-prop
[2] http://qt-project.org/doc/qt-5.0/qtquick/qtquick-qmltypereference.html#positioning
[3] http://qt-project.org/doc/qt-5.0/qtquick/qtquick-qmltypereference.html#model-view-types-and-data-storage-and-access

review: Needs Fixing

[Michał Sawicz (saviq)]

The Columns might be overkill, but you need to wrap the Labels in an Item anyway for the Row to not try and position them next to each other.

[Mirco Müller (macslow)]

Ah, ok. I'll fix all those issues.

[kevin gunn (kgunn72)]

wrt >> I'd rename Bubble to Notification
I worry a little that we are overloading "notification"
e.g. if back end is also called notification it might get confusing
am i being a nut? ultimately i'll defer to Saviq/Mirco

[Michał Sawicz (saviq)]

Sure, that's a valid concern. QML Notification component would end up as a QQmlNotification in C++ world, and anyway namespacing is helping here - it's going to be Unity::Notifications::Notification in C++ world. And even in QML it's recommended to used named imports (import Unity.Notifications 0.1 as Notifications) and then the components exposed from that plugin are available as Notifications.Notification, for example.

What's more, we'll never use (title case) Notification as exposed from the QML Unity.Notifications plugin, at least not in the shell. That object will only be created C++ side.

So, all in all I don't foresee naming conflicts, but to avoid confusion, I could agree to a different name ;). I'd like to avoid "Bubble" nevertheless ;D

[Mirco Müller (macslow)]

All mentioned suggestions were addressed, with two variations:

* I left the explicit sizes for icon and secondary-icon in place as a "safty-net", for those cases when rogue applications pass in too small or too large icons, thus the layout of the notification does not break away from the design-guideline.

* With the way anchors are used in summayLabel and bodyLabel now, there's no danger of them being accidentally placed next to each other.

Good to merge with trunk?

[Michał Sawicz (saviq)]

Yeah, explicit sizes of icons / assets is correct, of course.

[Michał Sawicz (saviq)]

* please order the properties on the component in a more structured way (at least public properties first, and secondaryIcon source just after iconSource)
* components should not have their dimensions predefined, unless they depend (as in this case the height) on their child objects
    width: parent.width
    height: childrenRect.height + units.gu (1)
* don't expose private properties on root objects of components - in this case I don't see the need for that separate property, you can simply use the condition itself
    property bool snapDecision: type == "Notifications.Type.SnapDecision"
* components should not define their overall animations - and you're not changing the opacity of it anyway - you're going to that in the transitions
    Behavior on opacity {NumberAnimation{}}
* I would prefer using a Row for the icons and summary+body, all the margin: units.gu(1) make it hard to read, and it would handle the invisible secondaryIcon automatically
* explicitly check for source !== undefined && source != "", you could also check for status == Image.Ready [1], but that might result in some movement while the image is being loaded
    visible: source
* set sourceSize [2] on Images that come from outside to reduce memory
* this is not correct, use bodyLabel.y + bodyLabel.height - height of items does not include anchor margins - and if you use a Row above, you will just anchor to the row's bottom anyway
  top: (icon.height >= summaryLabel.height + bodyLabel.height) ? icon.bottom : bodyLabel.bottom
* why this??
    parent: notification
* set visibility instead
    topMargin: snapDecision ? units.gu (1) : 0
* there's a binding loop - should go away when you move to using a Row, too
    Notifications.qml:48:19: QML Notification: Binding loop detected for property "height"
* don't use space between function name and parenthesis [units.gu(1), not units.gu (1)]
* use camelCase instead of underscores (action_label)
* no need to prefix action's values with action_
* Notifications should not have its own queue or the displayNotification method, it should be a ListView{}, not a Showable{ListView{}}
* Notifications - see above about predefined dimensions
    anchors.fill: parent
* don't add Notifications to Shell, yet, since there's no model to supply to it
* the tests are not really tests - there's no way they would fail, is there?

[1] http://qt-project.org/doc/qt-5.0/qtquick/qml-qtquick2-image.html#status-prop
[2] http://qt-project.org/doc/qt-5.0/qtquick/qml-qtquick2-image.html#sourceSize-prop

review: Needs Fixing

[Michał Sawicz (saviq)]

=========

48 + color: Qt.rgba(0, 0, 0, 0.85)
with that, shouldn't you be able to just drop the Item?

=========

54 + top: parent.top
unnecessary

=========

56 + height: childrenRect.height + contentColumn.anchors.margins * 2
since spacing / margins are units.gu(1) everywhere, let's just use contentColumn.spacing everywhere?

=========

81 + width: (body == "") ? units.gu(2) : units.gu(6)
82 + height: (body == "") ? units.gu(2) : units.gu(6)
we need to be able to collapse items that _do_ have body text, no? actually, it's the secondaryIcon that should stay on screen in collapsed state, even if there is a primary icon, no? IMO it's a perfect place to use States and Transitions [1]

expanded => collapsed transition as follows:
SequentialAnimation {
  ParallelAnimation {
    icon shrinks
    icon fades out
    labelColumn.height shrinks to summaryLabel.height
    labelColumn fades out
  }
  PropertyAction {
    icon becomes invisible
    bodyLabel becomes invisible
  }
}

collapsed => expanded transition as follows:
SequentialAnimation {
  PropertyAction {
    icon becomes visible
    bodyLabel becomes visible
  }
  ParallelAnimation {
    icon grows
    icon fades in
    labelColumn.height grows to labelColumn.implicitHeight
    labelColumn fades in
  }
}

you also need to bind labelColumn.clip to Transition.running [2] of the two above.

=========

107 + height: childrenRect.height
this shouldn't be needed in Positioners - they get sized by their children implicitly

=========

113 + width: parent.width - x
this should only be like that in labelColumn, the Labels themselves should anchor to parent.left/right

=========

124 + visible: body != ""
as mentioned above, use States and Transitions for that

=========

141 + visible: type == "Notifications.Type.SnapDecision"
145 + model: actions
reference by object - notification.type, notification.actions

=========

74 + width: parent.width
142 + width: childrenRect.width
you should anchor the Rows to parent.left/right instead

=========

because of the quality issues, let's drop sourceSize for now (sorry)

=========

46 + width: parent.width
191 + anchors.fill: parent
these need to go away (component dimensions should generally be handled in their parents, unless they depend on content dimensions):

=========

225 +
rogue newline

=========

237 +<<<<<<< TREE
238 +=======
239 +add_qml_test(DashPreview)
240 +add_qml_test(NotificationRenderer)
241 +>>>>>>> MERGE-SOURCE
conflict (DashPreview shouldn't be touched)

=========

add some controls next to the view so that you can manually drive the test (see other tests in qmluitests for examples)

=========

[1] http://qt-project.org/doc/qt-5.0/qtquick/qtquick-statesanimations-states.html
[2] http://qt-project.org/doc/qt-5.0/qtquick/qml-qtquick2-transition.html#running-prop

review: Needs Fixing

[Mirco Müller (macslow)]

> 81 + width: (body == "") ? units.gu(2) : units.gu(6)
> 82 + height: (body == "") ? units.gu(2) : units.gu(6)
> we need to be able to collapse items that _do_ have body text, no? actually,
> it's the secondaryIcon that should stay on screen in collapsed state, even if
> there is a primary icon, no?

No, this here is meant for the special "icon+summary" layout case Design wants. This is different from a collapsing snap-decision, which is only meant to be used on large screens.

> 141 + visible: type == "Notifications.Type.SnapDecision"
> 145 + model: actions
> reference by object - notification.type, notification.actions

That does not work. How can I make the object known? Do I need to change the way data is passed in the test?

> 46 + width: parent.width
> 191 + anchors.fill: parent
> these need to go away (component dimensions should generally be handled in
> their parents, unless they depend on content dimensions):

I could get rid of "anchors.fill: parent", but I have to keep "width: parent.width". That way the width of the ListView holding the notifications follows any embedding parent-item. Otherwise there's no way to tell the ListView what horizontal space it may occupy.

[Michał Sawicz (saviq)]

W dniu 09.04.2013 17:07, Mirco Müller pisze:
>> 81 + width: (body == "") ? units.gu(2) : units.gu(6)
>> > 82 + height: (body == "") ? units.gu(2) : units.gu(6)
>> > we need to be able to collapse items that _do_ have body text, no? actually,
>> > it's the secondaryIcon that should stay on screen in collapsed state, even if
>> > there is a primary icon, no?
> No, this here is meant for the special "icon+summary" layout case Design wants. This is different from a collapsing snap-decision, which is only meant to be used on large screens.

Right, but since the icon is that small, do we not want to get rid of
the UbuntuShape for it? We could easily switch the source of the icons
instead of the dimensions of them? Is the lack of a body the only
determinant of whether it's a icon+summary layout? Does that mean body
is required for the generic case? It doesn't make sense for the icon to
be an avatar, for example, when it's just 2x2 GUs?

>> > 141 + visible: type == "Notifications.Type.SnapDecision"
>> > 145 + model: actions
>> > reference by object - notification.type, notification.actions
> That does not work. How can I make the object known? Do I need to change the way data is passed in the test?

Hum? The top level UbuntuShape has «id: notification» and type and
actions properties, can you not access those for some reason?

>> > 46 + width: parent.width
>> > 191 + anchors.fill: parent
>> > these need to go away (component dimensions should generally be handled in
>> > their parents, unless they depend on content dimensions):
> I could get rid of "anchors.fill: parent", but I have to keep "width: parent.width". That way the width of the ListView holding the notifications follows any embedding parent-item. Otherwise there's no way to tell the ListView what horizontal space it may occupy.

That's the «width: parent.width» in Notification.qml:29, it should be in
Notifications.qml:29 - it's in the delegate where you should define the
dimensions (again, unless they depend on their content). I think the
rule of thumb is: top-level component in a file should not have its
dimensions defined, unless they depend on the contents.

Case in point: it's Shell.qml that will decide on the dimensions of
Notifications {}, it's Notifications.qml that will decide on the
dimensions of Notification {} - it's delegate. And you should use
anchors there, too.

Cheers,
--
Michał Sawicz <email address hidden>
Canonical Services Ltd.

[Michał Sawicz (saviq)]

Hey, make sure to merge trunk, soon, there's conflicts.

review: Needs Fixing

[Michael Zanetti (mzanetti)]

if possible remove wait()s from the tests. They take ~8 secs now while I think they could run in < 0.5 secs. (Makes a huge difference in Jenkins where all 300 tests we have are summed up)

review: Needs Fixing

[Michael Zanetti (mzanetti)]

I'd like to see tests for the different mouse interactions:

- notifcation is click-through or not (interactive/not interactive?)
- click buttons in notifications

review: Needs Fixing

[Michael Zanetti (mzanetti)]

604 + wait(100)

really required? I know only because view.count == model.count it doesn't mean that the delegates are created already. I think still wait(0) should be enough. (Fine as long it is set to WIP - just wanted to mention it for completemess)

* Regarding the if() around the signal spys: I think it would be better to add the conditions to the test data. Because if you decide on interactiveArea.visible, your test wouldn't catch a bug where interactiveArea is already visible by mistake. Better extending testdata with data like "isInteractive: true; hasButtons: false"

review: Needs Fixing

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:575
No commit message was specified in the merge proposal. Click on the following link and set the commit message (if you want a jenkins rebuild you need to trigger it yourself):
https://code.launchpad.net/~macslow/unity/phablet-notification-renderer/+merge/155512/+edit-commit-message

http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/709/
Executed test runs:
    FAILURE: http://s-jenkins:8080/job/unity-phablet-qmluitests/527/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/711/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-i386-ci/709/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/585/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-i386-ci/589/console

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity-phablet-ci/709/rebuild

review: Needs Fixing (continuous-integration)

[Michael Zanetti (mzanetti)]

debian/qml-phone-shell.install needs to be fixed:

dh_install: usr/share/qml-phone-shell/Notifications/Notification.qml exists in debian/tmp but is not installed to anywhere
dh_install: usr/share/qml-phone-shell/Notifications/Notifications.qml exists in debian/tmp but is not installed to anywhere
dh_install: usr/share/qml-phone-shell/Notifications/timings.js exists in debian/tmp but is not installed to anywhere
dh_install: usr/share/qml-phone-shell/Notifications/graphics/tile.png exists in debian/tmp but is not installed to anywhere
dh_install: usr/share/qml-phone-shell/Notifications/graphics/avatar3.jpg exists in debian/tmp but is not installed to anywhere
dh_install: usr/share/qml-phone-shell/Notifications/graphics/avatar1.jpg exists in debian/tmp but is not installed to anywhere
dh_install: usr/share/qml-phone-shell/Notifications/graphics/icon_phone.png exists in debian/tmp but is not installed to anywhere
dh_install: usr/share/qml-phone-shell/Notifications/graphics/avatar2.jpg exists in debian/tmp but is not installed to anywhere
dh_install: missing files, aborting

review: Needs Fixing

[Michael Zanetti (mzanetti)]

I don't think those images should be installed with the shell. They should rather be moved into tests/mocks or the like

review: Needs Fixing

[Michał Sawicz (saviq)]

Hey, no need to put it as Work In Progress when you go and fix review comments. Leaving it at Needs Review is fine.

[Michael Zanetti (mzanetti)]

224 + easing.type: Timings.easing
288 +var easing = "OutQuint";

Interesting... I just thought easings.type is a enum instead of a string. Looking it up in the docs it seems using strings is possible. Easing.OutQuint the preferred one though (I guess because it gives us code completion in QtCreator while strings don't):

<snip>

QML Basic Type: enumeration
An enumeration type consists of a set of named values.
An enumeration value may be specifed as either a string:

 Text { horizontalAlignment: "AlignRight" }

or as <Element>.<value>:

 Text { horizontalAlignment: Text.AlignRight }

The second form is preferred.

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:578
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/714/
Executed test runs:
    UNSTABLE: http://s-jenkins:8080/job/unity-phablet-qmluitests/533
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/716
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/716/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-i386-ci/714
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/590
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/590/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-i386-ci/594
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner/1248

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity-phablet-ci/714/rebuild

review: Needs Fixing (continuous-integration)

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:578
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/724/
Executed test runs:
    UNSTABLE: http://s-jenkins:8080/job/unity-phablet-qmluitests/549
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/726
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/726/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-i386-ci/724
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/600
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/600/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-i386-ci/604
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner/1266

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity-phablet-ci/724/rebuild

review: Needs Fixing (continuous-integration)

[Michał Sawicz (saviq)]

Tests
========================

627 + wait(0)

Looks unnecessary?

========

643 + interactiveAreaSpy.clear()
644 + clickThroughSpy.clear()

651 + clickThroughSpy.clear()

You should do that in a « function cleanup() » instead - it's called after every test case.

========

646 + interactiveAreaSpy.wait()
647 + verify(interactiveAreaSpy.count == 1)

653 + clickThroughSpy.wait()
654 + verify(clickThroughSpy.count == 1)

SignalSpy.wait() already verifies the count is 1.

========

648 + verify(clickThroughSpy.count == 0)

Should use compare().

========

649 + }
650 + else {

Please keep in one line.

========

638 + compare(icon.visible, data.iconVisible, "check for icon of notification" + (mockModel.count - 1))
639 + compare(interactiveArea.visible, data.interactiveAreaVisible, "check for interactive notification" + (mockModel.count - 1))

657 + compare(secondaryIcon.visible, data.secondaryIconVisible, "check for secondary icon" + (mockModel.count - 1))
658 + compare(summaryLabel.visible, data.summaryVisible, "check for summary-label" + (mockModel.count - 1))
659 + compare(bodyLabel.visible, data.bodyVisible, "check for body-label" + (mockModel.count - 1))
660 + compare(buttonRow.visible, data.buttonRowVisible, "check for buttons" + (mockModel.count - 1))

The messages could be improved, and also "notification1", "notification2" are not really useful - the tag: from _data() shows which test is being run.

========

662 + if(data.buttonRowVisible) {
663 + var buttonCancel = findChild(buttonRow, "button0")
664 + var buttonAccept = findChild(buttonRow, "button1")
665 +
666 + buttonSpy.target = buttonCancel
667 + buttonSpy.clear()
668 + mouseClick(buttonCancel, buttonCancel.width / 2, buttonCancel.height / 2)
669 + buttonSpy.wait()
670 + verify(buttonSpy.count == 1)
671 +
672 + buttonSpy.target = buttonAccept
673 + buttonSpy.clear()
674 + mouseClick(buttonAccept, buttonAccept.width / 2, buttonAccept.height / 2)
675 + buttonSpy.wait()
676 + verify(buttonSpy.count == 1)
677 + }

This only tests that Button.clicked signal is working. You should instead create an actionInvoked(string) signal on Notification and spy on this signal, making sure that SignalSpy.signalArguments are as expected.

Another run through the code itself
===================================

 46 + width: parent.width
 47 + height: childrenRect.height

Again, top-level components in a file should not have width or height defined. Use implicitHeight and get rid of width altogether - it's determined by the delegate definition in Notifications.qml.

========

 57 + visible: type == "Notifications.Type.Interactive"

MouseAreas are not graphical, use "enabled" instead.

========

 58 + onClicked: print("clicked " + actions.get(0).id)

Forward through a...

review: Needs Fixing

[Michał Sawicz (saviq)]

There's still some questions from https://code.launchpad.net/~macslow/unity/phablet-notification-renderer/+merge/155512/comments/346145 unanswered.

=========

 50 + visible: opacity > 0

Is that needed?

=========

 57 + visible: type == "Notifications.Type.Interactive"

Where is type defined?

=========

 68 + margins: units.gu(1)
 69 + }
 70 + height: childrenRect.height + contentColumn.anchors.margins
 71 + spacing: units.gu(1)

 76 + spacing: units.gu(1)

147 + spacing: units.gu(1)

163 + width: (topRow.width - units.gu(1)) / 2

I mentioned earlier a common spacing could be useful, go with the below instead:

 68 + margins: spacing
 69 + }
 70 + height: childrenRect.height + spacing
 71 + spacing: units.gu(1)

 76 + spacing: contentColumn.spacing

147 + spacing: contentColumn.spacing

163 + width: (topRow.width - buttonRow.spacing) / 2

=========

 81 + height: childrenRect.height

That's unnecessary - default in Positioners.

=========

 83 + UbuntuShape {
 84 + id: icon
 85 +
 86 + objectName: "icon"
 87 + width: (body == "") ? units.gu(2) : units.gu(6)
 88 + height: (body == "") ? units.gu(2) : units.gu(6)
 89 + image: Image {
 90 + id: avatarIcon
 91 +
 92 + fillMode: Image.PreserveAspectCrop
 93 + }
 94 + }
 95 +
 96 + Image {
 97 + id: secondaryIcon
 98 +
 99 + objectName: "secondaryIcon"
100 + width: units.gu(2)
101 + height: units.gu(2)
102 + visible: source !== undefined && source != "" && bodyLabel.visible
103 + fillMode: Image.PreserveAspectCrop
104 + }

I'm still not sure we want a 2x2 GU UbuntuShape, see the comment mentioned at the top of this one.

=========

149 + visible: type == "Notifications.Type.SnapDecision"

156 + model: actions

See the comment at the top.

=========

159 + id: button
160 +

Looks unused.

=========

163 + width: (topRow.width - units.gu(1)) / 2

Why topRow? Shouldn't this be buttonRow.width?

=========

166 + onClicked: print("clicked " + id)

Same as before - forward via a actionInvoked(string) signal.

=========

208 + delegate: Notification {
209 + objectName: "notification" + index
210 + iconSource: model.icon
211 + secondaryIconSource: model.secondaryIcon
212 + summary: model.summary
213 + body: model.body
214 + actions: model.actions
215 + }

Use anchors { left: parent.left; right: parent.right } to make the delegates fill Notifications horizontally. Also, "model" here is unnecessary - you can access the roles directly.

=========

309 +add_qml_test(Notifications NotificationRenderer)

326 === added file 'tests/qmltests/Notifications/tst_NotificationRenderer.qml'
327 --- tests/qmltests/Notifications/tst_NotificationRenderer.qml 1970-01-01 ...

review: Needs Fixing

[Michał Sawicz (saviq)]

Ah one more thing, please reduce the set of test assets to a minimum. You can use those from /graphics instead.

Obviously we need to clean those up, but let's not add more instead.

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:579
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/740/
Executed test runs:
    UNSTABLE: http://s-jenkins:8080/job/unity-phablet-qmluitests/568
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/741
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/741/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-i386-ci/739
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/616
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/616/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-i386-ci/620
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner/1284

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity-phablet-ci/740/rebuild

review: Needs Fixing (continuous-integration)

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:582
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/742/
Executed test runs:
    UNSTABLE: http://s-jenkins:8080/job/unity-phablet-qmluitests/572
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/744
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/744/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-i386-ci/742
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/618
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/618/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-i386-ci/622
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner/1286

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity-phablet-ci/742/rebuild

review: Needs Fixing (continuous-integration)

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:584
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/744/
Executed test runs:
    UNSTABLE: http://s-jenkins:8080/job/unity-phablet-qmluitests/575
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/746
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/746/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-i386-ci/744
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/620
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/620/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-i386-ci/624
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner/1289

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity-phablet-ci/744/rebuild

review: Needs Fixing (continuous-integration)

[Mirco Müller (macslow)]

Hm... CI of my branch failed because of a failure with test_hide_hud_click.

[Mirco Müller (macslow)]

qml_phone_shell.tests.testconfigurations.TestNexus10.test_hide_hud_click() to be precise

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:588
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/745/
Executed test runs:
    UNSTABLE: http://s-jenkins:8080/job/unity-phablet-qmluitests/578
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/747
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/747/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-i386-ci/745
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/621
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/621/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-i386-ci/625
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner/1291

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity-phablet-ci/745/rebuild

review: Needs Fixing (continuous-integration)

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:588
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/746/
Executed test runs:
    UNSTABLE: http://s-jenkins:8080/job/unity-phablet-qmluitests/581
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/748
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/748/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-i386-ci/746
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/622
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/622/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-i386-ci/626
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner/1294

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity-phablet-ci/746/rebuild

review: Needs Fixing (continuous-integration)

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:588
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/749/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/751
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/751/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-i386-ci/749
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/625
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/625/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-i386-ci/629
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner/1297

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity-phablet-ci/749/rebuild

review: Needs Fixing (continuous-integration)

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:588
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/750/
Executed test runs:
    UNSTABLE: http://s-jenkins:8080/job/unity-phablet-qmluitests/585
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/752
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/752/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-i386-ci/750
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/626
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/626/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-i386-ci/630
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner/1298

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity-phablet-ci/750/rebuild

review: Needs Fixing (continuous-integration)

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:589
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/763/
Executed test runs:
    SUCCESS: http://s-jenkins:8080/job/unity-phablet-qmluitests/621
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/765
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/765/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-i386-ci/763
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/639
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/639/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-i386-ci/643
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner/1313

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity-phablet-ci/763/rebuild

review: Approve (continuous-integration)

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:589
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/765/
Executed test runs:
    FAILURE: http://s-jenkins:8080/job/unity-phablet-qmluitests/631/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/767
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/767/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-i386-ci/765
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/641/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-i386-ci/645/console

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity-phablet-ci/765/rebuild

review: Needs Fixing (continuous-integration)

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:590
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/766/
Executed test runs:
    FAILURE: http://s-jenkins:8080/job/unity-phablet-qmluitests/632/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/768
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-armhf-ci/768/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-quantal-i386-ci/766
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/642/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-i386-ci/646/console

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity-phablet-ci/766/rebuild

review: Needs Fixing (continuous-integration)

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:590
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/775/
Executed test runs:
    SUCCESS: http://s-jenkins:8080/job/unity-phablet-qmluitests/652
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/651/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-i386-ci/655/console

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity-phablet-ci/775/rebuild

review: Needs Fixing (continuous-integration)

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:592
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/776/
Executed test runs:
    SUCCESS: http://s-jenkins:8080/job/unity-phablet-qmluitests/654
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/652
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/652/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-i386-ci/656
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner/1323

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity-phablet-ci/776/rebuild

review: Approve (continuous-integration)

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:592
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/777/
Executed test runs:
    SUCCESS: http://s-jenkins:8080/job/unity-phablet-qmluitests/655
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/653
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/653/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-i386-ci/657
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner/1324

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity-phablet-ci/777/rebuild

review: Approve (continuous-integration)

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:593
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/853/
Executed test runs:
    UNSTABLE: http://s-jenkins:8080/job/unity-phablet-qmluitests/774
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/729
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/729/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-i386-ci/733

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity-phablet-ci/853/rebuild

review: Needs Fixing (continuous-integration)

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:593
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/861/
Executed test runs:
    UNSTABLE: http://s-jenkins:8080/job/unity-phablet-qmluitests/784
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/737
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/737/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-i386-ci/741

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity-phablet-ci/861/rebuild

review: Needs Fixing (continuous-integration)

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:593
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/862/
Executed test runs:
    UNSTABLE: http://s-jenkins:8080/job/unity-phablet-qmluitests/786
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/738
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/738/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-i386-ci/742

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity-phablet-ci/862/rebuild

review: Needs Fixing (continuous-integration)

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:594
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/899/
Executed test runs:
    UNSTABLE: http://s-jenkins:8080/job/unity-phablet-qmluitests/834
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/775
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/775/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-i386-ci/779

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity-phablet-ci/899/rebuild

review: Needs Fixing (continuous-integration)

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:595
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/907/
Executed test runs:
    SUCCESS: http://s-jenkins:8080/job/unity-phablet-qmluitests/858
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/783/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-i386-ci/787
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner/1439

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity-phablet-ci/907/rebuild

review: Needs Fixing (continuous-integration)

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:595
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/908/
Executed test runs:
    SUCCESS: http://s-jenkins:8080/job/unity-phablet-qmluitests/860
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/784
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/784/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-i386-ci/788
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner/1441

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity-phablet-ci/908/rebuild

review: Needs Fixing (continuous-integration)

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:595
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/910/
Executed test runs:
    UNSTABLE: http://s-jenkins:8080/job/unity-phablet-qmluitests/865
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/786
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/786/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-i386-ci/790
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner/1443

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity-phablet-ci/910/rebuild

review: Needs Fixing (continuous-integration)

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:597
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/912/
Executed test runs:
    SUCCESS: http://s-jenkins:8080/job/unity-phablet-qmluitests/867
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/788
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/788/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-i386-ci/792
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner/1445

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity-phablet-ci/912/rebuild

review: Approve (continuous-integration)

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:598
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/913/
Executed test runs:
    SUCCESS: http://s-jenkins:8080/job/unity-phablet-qmluitests/871
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/789
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/789/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-i386-ci/793
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner/1447

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity-phablet-ci/913/rebuild

review: Needs Fixing (continuous-integration)

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:598
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/915/
Executed test runs:
    SUCCESS: http://s-jenkins:8080/job/unity-phablet-qmluitests/873
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/791
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/791/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-i386-ci/795
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner/1449

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity-phablet-ci/915/rebuild

review: Approve (continuous-integration)

[Michael Zanetti (mzanetti)]

Looks good to me now.

review: Approve

[Michał Sawicz (saviq)]

Hey, some last still-remaining tweaks:

=====

 77 + spacing: units.gu(1)

Should be contentColumn.spacing

=====

 58 + enabled: type == "Notifications.Type.Interactive"

150 + visible: type == "Notifications.Type.SnapDecision"

157 + model: actions

Should be notification.type ==, notification.actions

=====

162 + width: (topRow.width - buttonRow.spacing) / 2

Why topRow? Shouldn't this be buttonRow.width?

=====

That's it.

review: Needs Fixing

[Mirco Müller (macslow)]

Did those requested tweaks.

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:599
http://jenkins.qa.ubuntu.com/job/unity-phablet-ci/928/
Executed test runs:
    SUCCESS: http://s-jenkins:8080/job/unity-phablet-qmluitests/886
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/804
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-ci/804/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-i386-ci/808
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner/1458

Click here to trigger a rebuild:
http://s-jenkins:8080/job/unity-phablet-ci/928/rebuild

review: Approve (continuous-integration)

[Michał Sawicz (saviq)]

Yup!

review: Approve

[PS Jenkins bot (ps-jenkins)]

FAILED: Autolanding.
More details in the following jenkins job:
http://jenkins.qa.ubuntu.com/job/unity-phablet-autolanding/325/
Executed test runs:
    SUCCESS: http://s-jenkins:8080/job/unity-phablet-qmluitests/900
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-autolanding/259
        deb: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-armhf-autolanding/259/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-raring-i386-autolanding/259
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner/1480/console

review: Needs Fixing (continuous-integration)

[PS Jenkins bot (ps-jenkins)]

review: Approve (continuous-integration)

=== modified file 'CMakeLists.txt'
--- CMakeLists.txt	2013-04-26 14:54:51 +0000
+++ CMakeLists.txt	2013-05-15 15:17:27 +0000
@@ -183,6 +183,7 @@
     Panel
     Launcher
     SideStage
+    Notifications
     )
 
 install(DIRECTORY ${QML_DIRS}
 
=== added directory 'Notifications'
=== added file 'Notifications/Notification.qml'
--- Notifications/Notification.qml	1970-01-01 00:00:00 +0000
+++ Notifications/Notification.qml	2013-05-15 15:17:27 +0000
@@ -0,0 +1,153 @@
+/*
+ * Copyright (C) 2013 Canonical, Ltd.
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; version 3.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program.  If not, see <http://www.gnu.org/licenses/>.
+ */
+
+import QtQuick 2.0
+import Ubuntu.Components 0.1
+
+UbuntuShape {
+    id: notification
+
+    property string type
+    property alias iconSource: avatarIcon.source
+    property alias secondaryIconSource: secondaryIcon.source
+    property alias summary: summaryLabel.text
+    property alias body: bodyLabel.text
+    property var actions
+
+    signal actionInvoked(string buttonId)
+
+    implicitHeight: childrenRect.height
+    color: Qt.rgba(0, 0, 0, 0.85)
+    opacity: 0
+
+    MouseArea {
+        id: interactiveArea
+
+        anchors.fill: contentColumn
+        objectName: "interactiveArea"
+        enabled: notification.type == "Notifications.Type.Interactive"
+        onClicked: notification.actionInvoked(actions.get(0).id)
+    }
+
+    Column {
+        id: contentColumn
+
+        anchors {
+            left: parent.left
+            right: parent.right
+            top: parent.top
+            margins: spacing
+        }
+        height: childrenRect.height + spacing
+        spacing: units.gu(1)
+
+        Row {
+            id: topRow
+
+            spacing: contentColumn.spacing
+            anchors {
+                left: parent.left
+                right: parent.right
+            }
+
+            UbuntuShape {
+                id: icon
+
+                objectName: "icon"
+                width: units.gu(6)
+                height: units.gu(6)
+                visible: iconSource !== undefined && iconSource != ""
+                image: Image {
+                    id: avatarIcon
+
+                    fillMode: Image.PreserveAspectCrop
+                }
+            }
+
+            Image {
+                id: secondaryIcon
+
+                objectName: "secondaryIcon"
+                width: units.gu(2)
+                height: units.gu(2)
+                visible: source !== undefined && source != ""
+                fillMode: Image.PreserveAspectCrop
+            }
+
+            Column {
+                id: labelColumn
+                width: parent.width - x
+
+                Label {
+                    id: summaryLabel
+
+                    objectName: "summaryLabel"
+                    anchors {
+                        left: parent.left
+                        right: parent.right
+                    }
+                    fontSize: "medium"
+                    font.bold: true
+                    color: "#f3f3e7"
+                    elide: Text.ElideRight
+                }
+
+                Label {
+                    id: bodyLabel
+
+                    objectName: "bodyLabel"
+                    anchors {
+                        left: parent.left
+                        right: parent.right
+                    }
+                    visible: body != ""
+                    fontSize: "small"
+                    color: "#f3f3e7"
+                    opacity: 0.6
+                    wrapMode: Text.WordWrap
+                    maximumLineCount: 10
+                    elide: Text.ElideRight
+                }
+            }
+        }
+
+        Row {
+            id: buttonRow
+
+            objectName: "buttonRow"
+            spacing: contentColumn.spacing
+            layoutDirection: Qt.RightToLeft
+            visible: notification.type == "Notifications.Type.SnapDecision"
+            anchors {
+                left: parent.left
+                right: parent.right
+            }
+
+            Repeater {
+                model: notification.actions
+
+                Button {
+                    objectName: "button" + index
+                    color: Positioner.isFirstItem ? "#d85317" : "#cdcdcb"
+                    width: (buttonRow.width - buttonRow.spacing) / 2
+                    height: units.gu(4)
+                    text: label
+                    onClicked: notification.actionInvoked(id)
+                }
+            }
+        }
+    }
+}
 
=== added file 'Notifications/Notifications.qml'
--- Notifications/Notifications.qml	1970-01-01 00:00:00 +0000
+++ Notifications/Notifications.qml	2013-05-15 15:17:27 +0000
@@ -0,0 +1,77 @@
+/*
+ * Copyright (C) 2013 Canonical, Ltd.
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; version 3.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program.  If not, see <http://www.gnu.org/licenses/>.
+ */
+
+import QtQuick 2.0
+import Ubuntu.Components 0.1
+import "../Components"
+import "timings.js" as Timings
+
+ListView {
+    id: notificationRenderer
+
+    objectName: "notificationRenderer"
+    interactive: false
+
+    spacing: units.gu(.5)
+    delegate: Notification {
+        objectName: "notification" + index
+        anchors {
+            left: parent.left
+            right: parent.right
+        }
+        type: model.type
+        iconSource: model.icon
+        secondaryIconSource: model.secondaryIcon
+        summary: model.summary
+        body: model.body
+        actions: model.actions
+    }
+
+    populate: Transition {
+        NumberAnimation {
+            property: "opacity"
+            to: 1
+            duration: Timings.snapBeat
+            easing.type: Timings.easing
+        }
+    }
+
+    add: Transition {
+        NumberAnimation {
+            property: "opacity"
+            to: 1
+            duration: Timings.snapBeat
+            easing.type: Timings.easing
+        }
+    }
+
+    remove: Transition {
+        NumberAnimation {
+            property: "opacity"
+            to: 0
+            duration: Timings.fastBeat
+            easing.type: Timings.easing
+        }
+    }
+
+    displaced: Transition {
+        NumberAnimation {
+            properties: "x,y"
+            duration: Timings.fastBeat
+            easing.type: Timings.easing
+        }
+    }
+}
 
=== added file 'Notifications/timings.js'
--- Notifications/timings.js	1970-01-01 00:00:00 +0000
+++ Notifications/timings.js	2013-05-15 15:17:27 +0000
@@ -0,0 +1,23 @@
+/*
+ * Copyright (C) 2013 Canonical, Ltd.
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; version 3.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program.  If not, see <http://www.gnu.org/licenses/>.
+ */
+
+var rhythm     = 700;
+var fastBeat   = 0.5 * rhythm;
+var snapBeat   = 0.5 * fastBeat;
+var slowBeat   = rhythm;
+var sleepyBeat = slowBeat * 2;
+
+var easing     = Easing.OutQuint;
 
=== modified file 'debian/qml-phone-shell.install'
--- debian/qml-phone-shell.install	2013-04-23 15:58:17 +0000
+++ debian/qml-phone-shell.install	2013-05-15 15:17:27 +0000
@@ -7,6 +7,7 @@
 /usr/share/qml-phone-shell/Greeter/*
 /usr/share/qml-phone-shell/Hud/*
 /usr/share/qml-phone-shell/Launcher/*
+/usr/share/qml-phone-shell/Notifications/*
 /usr/share/qml-phone-shell/Panel/*
 /usr/share/qml-phone-shell/Shell.qml
 /usr/share/qml-phone-shell/SideStage/*
 
=== modified file 'tests/qmltests/CMakeLists.txt'
--- tests/qmltests/CMakeLists.txt	2013-05-14 13:42:33 +0000
+++ tests/qmltests/CMakeLists.txt	2013-05-15 15:17:27 +0000
@@ -48,6 +48,7 @@
 add_qml_test(Hud Hud)
 add_qml_test(Hud Result)
 add_qml_test(Launcher Launcher)
+add_qml_test(Notifications Notifications)
 add_qml_test(Panel IndicatorRow)
 add_qml_test(Panel Indicators)
 add_qml_test(Panel MenuContent)
 
=== added directory 'tests/qmltests/Notifications'
=== added file 'tests/qmltests/Notifications/tst_Notifications.qml'
--- tests/qmltests/Notifications/tst_Notifications.qml	1970-01-01 00:00:00 +0000
+++ tests/qmltests/Notifications/tst_Notifications.qml	2013-05-15 15:17:27 +0000
@@ -0,0 +1,335 @@
+/*
+ * Copyright (C) 2013 Canonical, Ltd.
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; version 3.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program.  If not, see <http://www.gnu.org/licenses/>.
+ */
+
+import QtQuick 2.0
+import QtTest 1.0
+import ".."
+import "../../../Notifications"
+import Ubuntu.Components 0.1
+import Unity.Test 0.1
+
+Row {
+    id: rootRow
+
+    ListModel {
+        id: mockModel
+    }
+
+    function addSnapDecisionNotification() {
+        var n = {
+            type: "Notifications.Type.SnapDecision",
+            summary: "Tom Ato",
+            body: "Lorem ipsum dolor sit amet, consetetur sadipscing elitr, sed diam nonumy eirmod tempor invidunt ut labore et dolore magna aliquyam erat, sed diam voluptua.",
+            icon: "../graphics/avatars/funky.png",
+            secondaryIcon: "../graphics/applicationIcons/facebook.png",
+            actions: [{ id: "ok_id", label: "Ok"},
+                      { id: "cancel_id", label: "Cancel"}]
+        }
+
+        mockModel.append(n)
+    }
+
+    function addEphemeralNotification() {
+        var n = {
+            type: "Notifications.Type.Ephemeral",
+            summary: "Cole Raby",
+            body: "I did not expect it to be that late.",
+            icon: "../graphics/avatars/amanda.png",
+            secondaryIcon: "../graphics/applicationIcons/facebook.png",
+            actions: []
+        }
+
+        mockModel.append(n)
+    }
+
+    function addEphemeralIconSummaryNotification() {
+        var n = {
+            type: "Notifications.Type.Ephemeral",
+            summary: "Photo upload completed",
+            body: "",
+            icon: "",
+            secondaryIcon: "../graphics/applicationIcons/facebook.png",
+            actions: []
+        }
+
+        mockModel.append(n)
+    }
+
+    function addInteractiveNotification() {
+        var n = {
+            type: "Notifications.Type.Interactive",
+            summary: "Interactive notification",
+            body: "This is a notification that can be clicked",
+            icon: "../graphics/avatars/anna_olsson.png",
+            secondaryIcon: "",
+            actions: [{ id: "reply_id", label: "Dummy"}],
+        }
+
+        mockModel.append(n)
+    }
+
+    function clearNotifications() {
+        mockModel.clear()
+    }
+
+    function remove1stNotification() {
+        if (mockModel.count > 0)
+            mockModel.remove(0)
+    }
+
+    Rectangle {
+        id: notificationsRect
+
+        width: units.gu(40)
+        height: units.gu(71)
+
+        MouseArea{
+            id: clickThroughCatcher
+
+            anchors.fill: parent
+        }
+
+        Notifications {
+            id: notifications
+
+            anchors.fill: parent
+            anchors.margins: units.gu(1)
+            model: mockModel
+        }
+    }
+
+    Rectangle {
+        id: interactiveControls
+
+        width: units.gu(30)
+        height: units.gu(71)
+        color: "grey"
+
+        Column {
+            spacing: units.gu(1)
+            anchors.fill: parent
+            anchors.margins: units.gu(1)
+
+            Button {
+                width: parent.width
+                text: "add a snap-decision"
+                onClicked: addSnapDecisionNotification()
+            }
+
+            Button {
+                width: parent.width
+                text: "add an ephemeral"
+                onClicked: addEphemeralNotification()
+            }
+
+            Button {
+                width: parent.width
+                text: "add an icon-summary"
+                onClicked: addEphemeralIconSummaryNotification()
+            }
+
+            Button {
+                width: parent.width
+                text: "add an interactive"
+                onClicked: addInteractiveNotification()
+            }
+
+            Button {
+                width: parent.width
+                text: "remove 1st notification"
+                onClicked: remove1stNotification()
+            }
+
+            Button {
+                width: parent.width
+                text: "clear model"
+                onClicked: clearNotifications()
+            }
+        }
+    }
+
+    UnityTestCase {
+        id: root
+        name: "NotificationRendererTest"
+        when: windowShown
+
+        function test_NotificationRenderer_data() {
+            return [
+            {
+                tag: "Snap Decision with secondary icon",
+                type: "Notifications.Type.SnapDecision",
+                summary: "Tom Ato",
+                body: "Lorem ipsum dolor sit amet, consetetur sadipscing elitr, sed diam nonumy eirmod tempor invidunt ut labore et dolore magna aliquyam erat, sed diam voluptua.",
+                icon: "../graphics/avatars/funky.png",
+                secondaryIcon: "../graphics/applicationIcons/facebook.png",
+                actions: [{ id: "ok_id", label: "Ok"},
+                          { id: "cancel_id", label: "Cancel"}],
+                summaryVisible: true,
+                bodyVisible: true,
+                interactiveAreaEnabled: false,
+                iconVisible: true,
+                secondaryIconVisible: true,
+                buttonRowVisible: true
+            },
+            {
+                tag: "Ephemeral notification - icon-summary layout",
+                type: "Notifications.Type.Ephemeral",
+                summary: "Photo upload completed",
+                body: "",
+                icon: "",
+                secondaryIcon: "../graphics/applicationIcons/facebook.png",
+                actions: [],
+                summaryVisible: true,
+                bodyVisible: false,
+                interactiveAreaEnabled: false,
+                iconVisible: false,
+                secondaryIconVisible: true,
+                buttonRowVisible: false
+            },
+            {
+                tag: "Ephemeral notification - check suppression of secondary icon for icon-summary layout",
+                type: "Notifications.Type.Ephemeral",
+                summary: "New comment successfully published",
+                body: "",
+                icon: "",
+                secondaryIcon: "../graphics/applicationIcons/facebook.png",
+                actions: [],
+                summaryVisible: true,
+                bodyVisible: false,
+                interactiveAreaEnabled: false,
+                iconVisible: false,
+                secondaryIconVisible: true,
+                buttonRowVisible: false
+            },
+            {
+                tag: "Interactive notification",
+                type: "Notifications.Type.Interactive",
+                summary: "Interactive notification",
+                body: "This is a notification that can be clicked",
+                icon: "../graphics/avatars/amanda.png",
+                secondaryIcon: "",
+                actions: [{ id: "reply_id", label: "Dummy"}],
+                summaryVisible: true,
+                bodyVisible: true,
+                interactiveAreaEnabled: true,
+                iconVisible: true,
+                secondaryIconVisible: false,
+                buttonRowVisible: false
+            },
+            {
+                tag: "Snap Decision without secondary icon",
+                type: "Notifications.Type.SnapDecision",
+                summary: "Bro Coly",
+                body: "At vero eos et accusam et justo duo dolores et ea rebum. Stet clita kasd gubergren, no sea takimata sanctus est Lorem ipsum dolor sit amet.",
+                icon: "../graphics/avatars/anna_olsson.png",
+                secondaryIcon: "",
+                actions: [{ id: "accept_id", label: "Accept"},
+                          { id: "reject_id", label: "Reject"}],
+                summaryVisible: true,
+                bodyVisible: true,
+                interactiveAreaEnabled: false,
+                iconVisible: true,
+                secondaryIconVisible: false,
+                buttonRowVisible: true
+            },
+            {
+                tag: "Ephemeral notification",
+                type: "Notifications.Type.Ephemeral",
+                summary: "Cole Raby",
+                body: "I did not expect it to be that late.",
+                icon: "../graphics/avatars/funky.png",
+                secondaryIcon: "../graphics/applicationIcons/facebook.png",
+                actions: [],
+                summaryVisible: true,
+                bodyVisible: true,
+                interactiveAreaEnabled: false,
+                iconVisible: true,
+                secondaryIconVisible: true,
+                buttonRowVisible: false
+            }
+            ]
+        }
+
+        SignalSpy {
+            id: clickThroughSpy
+
+            target: clickThroughCatcher
+            signalName: "clicked"
+        }
+
+        SignalSpy {
+            id: actionSpy
+            signalName: "actionInvoked"
+        }
+
+        function cleanup() {
+            clickThroughSpy.clear()
+            actionSpy.clear()
+        }
+
+        function test_NotificationRenderer(data) {
+            // populate model with some mock notifications
+            mockModel.append(data)
+
+            // make sure the model is properly updated before going on
+            tryCompare(notifications, "count", mockModel.count)
+
+            var notification = findChild(notifications, "notification" + (mockModel.count - 1))
+            var icon = findChild(notification, "icon")
+            var interactiveArea = findChild(notification, "interactiveArea")
+            var secondaryIcon = findChild(notification, "secondaryIcon")
+            var summaryLabel = findChild(notification, "summaryLabel")
+            var bodyLabel = findChild(notification, "bodyLabel")
+            var buttonRow = findChild(notification, "buttonRow")
+            waitForRendering(buttonRow)
+
+            compare(icon.visible, data.iconVisible, "avatar-icon visibility is incorrect")
+            compare(interactiveArea.enabled, data.interactiveAreaEnabled, "check for interactive area")
+
+            if(data.interactiveAreaEnabled) {
+                actionSpy.target = notification
+                mouseClick(notification, notification.width / 2, notification.height / 2)
+                actionSpy.wait()
+                compare(actionSpy.signalArguments[0][0], data.actions[0]["id"], "got wrong id for interactive action")
+                compare(clickThroughSpy.count, 0, "click on interactive notification fell through")
+            } else {
+                mouseClick(notification, notification.width / 2, notification.height / 2)
+                clickThroughSpy.wait()
+            }
+
+            compare(secondaryIcon.visible, data.secondaryIconVisible, "secondary-icon visibility is incorrect")
+            compare(summaryLabel.visible, data.summaryVisible, "summary-text visibility is incorrect")
+            compare(bodyLabel.visible, data.bodyVisible, "body-text visibility is incorrect")
+            compare(buttonRow.visible, data.buttonRowVisible, "button visibility is incorrect")
+
+            if(data.buttonRowVisible) {
+                var buttonCancel = findChild(buttonRow, "button1")
+                var buttonAccept = findChild(buttonRow, "button0")
+
+                waitForRendering(notification)
+                actionSpy.target = notification
+                mouseClick(buttonCancel, buttonCancel.width / 2, buttonCancel.height / 2)
+                actionSpy.wait()
+                compare(actionSpy.signalArguments[0][0], data.actions[1]["id"], "got wrong id for negative action")
+                actionSpy.clear()
+
+                mouseClick(buttonAccept, buttonAccept.width / 2, buttonAccept.height / 2)
+                actionSpy.wait()
+                compare(actionSpy.signalArguments[0][0], data.actions[0]["id"], "got wrong id positive action")
+            }
+        }
+    }
+}