Merge lp://staging/~saviq/unity8/fix-rotation-lp1614070 into lp://staging/unity8
Proposed by
Michał Sawicz
Status: | Rejected | ||||
---|---|---|---|---|---|
Rejected by: | Michał Sawicz | ||||
Proposed branch: | lp://staging/~saviq/unity8/fix-rotation-lp1614070 | ||||
Merge into: | lp://staging/unity8 | ||||
Diff against target: |
455 lines (+147/-44) 10 files modified
qml/Notifications/Notification.qml (+28/-9) qml/Notifications/NotificationMenuItemFactory.qml (+3/-0) qml/Notifications/Notifications.qml (+25/-10) qml/Shell.qml (+1/-1) tests/mocks/Unity/Notifications/MockNotificationModel.cpp (+6/-4) tests/mocks/Unity/Notifications/MockNotificationModel.h (+4/-2) tests/qmltests/Notifications/tst_Notifications.qml (+23/-6) tests/qmltests/Notifications/tst_OptionToggle.qml (+16/-1) tests/qmltests/Notifications/tst_SwipeToAct.qml (+18/-4) tests/qmltests/Notifications/tst_VisualSnapDecisionsQueue.qml (+23/-7) |
||||
To merge this branch: | bzr merge lp://staging/~saviq/unity8/fix-rotation-lp1614070 | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Lukáš Tinkl (community) | Disapprove | ||
Albert Astals Cid (community) | Approve | ||
Unity8 CI Bot | continuous-integration | Needs Fixing | |
Review via email:
|
Commit message
Revert r2586, topmostIsFullscreen was becoming stale
Description of the change
* Are there any related MPs required for this MP to build/function as expected? Please list.
N
* Did you perform an exploratory manual test run of your code change and any related functionality?
Y
* If you changed the packaging (debian), did you subscribe the ubuntu-unity team to this MP?
N/A
* If you changed the UI, has there been a design review?
N/A
To post a comment you must log in.
Unmerged revisions
- 2597. By Michał Sawicz
-
Revert r2586, topmostIsFullscreen was becoming stale
* Did you perform an exploratory manual test run of the code change and any related functionality?
Yes
* Did CI run pass? If not, please explain why.
Waiting before top approval