Merge lp://staging/~lukas-kde/unity8/fullscreenNotifications-lp1581498 into lp://staging/unity8
Proposed by
Lukáš Tinkl
Status: | Merged | ||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Approved by: | Michael Zanetti | ||||||||||||
Approved revision: | 2401 | ||||||||||||
Merged at revision: | 2432 | ||||||||||||
Proposed branch: | lp://staging/~lukas-kde/unity8/fullscreenNotifications-lp1581498 | ||||||||||||
Merge into: | lp://staging/unity8 | ||||||||||||
Prerequisite: | lp://staging/~lukas-kde/unity8/notificationExpansionLogicFixes | ||||||||||||
Diff against target: |
153 lines (+37/-14) 3 files modified
qml/Notifications/Notification.qml (+17/-6) qml/Notifications/NotificationMenuItemFactory.qml (+3/-0) qml/Notifications/Notifications.qml (+17/-8) |
||||||||||||
To merge this branch: | bzr merge lp://staging/~lukas-kde/unity8/fullscreenNotifications-lp1581498 | ||||||||||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Michael Zanetti (community) | Approve | ||
Unity8 CI Bot | continuous-integration | Needs Fixing | |
Albert Astals Cid (community) | Abstain | ||
Review via email: mp+294799@code.staging.launchpad.net |
Commit message
Fullscreen notification bug fixes
Description of the change
Fullscreen notification bug fixes
- don't let them be pushed down by other snap decisions (e.g. wifi dialog, USB debugging prompt); set their priority higher
- don't allow swiping them away, they must be acted upon
- perform the "reject" action unconditionally (fixes a discrepancy between the mock and the real model)
Fixes lp:1581498
To post a comment you must log in.
FAILED: Continuous integration, rev:2399 /unity8- jenkins. ubuntu. com/job/ lp-unity8- ci/1222/ /unity8- jenkins. ubuntu. com/job/ test-0- autopkgtest/ label=amd64, release= vivid+overlay, testname= qmluitests. sh/759 /unity8- jenkins. ubuntu. com/job/ test-0- autopkgtest/ label=amd64, release= xenial+ overlay, testname= qmluitests. sh/759 /unity8- jenkins. ubuntu. com/job/ build-0- fetch/1639 /unity8- jenkins. ubuntu. com/job/ build-1- sourcepkg/ release= vivid+overlay/ 1592 /unity8- jenkins. ubuntu. com/job/ build-1- sourcepkg/ release= xenial+ overlay/ 1592 /unity8- jenkins. ubuntu. com/job/ build-2- binpkg/ arch=amd64, release= vivid+overlay/ 1585 /unity8- jenkins. ubuntu. com/job/ build-2- binpkg/ arch=amd64, release= vivid+overlay/ 1585/artifact/ output/ *zip*/output. zip /unity8- jenkins. ubuntu. com/job/ build-2- binpkg/ arch=amd64, release= xenial+ overlay/ 1585 /unity8- jenkins. ubuntu. com/job/ build-2- binpkg/ arch=amd64, release= xenial+ overlay/ 1585/artifact/ output/ *zip*/output. zip /unity8- jenkins. ubuntu. com/job/ build-2- binpkg/ arch=armhf, release= vivid+overlay/ 1585 /unity8- jenkins. ubuntu. com/job/ build-2- binpkg/ arch=armhf, release= vivid+overlay/ 1585/artifact/ output/ *zip*/output. zip /unity8- jenkins. ubuntu. com/job/ build-2- binpkg/ arch=armhf, release= xenial+ overlay/ 1585 /unity8- jenkins. ubuntu. com/job/ build-2- binpkg/ arch=armhf, release= xenial+ overlay/ 1585/artifact/ output/ *zip*/output. zip /unity8- jenkins. ubuntu. com/job/ build-2- binpkg/ arch=i386, release= vivid+overlay/ 1585 /unity8- jenkins. ubuntu. com/job/ build-2- binpkg/ arch=i386, release= vivid+overlay/ 1585/artifact/ output/ *zip*/output. zip /unity8- jenkins. ubuntu. com/job/ build-2- binpkg/ arch=i386, release= xenial+ overlay/ 1585 /unity8- jenkins. ubuntu. com/job/ build-2- binpkg/ arch=i386, release= xenial+ overlay/ 1585/artifact/ output/ *zip*/output. zip
https:/
Executed test runs:
SUCCESS: https:/
UNSTABLE: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
Click here to trigger a rebuild: /unity8- jenkins. ubuntu. com/job/ lp-unity8- ci/1222/ rebuild
https:/