Merge lp://qastaging/~tiagosh/unity-2d/unity-2d-shell-homelens into lp://qastaging/unity-2d
Proposed by
Tiago Salem Herrmann
Status: | Merged |
---|---|
Approved by: | Albert Astals Cid |
Approved revision: | 952 |
Merged at revision: | 910 |
Proposed branch: | lp://qastaging/~tiagosh/unity-2d/unity-2d-shell-homelens |
Merge into: | lp://qastaging/unity-2d |
Diff against target: |
341 lines (+92/-64) 9 files modified
libunity-2d-private/src/lens.cpp (+15/-3) libunity-2d-private/src/lenses.cpp (+8/-5) libunity-2d-private/src/lenses.h (+2/-0) shell/app/shelldeclarativeview.cpp (+7/-0) shell/app/shelldeclarativeview.h (+2/-0) shell/dash/Dash.qml (+25/-17) shell/dash/LensBar.qml (+20/-19) shell/dash/LensView.qml (+7/-6) shell/dash/RendererGrid.qml (+6/-14) |
To merge this branch: | bzr merge lp://qastaging/~tiagosh/unity-2d/unity-2d-shell-homelens |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Albert Astals Cid (community) | Approve | ||
Gerry Boland | Pending | ||
Michał Sawicz | Pending | ||
Review via email:
|
This proposal supersedes a proposal from 2012-01-31.
Description of the change
This MR replaces the old home lens by the new one provided by unity.
The following libqtdee branch must be installed in order to compile this branch: lp:~tiagosh/dee-qt/dee-qt-local-models .
To post a comment you must log in.
Hi Tiago,
great work, thanks for this! But we need tests to be able to accept this.
I think testing the following will be sufficient:
1. open dash, check somehow that this Home lens is showing
2. open dash, type a character, check search results appear in Home lens
Consult the existing tests to get an idea where to start. Getting started guide here: /wiki.ubuntu. com/Unity2DTest ability /bazaar. launchpad. net/~gerboland/ unity-2d/ dash-test/ view/head: /tests/ places/ show_hide_ tests.rb
https:/
and this old test of mine might help you get started. Not it's for unity-2d, so things have changed a little
https:/
Ping me if you need a hand
-G