Compare commits

..

No commits in common. "6f51f321f626e4931431584a56649c915339c962" and "765b5e1a7cfe2bbd67d9795157cdd3287c30bbb3" have entirely different histories.

3 changed files with 3 additions and 11 deletions

View File

@ -80,8 +80,9 @@ class SearchResultsViewController: UIViewController, CollectionViewController {
break break
} }
let section = NSCollectionLayoutSection.list(using: config, layoutEnvironment: environment) let section = NSCollectionLayoutSection.list(using: config, layoutEnvironment: environment)
// we don't use the readable content inset here, because it insets the entire cell, rather than just the content if UIDevice.current.userInterfaceIdiom == .pad || UIDevice.current.userInterfaceIdiom == .mac {
// so the cell backgrounds not being full width looks weird section.contentInsetsReference = .readableContent
}
return section return section
} }
view = UICollectionView(frame: .zero, collectionViewLayout: layout) view = UICollectionView(frame: .zero, collectionViewLayout: layout)

View File

@ -304,12 +304,6 @@ private class SplitSecondaryNavigationController: EnhancedNavigationViewControll
} }
} }
override func pushViewController(_ viewController: UIViewController, animated: Bool) {
super.pushViewController(viewController, animated: animated)
configureSecondarySplitCloseButton(for: viewControllers.first!)
}
private func configureSecondarySplitCloseButton(for viewController: UIViewController) { private func configureSecondarySplitCloseButton(for viewController: UIViewController) {
guard viewController.navigationItem.leftBarButtonItem?.tag != ViewTags.splitNavCloseSecondaryButton else { guard viewController.navigationItem.leftBarButtonItem?.tag != ViewTags.splitNavCloseSecondaryButton else {
return return

View File

@ -48,9 +48,6 @@ class TrendHistoryView: UIView {
} }
let maxUses = history.max(by: { $0.uses < $1.uses })!.uses let maxUses = history.max(by: { $0.uses < $1.uses })!.uses
guard maxUses > 0 else {
return
}
// remove old layers if this view is being re-used // remove old layers if this view is being re-used
layer.sublayers?.forEach { $0.removeFromSuperlayer() } layer.sublayers?.forEach { $0.removeFromSuperlayer() }