comment cleanup

// FREEBIE
pull/1/head
Michael Kirk 7 years ago
parent 4aeff7ba67
commit f733c07d07

@ -528,8 +528,6 @@ class MediaGalleryViewController: UINavigationController, MediaGalleryDataSource
func ensureGalleryItemsLoaded(_ direction: GalleryDirection, item: MediaGalleryItem, amount: UInt, completion: ((IndexSet, [IndexPath]) -> Void)? = nil ) {
// TODO avoid copy?
// TODO read off main thread?
var galleryItems: [MediaGalleryItem] = self.galleryItems
var sections: [GalleryDate: [MediaGalleryItem]] = self.sections
var sectionDates: [GalleryDate] = self.sectionDates
@ -656,7 +654,6 @@ class MediaGalleryViewController: UINavigationController, MediaGalleryDataSource
let kGallerySwipeLoadBatchSize: UInt = 5
// TODO extract to public extension?
internal func galleryItem(after currentItem: MediaGalleryItem) -> MediaGalleryItem? {
Logger.debug("\(logTag) in \(#function)")

@ -4,7 +4,6 @@
import UIKit
// TODO Can we make this private to MediaPageViewController?
public struct MediaGalleryPage: Equatable {
public let viewController: MediaDetailViewController
@ -23,7 +22,7 @@ public struct MediaGalleryPage: Equatable {
}
public var image: UIImage {
// TODO cache this?
// TODO cache this
return galleryItem.fullSizedImage
}
@ -41,12 +40,10 @@ class MediaPageViewController: UIPageViewController, UIPageViewControllerDataSou
private var cachedPages: [MediaGalleryPage] = []
private var initialPage: MediaGalleryPage!
// FIXME can this be private?
public var currentPage: MediaGalleryPage! {
return cachedPages.first { $0.viewController == viewControllers?.first }
}
// FIXME can this be private?
public var currentItem: MediaGalleryItem! {
get {
return currentPage.galleryItem
@ -63,7 +60,6 @@ class MediaPageViewController: UIPageViewController, UIPageViewControllerDataSou
}
}
// TODO remove?
private let uiDatabaseConnection: YapDatabaseConnection
private let includeGallery: Bool

@ -331,7 +331,6 @@ NS_ASSUME_NONNULL_BEGIN
}
[mainSection addItem:[OWSTableItem itemWithCustomCellBlock:^{
// FIXME proper icon
return [weakSelf disclosureCellWithName:MediaStrings.allMedia iconName:@"actionsheet_camera_roll_black"];
}
actionBlock:^{

Loading…
Cancel
Save