Merge lp:~mzanetti/unity/phablet-hide-static-apps-while-search into lp:unity/phablet

Proposed by Michael Zanetti
Status: Merged
Approved by: Michał Sawicz
Approved revision: no longer in the source branch.
Merged at revision: 613
Proposed branch: lp:~mzanetti/unity/phablet-hide-static-apps-while-search
Merge into: lp:unity/phablet
Diff against target: 19 lines (+6/-3)
1 file modified
Dash/DashApps.qml (+6/-3)
To merge this branch: bzr merge lp:~mzanetti/unity/phablet-hide-static-apps-while-search
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Approve
Michał Sawicz Approve
Review via email: mp+159857@code.launchpad.net

Commit message

hide static parts of apps dash while searching for apps

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Michał Sawicz (saviq) :
review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'Dash/DashApps.qml'
2--- Dash/DashApps.qml 2013-04-15 14:31:47 +0000
3+++ Dash/DashApps.qml 2013-04-19 16:52:31 +0000
4@@ -103,9 +103,12 @@
5 model: SortFilterProxyModel {
6 model: categoryListModel
7 filterRole: 0 // 0 == modelName
8- filterRegExp: (mainStageApplicationsModel.count === 0
9- && sideStageApplicationModel.count === 0) ? RegExp("RunningApplicationsModel") : RegExp("")
10- invertMatch: true
11+ // FIXME: need to use invertMatch here, otherwise the filter won't update correctly
12+ // if filterRegExp is set before invertMatch. Bug in SortFilterProxyModel?
13+ filterRegExp: invertMatch ? ((mainStageApplicationsModel.count === 0
14+ && sideStageApplicationModel.count === 0) ? RegExp("RunningApplicationsModel") : RegExp("")) :
15+ RegExp("InstalledApplicationsModel")
16+ invertMatch: pageHeader.searchQuery.length == 0
17 }
18
19 onAtYEndChanged: if (atYEnd) endReached()

Subscribers

People subscribed via source and target branches