fix no search results shown accidentally

pull/555/head
Ryan Zhao 3 years ago
parent 9d9a3f92cb
commit aeabaea0ca

@ -2,6 +2,7 @@
import Foundation import Foundation
import UIKit import UIKit
import NVActivityIndicatorView
@objc @objc
public protocol GlobalSearchViewDelegate: AnyObject { public protocol GlobalSearchViewDelegate: AnyObject {
@ -14,6 +15,8 @@ public class GlobalSearchViewController: UITableViewController {
@objc @objc
public static let minimumSearchTextLength: Int = 2 public static let minimumSearchTextLength: Int = 2
private let maxSearchResultCount: Int = 200
@objc @objc
public weak var delegate: GlobalSearchViewDelegate? public weak var delegate: GlobalSearchViewDelegate?
@ -99,12 +102,12 @@ public class GlobalSearchViewController: UITableViewController {
refreshTimer?.invalidate() refreshTimer?.invalidate()
refreshTimer = WeakTimer.scheduledTimer(timeInterval: 0.1, target: self, userInfo: nil, repeats: false) { [weak self] _ in refreshTimer = WeakTimer.scheduledTimer(timeInterval: 0.1, target: self, userInfo: nil, repeats: false) { [weak self] _ in
guard let strongSelf = self else { guard let self = self else {
return return
} }
strongSelf.updateSearchResults(searchText: strongSelf.searchText) self.updateSearchResults(searchText: self.searchText)
strongSelf.refreshTimer = nil self.refreshTimer = nil
} }
} }
@ -123,8 +126,8 @@ public class GlobalSearchViewController: UITableViewController {
var searchResults: HomeScreenSearchResultSet? var searchResults: HomeScreenSearchResultSet?
self.dbReadConnection.asyncRead({[weak self] transaction in self.dbReadConnection.asyncRead({[weak self] transaction in
guard let strongSelf = self else { return } guard let self = self else { return }
searchResults = strongSelf.searcher.searchForHomeScreen(searchText: searchText, transaction: transaction) searchResults = self.searcher.searchForHomeScreen(searchText: searchText, maxSearchResults: self.maxSearchResultCount, transaction: transaction)
}, completionBlock: { [weak self] in }, completionBlock: { [weak self] in
AssertIsOnMainThread() AssertIsOnMainThread()
guard let self = self, let results = searchResults, self.lastSearchText == searchText else { return } guard let self = self, let results = searchResults, self.lastSearchText == searchText else { return }
@ -289,32 +292,37 @@ extension GlobalSearchViewController {
class EmptySearchResultCell: UITableViewCell { class EmptySearchResultCell: UITableViewCell {
static let reuseIdentifier = "EmptySearchResultCell" static let reuseIdentifier = "EmptySearchResultCell"
let messageLabel: UILabel private lazy var messageLabel: UILabel = {
let activityIndicator = UIActivityIndicatorView(style: .whiteLarge) let result = UILabel()
result.textAlignment = .center
result.numberOfLines = 3
result.textColor = Colors.text
result.text = NSLocalizedString("CONVERSATION_SEARCH_NO_RESULTS", comment: "")
return result
}()
private lazy var spinner: NVActivityIndicatorView = {
let result = NVActivityIndicatorView(frame: CGRect.zero, type: .circleStrokeSpin, color: Colors.text, padding: nil)
result.set(.width, to: 40)
result.set(.height, to: 40)
return result
}()
override init(style: UITableViewCell.CellStyle, reuseIdentifier: String?) { override init(style: UITableViewCell.CellStyle, reuseIdentifier: String?) {
self.messageLabel = UILabel()
super.init(style: style, reuseIdentifier: reuseIdentifier) super.init(style: style, reuseIdentifier: reuseIdentifier)
messageLabel.textAlignment = .center
messageLabel.numberOfLines = 3
contentView.addSubview(messageLabel) contentView.addSubview(messageLabel)
messageLabel.autoSetDimension(.height, toSize: 150) messageLabel.autoSetDimension(.height, toSize: 150)
messageLabel.autoPinEdge(toSuperviewMargin: .top, relation: .greaterThanOrEqual) messageLabel.autoPinEdge(toSuperviewMargin: .top, relation: .greaterThanOrEqual)
messageLabel.autoPinEdge(toSuperviewMargin: .leading, relation: .greaterThanOrEqual) messageLabel.autoPinEdge(toSuperviewMargin: .leading, relation: .greaterThanOrEqual)
messageLabel.autoPinEdge(toSuperviewMargin: .bottom, relation: .greaterThanOrEqual) messageLabel.autoPinEdge(toSuperviewMargin: .bottom, relation: .greaterThanOrEqual)
messageLabel.autoPinEdge(toSuperviewMargin: .trailing, relation: .greaterThanOrEqual) messageLabel.autoPinEdge(toSuperviewMargin: .trailing, relation: .greaterThanOrEqual)
messageLabel.autoVCenterInSuperview() messageLabel.autoVCenterInSuperview()
messageLabel.autoHCenterInSuperview() messageLabel.autoHCenterInSuperview()
messageLabel.setContentHuggingHigh() messageLabel.setContentHuggingHigh()
messageLabel.setCompressionResistanceHigh() messageLabel.setCompressionResistanceHigh()
contentView.addSubview(activityIndicator) contentView.addSubview(spinner)
activityIndicator.autoCenterInSuperview() spinner.autoCenterInSuperview()
} }
required init?(coder aDecoder: NSCoder) { required init?(coder aDecoder: NSCoder) {
@ -322,18 +330,13 @@ class EmptySearchResultCell: UITableViewCell {
} }
public func configure(searchText: String) { public func configure(searchText: String) {
if searchText.isEmpty { if searchText.count < GlobalSearchViewController.minimumSearchTextLength {
activityIndicator.color = Colors.text spinner.stopAnimating()
activityIndicator.isHidden = false spinner.startAnimating()
activityIndicator.startAnimating()
messageLabel.isHidden = true messageLabel.isHidden = true
messageLabel.text = nil
} else { } else {
activityIndicator.stopAnimating() spinner.stopAnimating()
activityIndicator.isHidden = true
messageLabel.isHidden = false messageLabel.isHidden = false
messageLabel.text = NSLocalizedString("CONVERSATION_SEARCH_NO_RESULTS", comment: "")
messageLabel.textColor = Colors.text
} }
} }
} }

Loading…
Cancel
Save