Merge lp://staging/~tpeeters/ubuntu-ui-toolkit/80-betterActive into lp://staging/ubuntu-ui-toolkit/staging

Proposed by Tim Peeters
Status: Merged
Approved by: Zsombor Egri
Approved revision: 1060
Merged at revision: 1048
Proposed branch: lp://staging/~tpeeters/ubuntu-ui-toolkit/80-betterActive
Merge into: lp://staging/ubuntu-ui-toolkit/staging
Diff against target: 198 lines (+55/-12)
7 files modified
components.api (+1/-0)
modules/Ubuntu/Components/MainView.qml (+5/-1)
modules/Ubuntu/Components/Page.qml (+2/-0)
modules/Ubuntu/Components/PageStack.qml (+11/-0)
modules/Ubuntu/Components/PageTreeNode.qml (+10/-8)
tests/unit/tst_components/tst_pagestack.qml (+23/-3)
tests/unit/tst_mainview/PageStack.qml (+3/-0)
To merge this branch: bzr merge lp://staging/~tpeeters/ubuntu-ui-toolkit/80-betterActive
Reviewer Review Type Date Requested Status
PS Jenkins bot continuous-integration Approve
Zsombor Egri Approve
Review via email: mp+219093@code.staging.launchpad.net

Commit message

Improve automatic setting of the active property of Pages, Tabs, PageStack.

Pages defined as children of PageStack are now not active by default until they are pushed on the stack, and inactive pages are not visible.

Description of the change

Improve automatic setting of the active property of Pages, Tabs, PageStack.

Pages defined as children of PageStack are now not active by default until they are pushed on the stack, and inactive pages are not visible.

To post a comment you must log in.
1051. By Tim Peeters

clean

1052. By Tim Peeters

keep the old default visibility for PageTreeNode

1053. By Tim Peeters

visible change in gallery no longer needed

1054. By Tim Peeters

add regression test

1055. By Tim Peeters

add regression test

1056. By Tim Peeters

wake up CI

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: Needs Fixing (continuous-integration)
1057. By Tim Peeters

update components.api

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

You have added a new property to the PageStack API, and still kept import version. That is not allowed anymore!

review: Needs Fixing
1058. By Tim Peeters

fix unit test

1059. By Tim Peeters

keep previous API

1060. By Tim Peeters

update components.api

Revision history for this message
Zsombor Egri (zsombi) wrote :

Looks good now

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

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