Skip to content

Commit 590ce54

Browse files
committed
Removing unnecessary comments
1 parent ee2e648 commit 590ce54

File tree

1 file changed

+5
-20
lines changed

1 file changed

+5
-20
lines changed

App/View Controllers/Posts/PostsPageViewController.swift

Lines changed: 5 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -398,10 +398,8 @@ final class PostsPageViewController: ViewController {
398398
.nextUnread where self.posts.isEmpty:
399399
let pageCount = self.numberOfPages > 0 ? "\(self.numberOfPages)" : "?"
400400
if #available(iOS 26.0, *) {
401-
// Use vertical view: show unknown current page with known total
402-
self.pageNumberView.currentPage = 0 // Will display as "?"
401+
self.pageNumberView.currentPage = 0
403402
self.pageNumberView.totalPages = self.numberOfPages > 0 ? self.numberOfPages : 0
404-
// iOS 26+ handles colors automatically
405403
} else {
406404
self.currentPageItem.title = "Page ? of \(pageCount)"
407405
}
@@ -495,7 +493,6 @@ final class PostsPageViewController: ViewController {
495493
item.accessibilityLabel = NSLocalizedString("compose.accessibility-label", comment: "")
496494
// Only set explicit tint color for iOS < 26
497495
if #available(iOS 26.0, *) {
498-
// Let iOS 26+ handle the color automatically
499496
} else {
500497
item.tintColor = theme["navigationBarTextColor"]
501498
}
@@ -592,7 +589,6 @@ final class PostsPageViewController: ViewController {
592589
item.accessibilityLabel = "Settings"
593590
// Only set explicit tint color for iOS < 26
594591
if #available(iOS 26.0, *) {
595-
// Let iOS 26+ handle the color automatically
596592
} else {
597593
item.tintColor = theme["toolbarTextColor"]
598594
}
@@ -613,7 +609,6 @@ final class PostsPageViewController: ViewController {
613609
item.accessibilityLabel = "Previous page"
614610
// Only set explicit tint color for iOS < 26
615611
if #available(iOS 26.0, *) {
616-
// Let iOS 26+ handle the color automatically
617612
} else {
618613
item.tintColor = theme["toolbarTextColor"]
619614
}
@@ -638,8 +633,7 @@ final class PostsPageViewController: ViewController {
638633
popover.barButtonItem = action.sender as? UIBarButtonItem
639634
}
640635
})
641-
642-
// Set up the bar button item based on iOS version
636+
643637
if #available(iOS 26.0, *) {
644638
let containerView = UIView()
645639
containerView.addSubview(pageNumberView)
@@ -652,7 +646,6 @@ final class PostsPageViewController: ViewController {
652646
])
653647
item.customView = containerView
654648
} else {
655-
// Use traditional text title for iOS 18 and below
656649
item.possibleTitles = ["2345 / 2345"]
657650
}
658651

@@ -674,7 +667,6 @@ final class PostsPageViewController: ViewController {
674667
item.accessibilityLabel = "Next page"
675668
// Only set explicit tint color for iOS < 26
676669
if #available(iOS 26.0, *) {
677-
// Let iOS 26+ handle the color automatically
678670
} else {
679671
item.tintColor = theme["toolbarTextColor"]
680672
}
@@ -706,7 +698,6 @@ final class PostsPageViewController: ViewController {
706698
item.accessibilityLabel = "Thread actions"
707699
// Only set explicit tint color for iOS < 26
708700
if #available(iOS 26.0, *) {
709-
// Let iOS 26+ handle the color automatically
710701
} else {
711702
item.tintColor = theme["toolbarTextColor"]
712703
}
@@ -792,7 +783,6 @@ final class PostsPageViewController: ViewController {
792783
currentPageItem.setTitleTextAttributes([.font: UIFont.preferredFontForTextStyle(.body, weight: .regular)], for: .normal)
793784
}
794785
} else {
795-
// Clear page display
796786
if #available(iOS 26.0, *) {
797787
pageNumberView.currentPage = 0
798788
pageNumberView.totalPages = 0
@@ -870,8 +860,7 @@ final class PostsPageViewController: ViewController {
870860
guard postsView.loadingView == nil else { return }
871861
let selectotron = Selectotron(postsViewController: self)
872862
present(selectotron, animated: true)
873-
874-
// For popover presentation with custom view, we need to set sourceView and sourceRect
863+
875864
if let popover = selectotron.popoverPresentationController {
876865
popover.sourceView = pageNumberView
877866
popover.sourceRect = pageNumberView.bounds
@@ -1687,7 +1676,6 @@ final class PostsPageViewController: ViewController {
16871676

16881677
// Update navigation bar button colors (only for iOS < 26)
16891678
if #available(iOS 26.0, *) {
1690-
// Let iOS 26+ handle colors automatically
16911679
} else {
16921680
composeItem.tintColor = theme["navigationBarTextColor"]
16931681
// Ensure the navigation bar itself uses the correct tint color for the back button
@@ -1718,18 +1706,15 @@ final class PostsPageViewController: ViewController {
17181706
postsView.toolbar.compactAppearance = appearance
17191707
postsView.toolbar.scrollEdgeAppearance = appearance
17201708
postsView.toolbar.compactScrollEdgeAppearance = appearance
1721-
1722-
// Update toolbar button text colors (only for iOS < 26)
1709+
17231710
if #available(iOS 26.0, *) {
1724-
// Let iOS 26+ handle colors automatically
17251711
} else {
17261712
backItem.tintColor = theme["toolbarTextColor"]
17271713
forwardItem.tintColor = theme["toolbarTextColor"]
17281714
settingsItem.tintColor = theme["toolbarTextColor"]
17291715
pageNumberView.textColor = theme["toolbarTextColor"] ?? UIColor.systemBlue
17301716
}
1731-
1732-
// Update toolbar items to refresh the actions button
1717+
17331718
updateToolbarItems()
17341719

17351720
messageViewController?.themeDidChange()

0 commit comments

Comments
 (0)