Merge lp:~tpeeters/ubuntu-ui-toolkit/20-toolbar into lp:ubuntu-ui-toolkit/staging
| Status: | Merged |
|---|---|
| Approved by: | Zsombor Egri on 2015-12-16 |
| Approved revision: | 1756 |
| Merged at revision: | 1772 |
| Proposed branch: | lp:~tpeeters/ubuntu-ui-toolkit/20-toolbar |
| Merge into: | lp:ubuntu-ui-toolkit/staging |
| Diff against target: |
753 lines (+210/-182) 16 files modified
components.api (+5/-0) examples/ubuntu-ui-toolkit-gallery/PageHeaders.qml (+137/-0) examples/ubuntu-ui-toolkit-gallery/WidgetsModel.qml (+5/-0) src/Ubuntu/Components/1.2/DeprecatedToolbar.qml (+1/-1) src/Ubuntu/Components/1.2/MainView.qml (+2/-2) src/Ubuntu/Components/1.2/PageTreeNode.qml (+1/-1) src/Ubuntu/Components/1.3/PageHeader.qml (+1/-0) src/Ubuntu/Components/1.3/Toolbar.qml (+0/-147) src/Ubuntu/Components/ComponentModule.pro (+2/-2) src/Ubuntu/Components/Styles/1.3/ToolbarStyle.qml (+32/-0) src/Ubuntu/Components/Styles/Styles.pro (+1/-0) src/Ubuntu/Components/Styles/qmldir (+1/-0) src/Ubuntu/Components/Themes/Ambiance/1.3/ToolbarStyle.qml (+17/-26) src/Ubuntu/Components/Themes/Ambiance/Ambiance.pro (+1/-1) src/Ubuntu/Components/Themes/Ambiance/qmldir (+2/-1) src/Ubuntu/Components/qmldir (+2/-1) |
| To merge this branch: | bzr merge lp:~tpeeters/ubuntu-ui-toolkit/20-toolbar |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Zsombor Egri (community) | 2015-12-15 | Approve on 2015-12-16 | |
| PS Jenkins bot | continuous-integration | Approve on 2015-12-15 | |
|
Review via email:
|
|||
Commit Message
Introduce the new Toolbar component for the header edit mode.
Description of the Change
The functionality of the Toolbar is already tested in the unit tests for ActionBar, because the Toolbar is basically just two ActionBars. In autopilot tests, simply use toolbar.
- 1752. By Tim Peeters on 2015-12-15
-
clean
- 1753. By Tim Peeters on 2015-12-15
-
remove tst_toolbar.qml
- 1754. By Tim Peeters on 2015-12-15
-
update components.api
- 1755. By Tim Peeters on 2015-12-15
-
use Component instead of Item for delegate type in example code.
- 1756. By Tim Peeters on 2015-12-15
-
remove FIXME from example code
| Tim Peeters (tpeeters) wrote : | # |
I couldn't keep 1.2/Toolbar.qml because then the qmldir has an internal and a 1.3 Toolbar component defined. I changed the name of ToolbarStyle as well to avoid confusion.
| PS Jenkins bot (ps-jenkins) wrote : | # |
PASSED: Continuous integration, rev:1751
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
| PS Jenkins bot (ps-jenkins) wrote : | # |
PASSED: Continuous integration, rev:1756
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
| Zsombor Egri (zsombi) wrote : | # |
Ok, fair point for renaming the old internal ToolbarStyle. For the unit tests, make sure you create one once additional functionality will be needed that differs from the ActionBar.

1.2 ToolbarStyle should not affect the 1.3 Toolbar component. Theme.createSty leComponent( ) will always use 1.2 style to look for the style.