Merge lp://staging/~tpeeters/ubuntu-ui-toolkit/optIn-newHeader into lp://staging/ubuntu-ui-toolkit

Proposed by Tim Peeters
Status: Merged
Approved by: Zsombor Egri
Approved revision: 949
Merged at revision: 988
Proposed branch: lp://staging/~tpeeters/ubuntu-ui-toolkit/optIn-newHeader
Merge into: lp://staging/ubuntu-ui-toolkit
Diff against target: 304 lines (+45/-83)
5 files modified
components.api (+1/-0)
modules/Ubuntu/Components/MainView.qml (+42/-16)
modules/Ubuntu/Components/Toolbar.qml (+1/-1)
tests/unit/tst_components/tst_toolbaritems.qml (+1/-2)
tests/unit_x11/tst_inversemousearea/InverseMouseAreaInListView.qml (+0/-64)
To merge this branch: bzr merge lp://staging/~tpeeters/ubuntu-ui-toolkit/optIn-newHeader
Reviewer Review Type Date Requested Status
PS Jenkins bot continuous-integration Approve
Zsombor Egri Approve
Review via email: mp+208662@code.staging.launchpad.net

Commit message

Add useDeprecatedToolbar property to MainView which can be used to disable the current toolbar. Unsetting that property will in future also enable new features in the header.

Description of the change

Add bool to MainView to disable the old toolbar and use the new header functionality when it is done. Also add a position property to the Panel to make it more useful for implementing new bottom-edge behavior.

We need this so that app developers can disable the toolbar and implement their own new bottom-edge behavior, see http://design.canonical.com/2014/03/loving-the-bottom-edge/

To post a comment you must log in.
Revision history for this message
Tim Peeters (tpeeters) wrote :

Check out https://code.launchpad.net/~tpeeters/+junk/vscale to see an example of how the Panel can be used.

924. By Tim Peeters

update components.api

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
925. By Tim Peeters

rename useOldToolbar to useDeprecatedToolbar

926. By Tim Peeters

only load toolbar when it is used

927. By Tim Peeters

update components.api

928. By Tim Peeters

toolbar always visible when loaded

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
929. By Tim Peeters

remove initialization test that was only correct by luck/good timing

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
930. By Tim Peeters

empty

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
931. By Tim Peeters

merge trunk

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
932. By Tim Peeters

huh? Why did jenkisn fail?

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
933. By Tim Peeters

stop Loader from eating events

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
934. By Tim Peeters

fix typo in Toolbar.qml comment

935. By Tim Peeters

fix parentage of toolbar

936. By Tim Peeters

merge trunk

937. By Tim Peeters

fix typo properly

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Tim Peeters (tpeeters) wrote :

Some failures in CI@home for unity8 and system settings: http://paste.ubuntu.com/7044989/
There is a bug in unity8 with qt5 that can cause these. Should be fixed with qt5.2.

938. By Tim Peeters

link bug

939. By Tim Peeters

merge trunk (translations only)

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Zsombor Egri (zsombi) wrote :

Things look good.

review: Approve
940. By Tim Peeters

merge trunk

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
941. By Tim Peeters

merge trunk

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
942. By Tim Peeters

merge trunk

943. By Tim Peeters

merge ima-bug1288876. See if jenkins CI passes now.

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Tim Peeters (tpeeters) wrote :

CI@home results:
- Several failures in music-app
- Several failures in ubuntu-clock-app
- Several failures in dialer-app: StateNotFoundError: Object not found with name '*' and properties {'objectName': 'historyList'}. (phone didn't have sim-card while testing)
- One error in unity8: unity8.indicators.tests.test_indicators.IndicatorTestCase.test_indicator_exists(Bluetooth) (phone has bluetooth disabled)

Revision history for this message
Tim Peeters (tpeeters) wrote :

CI@home full results: http://paste.ubuntu.com/7111217/

944. By Tim Peeters

revert ima-bug1288876 merge

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
945. By Tim Peeters

retry

Revision history for this message
Tim Peeters (tpeeters) wrote :

previous CI failed because device screen unlock failed because the device was not 'clean' (left-over installed packages from another MR were there).

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
946. By Tim Peeters

CI jobs fixed. retry

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
947. By Tim Peeters

merge trunk

948. By Tim Peeters

re-adding InverseMouseAreaInListView.qml which got lost in the previous trunk merge

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
949. By Tim Peeters

merge translations from trunk

Revision history for this message
Tim Peeters (tpeeters) wrote :

I don't know why the diff lists
237 === removed file 'tests/unit_x11/tst_inversemousearea/InverseMouseAreaInListView.qml'
238 --- tests/unit_x11/tst_inversemousearea/InverseMouseAreaInListView.qml 2014-03-20 15:30:06 +0000
239 +++ tests/unit_x11/tst_inversemousearea/InverseMouseAreaInListView.qml 1970-01-01 00:00:00 +0000

and

168 === added file 'tests/unit_x11/tst_inversemousearea/InverseMouseAreaInListView.qml'
169 --- tests/unit_x11/tst_inversemousearea/InverseMouseAreaInListView.qml 1970-01-01 00:00:00 +0000
170 +++ tests/unit_x11/tst_inversemousearea/InverseMouseAreaInListView.qml 2014-03-26 17:43:24 +0000

The supposedly removed and added files are identical. There was a conflict with that file when merging trunk but it seemed resolved.

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Tim Peeters (tpeeters) wrote :

ran clock-app tests 3 time without failures: http://paste.ubuntu.com/7184819/

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to status/vote changes: