[DASH] activity indicator not shown while search process is ongoing
Bug #1124567 reported by
Oren Horev
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| unity8 (Ubuntu) |
Fix Released
|
High
|
Nicolas d'Offay | ||
Bug Description
An activity indicator (spinner) should be shown for the duration of search latency. This should happen from the entry of 1st character until search is completed., and should resume when query changes for the duration of the new search.
Related branches
lp:~nicolas-doffay/unity8/search-indicator
- PS Jenkins bot (community): Approve (continuous-integration)
- Michał Sawicz: Approve
- Michal Hruby (community): Approve
-
Diff: 409 lines (+128/-9)7 files modifiedComponents/PageHeader.qml (+44/-1)
Dash/GenericScopeView.qml (+1/-0)
plugins/Unity/scope.cpp (+30/-5)
plugins/Unity/scope.h (+7/-2)
tests/mocks/Unity/fake_scope.cpp (+13/-0)
tests/mocks/Unity/fake_scope.h (+5/-1)
tests/qmltests/Components/tst_PageHeader.qml (+28/-0)
lp:~aacid/unity8/unrevert376
- PS Jenkins bot (community): Approve (continuous-integration)
- Michał Sawicz: Approve
-
Diff: 409 lines (+128/-9)7 files modifiedComponents/PageHeader.qml (+44/-1)
Dash/GenericScopeView.qml (+1/-0)
plugins/Unity/scope.cpp (+30/-5)
plugins/Unity/scope.h (+7/-2)
tests/mocks/Unity/fake_scope.cpp (+13/-0)
tests/mocks/Unity/fake_scope.h (+5/-1)
tests/qmltests/Components/tst_PageHeader.qml (+28/-0)
| Changed in manhattan: | |
| status: | New → Confirmed |
| tags: | added: unity-public |
| information type: | Private → Public |
| affects: | manhattan → unity8 |
| Changed in unity8: | |
| assignee: | Michał Sawicz (saviq) → Unity UI Team (unity-ui-team) |
| Changed in unity8: | |
| status: | Confirmed → Triaged |
| Changed in unity8: | |
| assignee: | Unity UI Team (unity-ui-team) → Nicolas d'Offay (nicolas-doffay) |
| status: | Triaged → In Progress |
| Changed in unity8: | |
| milestone: | none → ubuntu-13.09 |
| Changed in unity8: | |
| status: | In Progress → Fix Committed |
| Changed in unity8: | |
| status: | Fix Committed → In Progress |
| Changed in unity8: | |
| status: | In Progress → Fix Committed |
Revision history for this message
| Michał Sawicz (saviq) wrote | #1 |
| Changed in unity8: | |
| status: | Fix Committed → In Progress |
Revision history for this message
| PS Jenkins bot (ps-jenkins) wrote | #2 |
| Changed in unity8: | |
| status: | In Progress → Fix Committed |
| Changed in unity8: | |
| status: | Fix Committed → Fix Released |
| Changed in unity8 (Ubuntu): | |
| status: | New → Fix Released |
| Changed in unity8: | |
| status: | Fix Released → Fix Committed |
| Changed in unity8: | |
| status: | Fix Committed → Fix Released |
| tags: | added: has-test |
| Changed in unity8 (Ubuntu): | |
| assignee: | nobody → Nicolas d'Offay (nicolas-doffay) |
| importance: | Undecided → High |
| no longer affects: | unity8 |
To post a comment you must log in.
Had to revert this due to constant CPU usage of the ActivityIndicator - need to investigate a bug in the SDK.