Merge lp://staging/~mterry/unity8/greeter-arrangement into lp://staging/unity8
Status: | Needs review | ||||
---|---|---|---|---|---|
Proposed branch: | lp://staging/~mterry/unity8/greeter-arrangement | ||||
Merge into: | lp://staging/unity8 | ||||
Diff against target: |
1470 lines (+515/-396) 16 files modified
qml/Greeter/CoverPage.qml (+9/-4) qml/Greeter/Greeter.qml (+22/-4) qml/Greeter/LoginAreaContainer.qml (+4/-0) qml/Greeter/LoginList.qml (+7/-1) qml/Greeter/LoginPage.qml (+211/-0) qml/Greeter/NarrowView.qml (+35/-93) qml/Greeter/WideView.qml (+83/-164) qml/Launcher/Launcher.qml (+2/-2) qml/Rotation/RotationStates.qml (+1/-2) qml/Shell.qml (+13/-12) tests/qmltests/Greeter/TestView.qml (+3/-1) tests/qmltests/Greeter/tst_Greeter.qml (+8/-0) tests/qmltests/Greeter/tst_NarrowView.qml (+1/-3) tests/qmltests/Greeter/tst_WideView.qml (+31/-58) tests/qmltests/tst_OrientedShell.qml (+28/-22) tests/qmltests/tst_Shell.qml (+57/-30) |
||||
To merge this branch: | bzr merge lp://staging/~mterry/unity8/greeter-arrangement | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Unity8 CI Bot | continuous-integration | Needs Fixing | |
Albert Astals Cid (community) | Abstain | ||
Josh Arenson | Needs Fixing | ||
Review via email: mp+313404@code.staging.launchpad.net |
Commit message
Allow tablet greeter to rotate.
Description of the change
I simplified and consolidated code in NarrowView and WideView. They share a lot more, and should be more convergent as a result.
All this is per the following spec:
https:/
The futzing with mainApp vs focusedApplicat
* Are there any related MPs required for this MP to build/function as expected? Please list.
Just pre-reqs.
* Did you perform an exploratory manual test run of your code change and any related functionality?
Sort of? I tested the basics on a phone and a phone in fake-tablet mode, but haven't actually tested on a tablet yet as I don't have a good tablet that rotates.
* If you changed the packaging (debian), did you subscribe the ubuntu-unity team to this MP?
NA
* If you changed the UI, has there been a design review?
NA, just implementing spec.
Unmerged revisions
- 2591. By Michael Terry
-
Fix whitespace
- 2590. By Michael Terry
-
Fix some shell tests
- 2589. By Michael Terry
-
Merge from trunk
- 2588. By Michael Terry
-
Testing merge
- 2587. By Michael Terry
-
Merge from trunk
- 2586. By Michael Terry
-
Merge from trunk; this broke some tests, needs further investigation and testing
- 2585. By Michael Terry
-
Merge trunk
- 2584. By Michael Terry
-
Merge trunk
- 2583. By Michael Terry
-
Map supported orientations before using them
- 2582. By Michael Terry
-
Try to fix test, though I can't reproduce; and avoid binding loop
FAILED: Continuous integration, rev:2581 /unity8- jenkins. ubuntu. com/job/ lp-unity8- ci/2721/ /unity8- jenkins. ubuntu. com/job/ build/3575/ console /unity8- jenkins. ubuntu. com/job/ build-0- fetch/3603 /unity8- jenkins. ubuntu. com/job/ build-2- binpkg/ arch=amd64, release= xenial+ overlay/ 3449/console /unity8- jenkins. ubuntu. com/job/ build-2- binpkg/ arch=amd64, release= zesty/3449/ console /unity8- jenkins. ubuntu. com/job/ build-2- binpkg/ arch=armhf, release= xenial+ overlay/ 3449/console /unity8- jenkins. ubuntu. com/job/ build-2- binpkg/ arch=armhf, release= zesty/3449/ console /unity8- jenkins. ubuntu. com/job/ build-2- binpkg/ arch=i386, release= xenial+ overlay/ 3449/console /unity8- jenkins. ubuntu. com/job/ build-2- binpkg/ arch=i386, release= zesty/3449/ console
https:/
Executed test runs:
FAILURE: https:/
SUCCESS: https:/
FAILURE: https:/
FAILURE: https:/
FAILURE: https:/
FAILURE: https:/
FAILURE: https:/
FAILURE: https:/
Click here to trigger a rebuild: /unity8- jenkins. ubuntu. com/job/ lp-unity8- ci/2721/ rebuild
https:/