Merge lp:~mzanetti/reminders-app/improve-section-headers into lp:reminders-app

[Michael Zanetti]

Improve section headers when sorting

Previously section headers were always built on dateCreated. Now they adjust depending on the selected sorting mechanism.

[Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot)]

PASSED: Continuous integration, rev:358
http://91.189.93.70:8080/job/reminders-app-ci/685/
Executed test runs:
    SUCCESS: http://91.189.93.70:8080/job/generic-mediumtests-utopic/2181
        deb: http://91.189.93.70:8080/job/generic-mediumtests-utopic/2181/artifact/work/output/*zip*/output.zip
    SUCCESS: http://91.189.93.70:8080/job/reminders-app-utopic-amd64-ci/287
    SUCCESS: http://91.189.93.70:8080/job/reminders-app-vivid-amd64-ci/106

Click here to trigger a rebuild:
http://91.189.93.70:8080/job/reminders-app-ci/685/rebuild

review: Approve (continuous-integration)

[Riccardo Padovani (rpadovani)]

lgtm

review: Approve

=== modified file 'src/app/qml/ui/NotesPage.qml'
--- src/app/qml/ui/NotesPage.qml	2015-02-25 22:48:18 +0000
+++ src/app/qml/ui/NotesPage.qml	2015-02-28 02:15:48 +0000
@@ -208,8 +208,31 @@
             }
         }
 
-        section.criteria: ViewSection.FullString
-        section.property: "createdString"
+        section.criteria: {
+            switch (notes.sortOrder) {
+            case Notes.SortOrderTitleAscending:
+            case Notes.SortOrderTitleDescending:
+                return ViewSection.FirstCharacter;
+            }
+
+            return ViewSection.FullString
+        }
+        section.property: {
+            switch(notes.sortOrder) {
+            case Notes.SortOrderDateCreatedNewest:
+            case Notes.SortOrderDateCreatedOldest:
+                return "createdString";
+            case Notes.SortOrderDateUpdatedNewest:
+            case Notes.SortOrderDateUpdatedOldest:
+                return "updatedString";
+            case Notes.SortOrderTitleAscending:
+            case Notes.SortOrderTitleDescending:
+                return "title";
+            }
+            return "";
+        }
+
+
         section.delegate: Empty {
             height: units.gu(5)
             showDivider: false