Merge lp://staging/~zsombi/ubuntu-ui-toolkit/dynamic-tabs-reloaded into lp://staging/ubuntu-ui-toolkit
- dynamic-tabs-reloaded
- Merge into trunk
Status: | Superseded |
---|---|
Proposed branch: | lp://staging/~zsombi/ubuntu-ui-toolkit/dynamic-tabs-reloaded |
Merge into: | lp://staging/ubuntu-ui-toolkit |
Diff against target: |
1137 lines (+767/-49) 14 files modified
CHANGES (+1/-0) components.api (+8/-1) modules/Ubuntu/Components/Tab.qml (+0/-7) modules/Ubuntu/Components/Tabs.qml (+292/-20) modules/Ubuntu/Components/Themes/Ambiance/TabBarStyle.qml (+9/-2) modules/Ubuntu/Components/plugin/quickutils.cpp (+15/-0) modules/Ubuntu/Components/plugin/quickutils.h (+1/-0) modules/Ubuntu/Test/UbuntuTestCase.qml (+69/-1) modules/Ubuntu/Test/deployment.pri (+6/-1) tests/resources/navigation/Tabs.qml (+90/-1) tests/unit/runtest.sh (+1/-1) tests/unit_x11/tst_components/ExternalTab.qml (+21/-0) tests/unit_x11/tst_components/tst_tabs.qml (+151/-0) tests/unit_x11/tst_test/tst_ubuntutestcase.qml (+103/-15) |
To merge this branch: | bzr merge lp://staging/~zsombi/ubuntu-ui-toolkit/dynamic-tabs-reloaded |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
PS Jenkins bot | continuous-integration | Approve | |
Tim Peeters | Approve | ||
Cris Dywan | provided moveitembefore docs are fixed | Approve | |
Review via email:
|
This proposal has been superseded by a proposal from 2014-04-10.
Commit message
Extending Tabs with functions to dynamically add, move and remove tabs.
Description of the change

PS Jenkins bot (ps-jenkins) wrote : | # |

PS Jenkins bot (ps-jenkins) wrote : | # |
PASSED: Continuous integration, rev:901
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://

Tim Peeters (tpeeters) wrote : | # |
637 + * Copyright 2012 Canonical Ltd.
2013

Cris Dywan (kalikiana) wrote : | # |
I'm seeing a number of errors while using the tabbar test:
modules/
coming from connectToRepeaters
modules/
coming from insertTab
modules/
coming from anchors.top on the Repeater, not sure what this is

Cris Dywan (kalikiana) wrote : | # |
I'm liking the documentation which is quite decent. I love that the API keeps the different types of tabs behind the scenes.
> // but move only if there are more than on eitems in the list
Typo.
// QQuickItem *parentItem = item->parentItem();
Please remove this one.

PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:902
http://
Executed test runs:
FAILURE: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
FAILURE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://

PS Jenkins bot (ps-jenkins) wrote : | # |
PASSED: Continuous integration, rev:902
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://

Zsombor Egri (zsombi) wrote : | # |
> 637 + * Copyright 2012 Canonical Ltd.
>
> 2013
It's 2014 now :)

Zsombor Egri (zsombi) wrote : | # |
> I'm liking the documentation which is quite decent. I love that the API keeps
> the different types of tabs behind the scenes.
>
> > // but move only if there are more than on eitems in the list
> Typo.
>
> // QQuickItem *parentItem = item->parentItem();
> Please remove this one.
Fixed in revno 903

PS Jenkins bot (ps-jenkins) wrote : | # |
PASSED: Continuous integration, rev:903
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://

PS Jenkins bot (ps-jenkins) wrote : | # |
PASSED: Continuous integration, rev:904
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://

Tim Peeters (tpeeters) wrote : | # |
+/*!
463 + * \internal
464 + * Moves a given item to the specified index in its parent's child list.
465 + */
466 +void QuickUtils:
description doesn't exactly match the function specs.
* Moves a given item before the specified item in their parent's child list
?

Cris Dywan (kalikiana) wrote : | # |
That one escaped me; I'll give green light with the above sentence fixed.

Cris Dywan (kalikiana) wrote : | # |
CI@HOME OK http://

Tim Peeters (tpeeters) wrote : | # |
note that you didn't fix the doc yet

Tim Peeters (tpeeters) wrote : | # |
I changed the status back to needs review, because we need to re-do the CI@home with qt52.

Zsombor Egri (zsombi) wrote : | # |
> note that you didn't fix the doc yet
Sorry, done now.

PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:906
http://
Executed test runs:
FAILURE: http://
SUCCESS: http://
FAILURE: http://
SUCCESS: http://
deb: http://
FAILURE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://

Tim Peeters (tpeeters) wrote : | # |
> modules/
> defined
This is still broken. Please add the include in Tabs.qml.

Tim Peeters (tpeeters) wrote : | # |
> I'm seeing a number of errors while using the tabbar test:
>
> modules/
> 'indexOf' of undefined
> coming from connectToRepeaters
> modules/
> defined
> coming from insertTab
> modules/
> Cannot read property of null
> coming from anchors.top on the Repeater, not sure what this is
please double-check that all these are fixed

Zsombor Egri (zsombi) wrote : | # |
>
> > modules/
> > defined
>
> This is still broken. Please add the include in Tabs.qml.
I'm wondering why was it passing till now???

PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:907
http://
Executed test runs:
SUCCESS: http://
FAILURE: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
deb: http://
FAILURE: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://

PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:908
http://
Executed test runs:
SUCCESS: http://
FAILURE: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
deb: http://
FAILURE: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://

PS Jenkins bot (ps-jenkins) wrote : | # |
PASSED: Continuous integration, rev:909
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://

PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:911
http://
Executed test runs:
FAILURE: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
ABORTED: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
Click here to trigger a rebuild:
http://

PS Jenkins bot (ps-jenkins) wrote : | # |
PASSED: Continuous integration, rev:911
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
Click here to trigger a rebuild:
http://

PS Jenkins bot (ps-jenkins) wrote : | # |
PASSED: Continuous integration, rev:912
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
Click here to trigger a rebuild:
http://
- 913. By Zsombor Egri
-
staging merge
FAILED: Continuous integration, rev:900 jenkins. qa.ubuntu. com/job/ ubuntu- ui-toolkit- ci/1483/ jenkins. qa.ubuntu. com/job/ generic- mediumtests- trusty/ 1856/console jenkins. qa.ubuntu. com/job/ generic- mediumtests- trusty- touch/1773/ console jenkins. qa.ubuntu. com/job/ ubuntu- ui-toolkit- trusty- amd64-ci/ 431/console jenkins. qa.ubuntu. com/job/ ubuntu- ui-toolkit- trusty- armhf-ci/ 431/console jenkins. qa.ubuntu. com/job/ generic- mediumtests- builder- trusty- amd64/1856/ console jenkins. qa.ubuntu. com/job/ generic- mediumtests- builder- trusty- armhf/1773/ console
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild: s-jenkins. ubuntu- ci:8080/ job/ubuntu- ui-toolkit- ci/1483/ rebuild
http://