Handle no results

In line with other messaging apps, we intentionally don't show a "No Results"
cell. We simply don't display any cells. Though we could easily modify this in
the future.

// FREEBIE

Signal/src/ViewControllers/HomeView/ConversationSearchViewController.swift

@@ -19,9 +19,10 @@ class ConversationSearchViewController: UITableViewController {
     }
 
     enum SearchSection: Int {
-        case conversations = 0
+        case noResults
-        case contacts = 1
+        case conversations
-        case messages = 2
+        case contacts
+        case messages
     }
 
     // MARK: View Lifecyle
@@ -48,6 +49,8 @@ class ConversationSearchViewController: UITableViewController {
         }
 
         switch searchSection {
+        case .noResults:
+            owsFail("\(logTag) shouldn't be able to tap 'no results' section")
         case .conversations:
             let sectionResults = searchResultSet.conversations
             guard let searchResult = sectionResults[safe: indexPath.row] else {
@@ -88,6 +91,9 @@ class ConversationSearchViewController: UITableViewController {
         }
 
         switch searchSection {
+        case .noResults:
+            // We don't display a "no results" cell, we simply display no results.
+            return 0
         case .conversations:
             return searchResultSet.conversations.count
         case .contacts:
@@ -104,6 +110,8 @@ class ConversationSearchViewController: UITableViewController {
         }
 
         switch searchSection {
+        case .noResults:
+            return UITableViewCell()
         case .conversations:
             guard let cell = tableView.dequeueReusableCell(withIdentifier: ConversationSearchResultCell.reuseIdentifier) as? ConversationSearchResultCell else {
                 return UITableViewCell()
@@ -150,6 +158,8 @@ class ConversationSearchViewController: UITableViewController {
         }
 
         switch searchSection {
+        case .noResults:
+            return nil
         case .conversations:
             if searchResultSet.conversations.count > 0 {
                 return NSLocalizedString("SEARCH_SECTION_CONVERSATIONS", comment: "section header for search results that match existing conversations (either group or contact conversations)")

Signal/translations/en.lproj/Localizable.strings

@@ -986,9 +986,6 @@
 /* A label for conversations with blocked users. */
 "HOME_VIEW_BLOCKED_CONTACT_CONVERSATION" = "Blocked";
 
-/* Placeholder text for search bar which filters conversations. */
-"HOME_VIEW_CONVERSATION_SEARCHBAR_PLACEHOLDER" = "Search";
-
 /* Title for the home view's 'archive' mode. */
 "HOME_VIEW_TITLE_ARCHIVE" = "Archive";
 

SignalMessaging/utils/ConversationSearcher.swift

@@ -40,6 +40,10 @@ public class SearchResultSet {
     public class var empty: SearchResultSet {
         return SearchResultSet(conversations: [], contacts: [], messages: [])
     }
+
+    public var isEmpty: Bool {
+        return conversations.isEmpty && contacts.isEmpty && messages.isEmpty
+    }
 }
 
 @objc