Merge lp:~brandontschaefer/unity/fix-711199 into lp:unity
Proposed by
Brandon Schaefer
Status: | Merged |
---|---|
Approved by: | Tim Penhey |
Approved revision: | no longer in the source branch. |
Merged at revision: | 2093 |
Proposed branch: | lp:~brandontschaefer/unity/fix-711199 |
Merge into: | lp:unity |
Diff against target: |
300 lines (+147/-6) 5 files modified
plugins/unityshell/src/DashView.cpp (+36/-5) plugins/unityshell/src/DashView.h (+3/-0) plugins/unityshell/src/LensView.cpp (+58/-1) plugins/unityshell/src/LensView.h (+5/-0) tests/autopilot/autopilot/tests/test_dash.py (+45/-0) |
To merge this branch: | bzr merge lp:~brandontschaefer/unity/fix-711199 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Tim Penhey (community) | Approve | ||
Alex Launi (community) | Approve | ||
Michal Hruby (community) | Approve | ||
Marco Trevisan (Treviño) | Approve | ||
Brandon Schaefer (community) | Approve | ||
Review via email: mp+89192@code.launchpad.net |
Description of the change
= Problem description =
If no results are returned from a lens there is no message informing the user so.
= The fix =
In DashView:
shown.
A default message is used if the lens doesn't provide one.
There is also a timer that waits for 150ms then hides the
message, if the search is taking a while.
= Test coverage =
There is an autopilot test for this now!
To post a comment you must log in.
Need to not use a work around for the start up problem. Looking into removing the call in LensView: bazaar. launchpad. net/~unity- team/unity/ trunk/view/ head:/plugins/ unityshell/ src/LensView. cpp#L350
http://
Removing that if statement which calls Search("") and OnSearchFinished is getting emitted with 0 results for ALL lenses, even though they have them. So far testing shows this code can be removed, but want to do more testing by killing the lenses and restarting them.