Merge lp:~unity-team/unity8/shellRotation into lp:unity8
| Status: | Merged |
|---|---|
| Approved by: | Michael Zanetti on 2015-06-02 |
| Approved revision: | 1681 |
| Merged at revision: | 1804 |
| Proposed branch: | lp:~unity-team/unity8/shellRotation |
| Merge into: | lp:unity8 |
| Prerequisite: | lp:~dandrader/unity8/unityCommandLineParser |
| Diff against target: |
6915 lines (+4326/-608) 78 files modified
CMakeLists.txt (+3/-0) data/unity8-dash.desktop.in (+1/-0) debian/changelog (+6/-0) debian/control (+5/-4) debian/unity8.install (+3/-0) plugins/Greeter/Unity/Launcher/CMakeLists.txt (+1/-1) plugins/Unity/Launcher/CMakeLists.txt (+1/-1) plugins/Utils/CMakeLists.txt (+1/-0) plugins/Utils/plugin.cpp (+4/-1) plugins/Utils/windowscreenshotprovider.cpp (+59/-0) plugins/Utils/windowscreenshotprovider.h (+31/-0) qml/CMakeLists.txt (+1/-0) qml/Components/WindowScreenshot.qml (+35/-0) qml/Dash/DashApplication.qml (+3/-3) qml/Dash/graphics/phone/screenshots/gmail-webapp.svg (+343/-0) qml/Dash/graphics/phone/screenshots/ubuntu-weather-app.svg (+201/-0) qml/DeviceConfiguration.qml (+81/-0) qml/Greeter/Greeter.qml (+2/-0) qml/Greeter/NarrowView.qml (+1/-0) qml/Greeter/WideView.qml (+1/-0) qml/OrientedShell.qml (+185/-0) qml/Panel/Panel.qml (+9/-0) qml/Rotation/HalfLoopRotationAnimation.qml (+46/-0) qml/Rotation/ImmediateRotationAction.qml (+45/-0) qml/Rotation/NinetyRotationAnimation.qml (+90/-0) qml/Rotation/RotationStates.qml (+278/-0) qml/Shell.qml (+112/-49) qml/Stages/ApplicationWindow.qml (+53/-18) qml/Stages/DesktopStage.qml (+23/-0) qml/Stages/OrientationChangeAnimation.qml (+241/-0) qml/Stages/PhoneStage.qml (+137/-38) qml/Stages/SessionContainer.qml (+2/-8) qml/Stages/SpreadDelegate.qml (+265/-24) qml/Stages/SurfaceContainer.qml (+2/-2) qml/Stages/TabletStage.qml (+149/-19) qml/Stages/TransformedSpreadDelegate.qml (+50/-6) qml/Stages/TransformedTabletSpreadDelegate.qml (+5/-2) run.sh (+1/-1) src/ApplicationArguments.cpp (+22/-0) src/ApplicationArguments.h (+7/-11) src/CMakeLists.txt (+6/-7) src/Dash/CMakeLists.txt (+7/-1) src/Dash/main.cpp (+12/-6) src/UnityCommandLineParser.cpp (+5/-0) src/UnityCommandLineParser.h (+2/-0) src/main.cpp (+18/-8) tests/mocks/Unity/Application/ApplicationInfo.cpp (+45/-4) tests/mocks/Unity/Application/ApplicationInfo.h (+9/-1) tests/mocks/Unity/Application/ApplicationManager.cpp (+39/-2) tests/mocks/Unity/Application/CMakeLists.txt (+1/-2) tests/mocks/Unity/Application/MirSurfaceItem.cpp (+12/-29) tests/mocks/Unity/Application/MirSurfaceItem.h (+29/-29) tests/mocks/Unity/Application/MirSurfaceItem.qml (+12/-14) tests/mocks/Unity/Application/SurfaceManager.cpp (+4/-34) tests/mocks/Unity/Application/SurfaceManager.h (+2/-5) tests/mocks/Unity/Application/UbuntuKeyboardInfo.cpp (+33/-1) tests/mocks/Unity/Application/UbuntuKeyboardInfo.h (+11/-5) tests/mocks/Unity/Application/VirtualKeyboard.cpp (+44/-0) tests/mocks/Unity/Application/VirtualKeyboard.h (+42/-0) tests/mocks/Unity/Application/VirtualKeyboard.qml (+7/-1) tests/mocks/Unity/Launcher/MockLauncherModel.cpp (+3/-1) tests/mocks/Utils/CMakeLists.txt (+1/-0) tests/mocks/Utils/plugin.cpp (+3/-0) tests/plugins/Greeter/Unity/Launcher/CMakeLists.txt (+1/-1) tests/plugins/Unity/Launcher/CMakeLists.txt (+0/-1) tests/plugins/Unity/Launcher/launchermodeltest.cpp (+3/-1) tests/qmltests/CMakeLists.txt (+1/-0) tests/qmltests/Panel/tst_IndicatorsMenu.qml (+0/-2) tests/qmltests/Panel/tst_Panel.qml (+0/-2) tests/qmltests/Stages/tst_ApplicationWindow.qml (+14/-23) tests/qmltests/Stages/tst_PhoneStage.qml (+5/-58) tests/qmltests/Stages/tst_SessionContainer.qml (+1/-64) tests/qmltests/Stages/tst_SpreadDelegate.qml (+173/-38) tests/qmltests/Stages/tst_TabletStage.qml (+51/-0) tests/qmltests/tst_OrientedShell.qml (+1151/-0) tests/qmltests/tst_Shell.qml (+15/-69) tests/qmltests/tst_ShellWithPin.qml (+6/-10) tests/utils/modules/Unity/Test/UnityTestCase.qml (+53/-1) |
| To merge this branch: | bzr merge lp:~unity-team/unity8/shellRotation |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Michael Zanetti (community) | 2015-04-16 | Approve on 2015-06-02 | |
| PS Jenkins bot | continuous-integration | 2015-04-16 | Needs Fixing on 2015-05-21 |
| Nick Dedekind (community) | Abstain on 2015-05-12 | ||
|
Review via email:
|
|||
This proposal supersedes a proposal from 2015-02-03.
Commit Message
Shell rotation
New dependency: libandroid-
Updated dependencies: libunity-api-dev (>= 7.97), unity-applicati
Description of the Change
* Are there any related MPs required for this MP to build/function as expected? Please list.
It's all in this PPA: https:/
https:/
https:/
https:/
https:/
* Did you perform an exploratory manual test run of your code change and any related functionality?
Yes
* Did you make sure that your branch does not contain spurious tags?
Yes
* If you changed the packaging (debian), did you subscribe the ubuntu-unity team to this MP?
Not applicable
* If you changed the UI, has there been a design review?
Not sure, I guess they seen it by now. At least it's following the design spec for that.
| Albert Astals Cid (aacid) wrote : | # |
| Michael Zanetti (mzanetti) wrote : | # |
1202 + d.transitioning = true;
I don't really like the fact that all the rotation animations reach out of context and operate on another object's "d" object. Can we make that more readable?
| Michael Zanetti (mzanetti) wrote : | # |
1627 + * Copyright (C) 2013-2014 Canonical, Ltd.
You probably want to do another round of updating those headers to 2015
| Daniel d'Andrada (dandrader) wrote : | # |
On 05/03/15 08:22, Michael Zanetti wrote:
> Review: Needs Fixing
>
> 1202 + d.transitioning = true;
>
> I don't really like the fact that all the rotation animations reach out of context and operate on another object's "d" object. Can we make that more readable?
Done.
| Daniel d'Andrada (dandrader) wrote : | # |
On 05/03/15 08:45, Michael Zanetti wrote:
> Review: Needs Fixing
>
> 1627 + * Copyright (C) 2013-2014 Canonical, Ltd.
>
> You probably want to do another round of updating those headers to 2015
Done.
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1655
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
| Michael Zanetti (mzanetti) wrote : | # |
please don't rebase this branch any more. there are branches building on top of that which throw > 30 conflicts each time you rebase... thanks.
| Daniel d'Andrada (dandrader) wrote : | # |
On 09/03/15 08:54, Michael Zanetti wrote:
> please don't rebase this branch any more. there are branches building on top of that which throw > 30 conflicts each time you rebase... thanks.
Oh, that sucks. I thought bzr would be smarter than that. Sorry.
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1656
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1657
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1658
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
| Albert Astals Cid (aacid) wrote : | # |
Text conflict in src/main.cpp
Text conflict in tests/qmltests/
Contents conflict in tests/qmltests/
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1658
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
| Daniel d'Andrada (dandrader) wrote : | # |
On 19/03/2015 05:14, Albert Astals Cid wrote:
> Text conflict in src/main.cpp
> Text conflict in tests/qmltests/
> Contents conflict in tests/qmltests/
Fixed.
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1659
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1660
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1663
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1664
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
| Albert Astals Cid (aacid) wrote : | # |
Text conflict in plugins/
Text conflict in qml/Shell.qml
Text conflict in qml/Stages/
Text conflict in src/main.cpp
Conflict adding file tests/mocks/
Conflict adding file tests/mocks/
Text conflict in tests/qmltests/
7 conflicts encountered.ly phase:adding file 0/89492
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1664
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
| Daniel d'Andrada (dandrader) wrote : | # |
On 14/04/15 04:21, Albert Astals Cid wrote:
> Text conflict in plugins/
> Text conflict in qml/Shell.qml
> Text conflict in qml/Stages/
> Text conflict in src/main.cpp
> Conflict adding file tests/mocks/
> Conflict adding file tests/mocks/
> Text conflict in tests/qmltests/
> 7 conflicts encountered.ly phase:adding file 0/89492
>
Fixed. Thanks for spotting that.
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1665
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1666
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1667
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
- 1668. By Daniel d'Andrada on 2015-04-20
-
Fix bad merge (add UnityCommandLin
eParser: :deviceName back)
| Michael Zanetti (mzanetti) wrote : | # |
2970+ // break binding
2971+ appWindowWithSh
This looks a bit odd... Wouldn't it be better to not use a binding in the first place - or - if using a binding, not breaking it? Seems like it makes things weird to use if bindings are broken randomly.
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1668
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
| Michael Zanetti (mzanetti) wrote : | # |
Visual glitch:
* Open a bunch of apps in portrait
* focus an app that supports landscape
* rotate to landscape
* use right edge to switch to another app
* rotate to portrait
* ender spread
=> the landscape app's size doesn't match the others. During the movement in the spread it should be scaled so that all surfaces have the same size when stacked up on the left edge. There is already a mechanism in place that does this for fullscreen vs non-fullscreen apps. You can probably reuse that somehow.
| Michael Zanetti (mzanetti) wrote : | # |
hmm... the dialer app breaks in landscape. We probably need to patch that to lock to portrait.
| Michael Zanetti (mzanetti) wrote : | # |
Another visual glitch:
* open an app supporting landscape and rotate to landscape
* press the power button
* press the power button again
=> the lock screen shows up squeezed to landscape and then it rotates. I guess in this case we should skip the rotation animation.
| Michael Zanetti (mzanetti) wrote : | # |
* open some apps supporting landscape but keep them in portrait before putting them to background
* now rotate the device to landscape
* use the right edge to switch to some other app
=> it'll animate to front, leaving some empty space where the panel has been before. then it'll resize in a jump to adjust to the new panel position and only after that it starts the rotation animation. Could we skip the intermediate resize jump somehow?
I think there would be two possibilities here. a) smoothly scale to the new panel location during the animation out of the spread or b) don't adjust to the new panel location before the rotation animation has started.
- 1669. By Daniel d'Andrada on 2015-04-22
-
Refactor TransformedSpre
adDelegate scaling logic into something I can understand
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1669
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
- 1670. By Daniel d'Andrada on 2015-04-22
-
Ensure that windows in the spread stack perfectly on the left of the spread
| Daniel d'Andrada (dandrader) wrote : | # |
> Visual glitch:
>
> * Open a bunch of apps in portrait
> * focus an app that supports landscape
> * rotate to landscape
> * use right edge to switch to another app
> * rotate to portrait
> * ender spread
>
> => the landscape app's size doesn't match the others. During the movement in
> the spread it should be scaled so that all surfaces have the same size when
> stacked up on the left edge. There is already a mechanism in place that does
> this for fullscreen vs non-fullscreen apps. You can probably reuse that
> somehow.
Fixed.
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1670
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
| Albert Astals Cid (aacid) wrote : | # |
Text conflict in debian/control
Text conflict in tests/plugins/
2 conflicts encountered.
- 1671. By Daniel d'Andrada on 2015-04-23
-
Merge trunk
[ Albert Astals Cid ]
* Compile with Qt 5.5 (LP: #1437238)
* Different way of top aligning labels when the other one in the row
is multiline (LP: #1442085)
* make pot_file
[ Andrea Cimitan ]
* Set sourceSize for DashBackground.qml Image
[ CI Train Bot ]
* New rebuild forced.
* Resync trunk. added: po/sk.po
[ Daniel d'Andrada ]
* Move handling of command line options to a separate class
* Refactor tst_PhysicalKeysMapper. qml
[ Leo Arias ]
* For autopilot tests, use the device simulation scenarios from the
toolkit.
[ Leonardo Arias Fonseca ]
* For autopilot tests, use the device simulation scenarios from the
toolkit.
[ Michael Zanetti ]
* Make sure dnd mode is ended properly when drag gesture is cancelled
(LP: #1444949)
[ Michał Sawicz ]
* Fix flake8 warnings (LP: #1444170)
* Move mock indicator service to unity8-fake-env, as it's a binary-
dependent package.
[ Nick Dedekind ]
* Use asynchronous dbus requests for property updates. (LP: #1436982)
| Daniel d'Andrada (dandrader) wrote : | # |
On 23/04/15 04:11, Albert Astals Cid wrote:
> Text conflict in debian/control
> Text conflict in tests/plugins/
> 2 conflicts encountered.
Fixed.
- 1672. By Daniel d'Andrada on 2015-04-23
-
Fix the positioning of the application window screenshot
| Daniel d'Andrada (dandrader) wrote : | # |
On 20/04/15 14:14, Michael Zanetti wrote:
> * open some apps supporting landscape but keep them in portrait before putting them to background
> * now rotate the device to landscape
> * use the right edge to switch to some other app
>
> => it'll animate to front, leaving some empty space where the panel has been before. then it'll resize in a jump to adjust to the new panel position and only after that it starts the rotation animation. [...]
Fixed.
- 1673. By Michael Zanetti on 2015-04-23
-
[ CI Train Bot ]
* New rebuild forced.
[ Daniel d'Andrada ]
* Tapping home key shows unity8-dash home added:
plugins/Utils/ElapsedTi mer.h plugins/ Utils/HomeKeyWa tcher.cpp
plugins/Utils/HomeKeyWa tcher.h plugins/ Utils/Timer. cpp
plugins/Utils/Timer. h tests/plugins/ Utils/homekeywa tchertest. cpp
[ Michał Sawicz ]
* Tapping home key shows unity8-dash home added:
plugins/Utils/ElapsedTi mer.h plugins/ Utils/HomeKeyWa tcher.cpp
plugins/Utils/HomeKeyWa tcher.h plugins/ Utils/Timer. cpp
plugins/Utils/Timer. h tests/plugins/ Utils/homekeywa tchertest. cpp - 1674. By Michael Zanetti on 2015-04-23
-
fix bad merge
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1671
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1674
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
- 1675. By Daniel d'Andrada on 2015-04-24
-
Don't animate rotation when greeter is shown or while display is off
| Daniel d'Andrada (dandrader) wrote : | # |
> Another visual glitch:
>
> * open an app supporting landscape and rotate to landscape
> * press the power button
> * press the power button again
>
> => the lock screen shows up squeezed to landscape and then it rotates. I guess
> in this case we should skip the rotation animation.
Done.
| Daniel d'Andrada (dandrader) wrote : | # |
> 2970+ // break binding
> 2971+ appWindowWithSh
> appWindowWithSh
>
> This looks a bit odd... Wouldn't it be better to not use a binding in the
> first place - or - if using a binding, not breaking it? Seems like it makes
> things weird to use if bindings are broken randomly.
No. That's the main point of this state, keepSceneRotati
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1675
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
| Michael Zanetti (mzanetti) wrote : | # |
I've beeen giving it a test on the N7 during my flight yesterday. There are two things:
a) http://
To repro open some main and side stage apps, focus a rotatable main stage app, rotate to portrait, go to spread, select a side stage app. It should rotate to landscape I'd say.
Also not really sure what should happen if you're on portrait and nextInStack is a side stage app... That's an open design question I guess. If you have an idea on that, let me know, otherwise, let's leave it as is. Getting rotation on phone seems more important than having pixel-perfect spread/
b) in the spread, side stage and main stage items have different heights. I had a chat with design... We can probably leave it as is until we have a real tablet product.
| Daniel d'Andrada (dandrader) wrote : | # |
On 27/04/15 07:59, Michael Zanetti wrote:
> Review: Needs Information
>
> I've beeen giving it a test on the N7 during my flight yesterday. There are two things:
>
> a) http://
>
> To repro open some main and side stage apps, focus a rotatable main stage app, rotate to portrait, go to spread, select a side stage app. It should rotate to landscape I'd say.
>
> Also not really sure what should happen if you're on portrait and nextInStack is a side stage app... That's an open design question I guess. If you have an idea on that, let me know, otherwise, let's leave it as is. Getting rotation on phone seems more important than having pixel-perfect spread/
This was the best compromise I could get.
Rotating to landscape but staying in in side stage size wouldn't look
any better. Imagine you then minimize the side stage, rotate the device
back to landscape and then show the side stage again. You would get a
landscape side stage that would rotate to portrait. Pretty horrible. Not
to mention the extra complexity involved. To avoid that we could
pre-rotate the sidestage app back to portrait behind the scenes but
there are 3 problems with that: 1 - resizing two surfaces at roughly the
same time crashes mir (or qtmir maybe). I don't wanna be blocked on that
bug. 2 - We would have to keep the side stage app running behind the
scenes for a bit to be able to resize it. 3 - more code complexity and
work for something we really have no design for and the whole thing
could pretty easily get scrapped on the next design.
> b) in the spread, side stage and main stage items have different heights. I had a chat with design... We can probably leave it as is until we have a real tablet product.
>
Yes, please! It's very likely that this is an existing problem with the
current tablet code.
| Michael Zanetti (mzanetti) wrote : | # |
> > Visual glitch:
> >
> > * Open a bunch of apps in portrait
> > * focus an app that supports landscape
> > * rotate to landscape
> > * use right edge to switch to another app
> > * rotate to portrait
> > * ender spread
> >
> > => the landscape app's size doesn't match the others. During the movement in
> > the spread it should be scaled so that all surfaces have the same size when
> > stacked up on the left edge. There is already a mechanism in place that does
> > this for fullscreen vs non-fullscreen apps. You can probably reuse that
> > somehow.
>
> Fixed.
I can still see this: http://
See how the landscape contacts app is smaller than the other (portrait) apps.
| Michał Sawicz (saviq) wrote : | # |
W dniu 28.04.2015 o 18:56, Michael Zanetti pisze:
> See how the landscape contacts app is smaller than the other (portrait) apps.
That highlighted one more issue - screenshots should be saved with
current orientation :)
- 1676. By Daniel d'Andrada on 2015-04-28
-
Make all items in the spread have the very same size when stacked on the left side
| Daniel d'Andrada (dandrader) wrote : | # |
> > > Visual glitch:
> > >
> > > * Open a bunch of apps in portrait
> > > * focus an app that supports landscape
> > > * rotate to landscape
> > > * use right edge to switch to another app
> > > * rotate to portrait
> > > * ender spread
> > >
> > > => the landscape app's size doesn't match the others. During the movement
> in
> > > the spread it should be scaled so that all surfaces have the same size
> when
> > > stacked up on the left edge. There is already a mechanism in place that
> does
> > > this for fullscreen vs non-fullscreen apps. You can probably reuse that
> > > somehow.
> >
> > Fixed.
>
> I can still see this: http://
>
> See how the landscape contacts app is smaller than the other (portrait) apps.
Fixed.
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1676
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
| Albert Astals Cid (aacid) wrote : | # |
Yesterday at some point we got http://
I'll try to see if I can reproduce it.
| Daniel d'Andrada (dandrader) wrote : | # |
Whatever bug still remains, it can't be a show stopper and therefore could be dealt with later. The merging of this branch is long overdue.
| Michael Zanetti (mzanetti) wrote : | # |
> Whatever bug still remains, it can't be a show stopper and therefore could be
> dealt with later. The merging of this branch is long overdue.
I think the fact that the dialer app breaks is a showstopper. You didn't address that yet, did you? Besides, landing is frozen atm.
| Daniel d'Andrada (dandrader) wrote : | # |
On 29/04/15 09:50, Michael Zanetti wrote:
>> Whatever bug still remains, it can't be a show stopper and therefore could be
>> dealt with later. The merging of this branch is long overdue.
> I think the fact that the dialer app breaks is a showstopper. You didn't address that yet, did you? Besides, landing is frozen atm.
You mean that the dialer app looks bad in landscape? It's app to them to
provide the supported orientations in their desktop file
| Michael Zanetti (mzanetti) wrote : | # |
| Albert Astals Cid (aacid) wrote : | # |
Text conflict in qml/Stages/
Text conflict in qml/Stages/
2 conflicts encountered.
| Nick Dedekind (nick-dedekind) wrote : | # |
I'm pretty sure u8 shouldn't be depending on libandroid-
- 1677. By Daniel d'Andrada on 2015-05-11
-
Merge trunk
[ Albert Astals Cid ]
* Make runtests fake a test error if make fails
* Make the test more stable
* Use dbus-test-runner instead of dbus-launch
* DirectionalDragArea: improvements & API grooming (LP: #1417920)
* Fix EdgeDragEvaluator when a drag can happen both ways
(Direction.Horizontal)
[ Josh Arenson ]
* Remove panelHeight property as it is unused.
[ Leo Arias ]
* Initial clean up of the autopilot tests set up. Removed the touch
device from the test case. Moved the restart of unity to a fixture.
Removed the unused DragMixin. Updated the setUpClass to use
process_helpers. Removed the workaround for bug #1238417, already
fixed. Use the toolkit helper to set the testability environment
variable. Fixed the indicators base class that was restarting unity
twice. (LP: #1238417, #1447206)
* Use the base class from the toolkit in autopilot tests.
[ Michael Zanetti ]
* emit application-stop when we're going down (LP: #1326513)
[ Michał Sawicz ]
* UNITY_SCOPES_LIST is no more
[ handsome_feng<email address hidden> ]
* When click the favorite scope in Dash Manager , it just return to
the corresponding scope page. (LP: #1447056) - 1678. By Daniel d'Andrada on 2015-05-11
-
Improve test code
| Daniel d'Andrada (dandrader) wrote : | # |
On 11/05/15 04:23, Albert Astals Cid wrote:
> Text conflict in qml/Stages/
> Text conflict in qml/Stages/
> 2 conflicts encountered.
Fixed
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1678
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
| Michael Zanetti (mzanetti) wrote : | # |
> I'm pretty sure u8 shouldn't be depending on libandroid-
> doesn't sound right. Perhaps this should be resolved in qtubuntu somehow?
This is a fair point. At this point I probably wouldn't open that can of worms any more. The device detection is very likely to be expanded a lot in the near future. I'd say we clean this up when the bigger picture gets more visible.
| Nick Dedekind (nick-dedekind) wrote : | # |
windowgeometry command line option doesn't seem to work in either u8 or dash (does in trunk).
As I haven't really been over the MP; if it's not supposed to work any more, then the option should be removed. At the moment it's just opening a small window, which needs to be resized every time.
| Nick Dedekind (nick-dedekind) wrote : | # |
file://
Seems you removed the ApplicationArgu
- 1679. By Daniel d'Andrada on 2015-05-12
-
Make -windowgeometry work again
| Daniel d'Andrada (dandrader) wrote : | # |
On 12/05/15 05:38, Nick Dedekind wrote:
> Review: Needs Fixing
>
> windowgeometry command line option doesn't seem to work in either u8 or dash (does in trunk).
> As I haven't really been over the MP; if it's not supposed to work any more, then the option should be removed. At the moment it's just opening a small window, which needs to be resized every time.
Fixed
| Daniel d'Andrada (dandrader) wrote : | # |
On 12/05/15 06:20, Nick Dedekind wrote:
> file://
>
> Seems you removed the ApplicationArgu
Yeah, not using this approach anymore.
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1679
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
| Albert Astals Cid (aacid) wrote : | # |
Text conflict in qml/Stages/
Text conflict in qml/Stages/
2 conflicts encountered.
- 1680. By Daniel d'Andrada on 2015-05-21
-
Merge trunk
[ Albert Astals Cid ]
* Add overrides to override functions
* Implement "rating-edit" preview widget (LP: #1318144)
* Make the DashContent::test_mainNavig ation test more stable (LP:
#1450809)
* Use art height as implicitHeight when the header is overlayed and
there's no summary
[ CI Train Bot ]
* New rebuild forced.
* Resync trunk.
[ Daniel d'Andrada ]
* Introducing FloatingFlickable
* Make Ubuntu.Gestures components install TouchRegistry by themselves
[ Michael Terry ]
* Fix a possible crash in our PAM threading code. (LP: #1425362) (LP:
#1425362)
* Fix the lockscreen becoming unresponsive after testing an app on the
device from QtCreator. (LP: #1435364)
[ Nick Dedekind ]
* Fixed desktop stage app focus.
* Fixed issue in laggy indicator autpilot tests (LP: #1446846)
[ Albert Astals Cid ]
* Workarounds for concierge mode.
[ CI Train Bot ]
* New rebuild forced.
* Resync trunk. - 1681. By Daniel d'Andrada on 2015-05-21
-
Fix build regarding unity-api and add screenshot provider to the mock Utils plugin
| Daniel d'Andrada (dandrader) wrote : | # |
On 21/05/2015 09:03, Albert Astals Cid wrote:
> Text conflict in qml/Stages/
> Text conflict in qml/Stages/
> 2 conflicts encountered.
>
Fixed.
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1681
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://

Text conflict in tests/mocks/ Unity/Applicati on/ApplicationM anager. h tst_Shell. qml
Text conflict in tests/qmltests/
2 conflicts encountered.