Merge lp:~mzanetti/reminders-app/fix-filter-init into lp:reminders-app

Proposed by Michael Zanetti
Status: Merged
Approved by: Riccardo Padovani
Approved revision: 353
Merged at revision: 357
Proposed branch: lp:~mzanetti/reminders-app/fix-filter-init
Merge into: lp:reminders-app
Prerequisite: lp:~mzanetti/reminders-app/fix-save-unloaded
Diff against target: 29 lines (+3/-2)
2 files modified
src/app/qml/ui/NotesPage.qml (+2/-2)
src/libqtevernote/notes.cpp (+1/-0)
To merge this branch: bzr merge lp:~mzanetti/reminders-app/fix-filter-init
Reviewer Review Type Date Requested Status
Riccardo Padovani Approve
Ubuntu Phone Apps Jenkins Bot continuous-integration Approve
Review via email: mp+251017@code.launchpad.net

Commit message

small fixes

* fixes an issue where notebooks would be empty sometimes
* drop the old error label that's sometimes painted on top of other text

To post a comment you must log in.
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
Riccardo Padovani (rpadovani) wrote :

Why did you add m_onlySearchResults?

review: Needs Information
Revision history for this message
Michael Zanetti (mzanetti) wrote :

for
* fixes an issue where notebooks would be empty sometimes

(see inline explanation)

Revision history for this message
Riccardo Padovani (rpadovani) wrote :

Thanks for the explanation :-)

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/app/qml/ui/NotesPage.qml'
2--- src/app/qml/ui/NotesPage.qml 2015-02-23 11:31:34 +0000
3+++ src/app/qml/ui/NotesPage.qml 2015-02-25 23:19:24 +0000
4@@ -231,11 +231,11 @@
5 }
6 Label {
7 anchors.centerIn: parent
8- visible: !notes.loading && (notes.error || notesListView.count == 0)
9+ visible: !notes.loading && notesListView.count == 0
10 width: parent.width - units.gu(4)
11 wrapMode: Text.WordWrap
12 horizontalAlignment: Text.AlignHCenter
13- text: notes.error ? notes.error : i18n.tr("No notes available. You can create new notes using the \"Add note\" button.")
14+ text: i18n.tr("No notes available. You can create new notes using the \"Add note\" button.")
15 }
16
17 Scrollbar {
18
19=== modified file 'src/libqtevernote/notes.cpp'
20--- src/libqtevernote/notes.cpp 2015-02-17 21:35:50 +0000
21+++ src/libqtevernote/notes.cpp 2015-02-25 23:19:24 +0000
22@@ -26,6 +26,7 @@
23 Notes::Notes(QObject *parent) :
24 QSortFilterProxyModel(parent),
25 m_onlyReminders(false),
26+ m_onlySearchResults(false),
27 m_showDeleted(false),
28 m_sortOrder(SortOrderDateCreatedNewest)
29 {

Subscribers

People subscribed via source and target branches