Respond to CR.

Signal/src/ViewControllers/ConversationView/Cells/OWSContactShareView.m

@@ -235,7 +235,7 @@ NS_ASSUME_NONNULL_BEGIN
     [stackView autoPinTrailingToSuperviewMarginWithInset:self.iconHMargin];
     [stackView autoVCenterInSuperview];
     // Ensure that the cell's contents never overflow the cell bounds.
-    // We pin pin to the superview _edge_ and not _margin_ for the purposes
+    // We pin to the superview _edge_ and not _margin_ for the purposes
     // of overflow, so that changes to the margins do not trip these safe guards.
     [stackView autoPinEdgeToSuperviewEdge:ALEdgeTop withInset:0 relation:NSLayoutRelationGreaterThanOrEqual];
     [stackView autoPinEdgeToSuperviewEdge:ALEdgeBottom withInset:0 relation:NSLayoutRelationGreaterThanOrEqual];

Signal/src/ViewControllers/HomeView/ConversationSearchViewController.swift

@@ -170,10 +170,10 @@ class ConversationSearchViewController: UITableViewController {
             }
 
             var overrideSnippet = NSAttributedString()
-            var overrideTimestamp: NSNumber?
+            var overrideDate: Date?
-            if let messageId = searchResult.messageId {
+            if searchResult.messageId != nil {
-                if let messageTimestamp = searchResult.messageTimestamp {
+                if let messageDate = searchResult.messageDate {
-                    overrideTimestamp = NSNumber(value: messageTimestamp)
+                    overrideDate = messageDate
                 } else {
                     owsFail("\(ConversationSearchViewController.logTag) message search result is missing message timestamp")
                 }
@@ -183,14 +183,7 @@ class ConversationSearchViewController: UITableViewController {
                 // a snippet for conversations that reflects the latest
                 // contents.
                 if let messageSnippet = searchResult.snippet {
-                    // YDB uses bold tags to highlight matches within the snippet.
+                    overrideSnippet = NSAttributedString(string: messageSnippet)
-                    let filteredSnippet = messageSnippet
-                        .replacingOccurrences(of: "<b>", with: "")
-                        .replacingOccurrences(of: "</b>", with: "")
-                        .replacingOccurrences(of: "<B>", with: "")
-                        .replacingOccurrences(of: "</B>", with: "")
-
-                    overrideSnippet = NSAttributedString(string: filteredSnippet)
                 } else {
                     owsFail("\(ConversationSearchViewController.logTag) message search result is missing message snippet")
                 }
@@ -200,7 +193,7 @@ class ConversationSearchViewController: UITableViewController {
                            contactsManager: contactsManager,
                            blockedPhoneNumber: self.blockedPhoneNumberSet,
                            overrideSnippet: overrideSnippet,
-                overrideTimestamp: overrideTimestamp)
+                           overrideDate: overrideDate)
 
             return cell
         }

Signal/src/ViewControllers/HomeView/HomeViewCell.h

@@ -20,7 +20,7 @@ NS_ASSUME_NONNULL_BEGIN
             contactsManager:(OWSContactsManager *)contactsManager
       blockedPhoneNumberSet:(NSSet<NSString *> *)blockedPhoneNumberSet
             overrideSnippet:(nullable NSAttributedString *)overrideSnippet
-          overrideTimestamp:(nullable NSNumber *)overrideTimestamp;
+               overrideDate:(nullable NSDate *)overrideDate;
 
 @end
 

Signal/src/ViewControllers/HomeView/HomeViewCell.m

@@ -93,10 +93,14 @@ NS_ASSUME_NONNULL_BEGIN
     [self.payloadView autoPinLeadingToTrailingEdgeOfView:self.avatarView offset:self.avatarHSpacing];
     [self.payloadView autoVCenterInSuperview];
     // Ensure that the cell's contents never overflow the cell bounds.
-    // We pin pin to the superview _edge_ and not _margin_ for the purposes
+    // We pin to the superview _edge_ and not _margin_ for the purposes
     // of overflow, so that changes to the margins do not trip these safe guards.
-    [self.payloadView autoPinEdgeToSuperviewEdge:ALEdgeTop withInset:0 relation:NSLayoutRelationGreaterThanOrEqual];
+    [self.payloadView autoPinEdgeToSuperviewEdge:ALEdgeTop
-    [self.payloadView autoPinEdgeToSuperviewEdge:ALEdgeBottom withInset:0 relation:NSLayoutRelationGreaterThanOrEqual];
+                                       withInset:kMinVMargin
+                                        relation:NSLayoutRelationGreaterThanOrEqual];
+    [self.payloadView autoPinEdgeToSuperviewEdge:ALEdgeBottom
+                                       withInset:kMinVMargin
+                                        relation:NSLayoutRelationGreaterThanOrEqual];
     // We pin the payloadView traillingEdge later, as part of the "Unread Badge" logic.
 
     self.nameLabel = [UILabel new];
@@ -136,16 +140,6 @@ NS_ASSUME_NONNULL_BEGIN
     [self.unreadLabel autoCenterInSuperview];
     [self.unreadLabel setContentHuggingHigh];
     [self.unreadLabel setCompressionResistanceHigh];
-
-    // Ensure that the cell's contents never overflow the cell bounds.
-    // We pin pin to the superview _edge_ and not _margin_ for the purposes
-    // of overflow, so that changes to the margins do not trip these safe guards.
-    [self.payloadView autoPinEdgeToSuperviewEdge:ALEdgeTop
-                                       withInset:kMinVMargin
-                                        relation:NSLayoutRelationGreaterThanOrEqual];
-    [self.payloadView autoPinEdgeToSuperviewEdge:ALEdgeBottom
-                                       withInset:kMinVMargin
-                                        relation:NSLayoutRelationGreaterThanOrEqual];
 }
 
 + (NSString *)cellReuseIdentifier
@@ -171,14 +165,14 @@ NS_ASSUME_NONNULL_BEGIN
               contactsManager:contactsManager
         blockedPhoneNumberSet:blockedPhoneNumberSet
               overrideSnippet:nil
-            overrideTimestamp:nil];
+                 overrideDate:nil];
 }
 
 - (void)configureWithThread:(ThreadViewModel *)thread
             contactsManager:(OWSContactsManager *)contactsManager
       blockedPhoneNumberSet:(NSSet<NSString *> *)blockedPhoneNumberSet
             overrideSnippet:(nullable NSAttributedString *)overrideSnippet
-          overrideTimestamp:(nullable NSNumber *)overrideTimestamp
+               overrideDate:(nullable NSDate *)overrideDate
 {
     OWSAssertIsOnMainThread();
     OWSAssert(thread);
@@ -213,9 +207,8 @@ NS_ASSUME_NONNULL_BEGIN
     // override any font attributes.
     self.snippetLabel.font = [self snippetFont];
 
-    self.dateTimeLabel.text = (overrideTimestamp
+    self.dateTimeLabel.text
-            ? [self stringForDate:[NSDate ows_dateWithMillisecondsSince1970:overrideTimestamp.unsignedLongLongValue]]
+        = (overrideDate ? [self stringForDate:overrideDate] : [self stringForDate:thread.lastMessageDate]);
-            : [self stringForDate:thread.lastMessageDate]);
 
     if (hasUnreadMessages) {
         self.dateTimeLabel.textColor = [UIColor ows_blackColor];

Signal/src/ViewControllers/HomeView/HomeViewController.m

@@ -789,7 +789,7 @@ NSString *const kArchivedConversationsReuseIdentifier = @"kArchivedConversations
     [stackView autoPinEdgeToSuperviewMargin:ALEdgeLeading relation:NSLayoutRelationGreaterThanOrEqual];
     [stackView autoPinEdgeToSuperviewMargin:ALEdgeTrailing relation:NSLayoutRelationGreaterThanOrEqual];
     // Ensure that the cell's contents never overflow the cell bounds.
-    // We pin pin to the superview _edge_ and not _margin_ for the purposes
+    // We pin to the superview _edge_ and not _margin_ for the purposes
     // of overflow, so that changes to the margins do not trip these safe guards.
     [stackView autoPinEdgeToSuperviewEdge:ALEdgeTop withInset:0 relation:NSLayoutRelationGreaterThanOrEqual];
     [stackView autoPinEdgeToSuperviewEdge:ALEdgeBottom withInset:0 relation:NSLayoutRelationGreaterThanOrEqual];

SignalMessaging/Views/ContactTableViewCell.m

@@ -109,7 +109,7 @@ const CGFloat kContactTableViewCellAvatarTextMargin = 12;
     [_nameContainerView autoPinTrailingToSuperviewMargin];
 
     // Ensure that the cell's contents never overflow the cell bounds.
-    // We pin pin to the superview _edge_ and not _margin_ for the purposes
+    // We pin to the superview _edge_ and not _margin_ for the purposes
     // of overflow, so that changes to the margins do not trip these safe guards.
     [_avatarView autoPinEdgeToSuperviewEdge:ALEdgeTop
                                   withInset:kMinVMargin

SignalMessaging/utils/ConversationSearcher.swift

@@ -9,17 +9,17 @@ public class ConversationSearchResult: Comparable {
     public let thread: ThreadViewModel
 
     public let messageId: String?
-    public let messageTimestamp: UInt64?
+    public let messageDate: Date?
 
     public let snippet: String?
 
     private let sortKey: UInt64
 
-    init(thread: ThreadViewModel, sortKey: UInt64, messageId: String? = nil, messageTimestamp: UInt64? = nil, snippet: String? = nil) {
+    init(thread: ThreadViewModel, sortKey: UInt64, messageId: String? = nil, messageDate: Date? = nil, snippet: String? = nil) {
         self.thread = thread
         self.sortKey = sortKey
         self.messageId = messageId
-        self.messageTimestamp = messageTimestamp
+        self.messageDate = messageDate
         self.snippet = snippet
     }
 
@@ -128,7 +128,7 @@ public class ConversationSearcher: NSObject {
                 let searchResult = ConversationSearchResult(thread: threadViewModel,
                                                             sortKey: sortKey,
                                                             messageId: message.uniqueId,
-                                                            messageTimestamp: message.timestamp,
+                                                            messageDate: NSDate.ows_date(withMillisecondsSince1970: message.timestamp),
                                                             snippet: snippet)
 
                 messages.append(searchResult)

SignalServiceKit/src/Storage/FullTextSearchFinder.swift

@@ -50,8 +50,10 @@ public class FullTextSearchFinder: NSObject {
 
         let maxSearchResults = 500
         var searchResultCount = 0
-        // (snippet: String, collection: String, key: String, object: Any, stop: UnsafeMutablePointer<ObjCBool>)
+        let snippetOptions = YapDatabaseFullTextSearchSnippetOptions()
-        ext.enumerateKeysAndObjects(matching: prefixQuery, with: nil) { (snippet: String, _: String, _: String, object: Any, stop: UnsafeMutablePointer<ObjCBool>) in
+        snippetOptions.startMatchText = ""
+        snippetOptions.endMatchText = ""
+        ext.enumerateKeysAndObjects(matching: prefixQuery, with: snippetOptions) { (snippet: String, _: String, _: String, object: Any, stop: UnsafeMutablePointer<ObjCBool>) in
             guard searchResultCount < maxSearchResults else {
                 stop.pointee = true
                 return