Add an action to the newly introduced FloatingDateBadge that scrolls the timeline to that day. (#5350)

and replace the TimelnieTableViewController's `floatingDateText` with a pure date.
This commit is contained in:
Stefan Ceriu
2026-04-14 11:27:27 +03:00
committed by GitHub
parent 9082a9efed
commit 03fbf8fbd5
11 changed files with 73 additions and 37 deletions

View File

@@ -110,7 +110,9 @@ struct RoomScreen: View {
@ViewBuilder
private var dateBadge: some View {
if !isVoiceOverEnabled {
FloatingDateBadge(dateText: timelineContext.floatingDateText)
FloatingDateBadge(dateText: timelineContext.floatingDate?.formattedDateSeparator()) {
timelineContext.send(viewAction: .scrollToFirstItemForCurrentDate)
}
}
}

View File

@@ -32,8 +32,10 @@ struct ThreadTimelineScreen: View {
.toolbarBackground(.visible, for: .navigationBar) // Fix the toolbar's background.
.timelineMediaPreview(viewModel: $context.mediaPreviewViewModel)
.overlay(alignment: .top) {
FloatingDateBadge(dateText: timelineContext.floatingDateText)
.padding(.top, 13)
FloatingDateBadge(dateText: timelineContext.floatingDate?.formattedDateSeparator()) {
timelineContext.send(viewAction: .scrollToFirstItemForCurrentDate)
}
.padding(.top, 13)
}
.overlay(alignment: .bottomTrailing) {
TimelineScrollToBottomButton(isVisible: isAtBottomAndLive) {

View File

@@ -56,6 +56,7 @@ enum TimelineViewAction {
case paginateBackwards
case paginateForwards
case scrollToBottom
case scrollToFirstItemForCurrentDate
case displayTimelineItemMenu(itemID: TimelineItemIdentifier)
case handleTimelineItemMenuAction(itemID: TimelineItemIdentifier, action: TimelineItemMenuAction)
@@ -152,8 +153,8 @@ struct TimelineViewState: BindableState {
struct TimelineViewStateBindings {
var isScrolledToBottom = true
/// The formatted date text for the floating date badge shown while scrolling.
var floatingDateText: String?
/// The timestamp of the topmost visible item, used to drive the floating date badge while scrolling.
var floatingDate: Date?
/// The state of wether reactions listed on the timeline are expanded/collapsed.
/// Key is itemID, value is the collapsed state.
@@ -247,6 +248,7 @@ struct TimelineState {
/// These can be removed when we have full swiftUI and moved as @State values in the view
var scrollToBottomPublisher = PassthroughSubject<Void, Never>()
var scrollToFirstItemForDatePublisher = PassthroughSubject<Void, Never>()
var itemsDictionary = OrderedDictionary<TimelineItemIdentifier.UniqueID, RoomTimelineItemViewState>()

View File

@@ -145,7 +145,7 @@ class TimelineTableViewController: UIViewController {
}
@Binding private var isScrolledToBottom: Bool
@Binding private var floatingDateText: String?
@Binding private var floatingDate: Date?
/// A work item used to auto-hide the floating date badge after scrolling stops.
private var floatingDateHideWorkItem: DispatchWorkItem?
@@ -179,11 +179,12 @@ class TimelineTableViewController: UIViewController {
init(coordinator: TimelineViewRepresentable.Coordinator,
isScrolledToBottom: Binding<Bool>,
floatingDateText: Binding<String?>,
scrollToBottomPublisher: PassthroughSubject<Void, Never>) {
floatingDate: Binding<Date?>,
scrollToBottomPublisher: PassthroughSubject<Void, Never>,
scrollToFirstItemForDatePublisher: PassthroughSubject<Void, Never>) {
self.coordinator = coordinator
_isScrolledToBottom = isScrolledToBottom
_floatingDateText = floatingDateText
_floatingDate = floatingDate
super.init(nibName: nil, bundle: nil)
@@ -209,6 +210,12 @@ class TimelineTableViewController: UIViewController {
}
.store(in: &cancellables)
scrollToFirstItemForDatePublisher
.sink { [weak self] _ in
self?.scrollToFirstItemForCurrentDate()
}
.store(in: &cancellables)
paginatePublisher
.collect(.byTime(DispatchQueue.main, 0.1))
.sink { [weak self] _ in
@@ -518,10 +525,10 @@ extension TimelineTableViewController: UITableViewDelegate {
// MARK: - Floating Date Badge
extension TimelineTableViewController {
/// Computes the formatted date text for the topmost visible timeline item
/// Computes the timestamp for the topmost visible timeline item
/// and updates the floating date binding.
func updateFloatingDate() {
guard let dateText = newestVisibleDateText() else {
guard let date = newestVisibleDate() else {
return
}
@@ -530,9 +537,9 @@ extension TimelineTableViewController {
// to extend the display duration of the floating date.
scheduleFloatingDateHide()
// Only update on changes to avoid needless SwiftUI recomputation.
if floatingDateText != dateText {
floatingDateText = dateText
// Only update when the calendar day changes to avoid needless SwiftUI recomputation.
if floatingDate.map({ !Calendar.current.isDate($0, inSameDayAs: date) }) ?? true {
floatingDate = date
}
}
@@ -540,18 +547,33 @@ extension TimelineTableViewController {
func scheduleFloatingDateHide() {
floatingDateHideWorkItem?.cancel()
let workItem = DispatchWorkItem { [weak self] in
self?.floatingDateText = nil
self?.floatingDate = nil
}
floatingDateHideWorkItem = workItem
DispatchQueue.main.asyncAfter(deadline: .now() + 2.0, execute: workItem)
}
/// Returns the formatted date text for the newest visible timeline item.
/// Scrolls to the first (oldest) item on the same calendar day as the current floating date.
private func scrollToFirstItemForCurrentDate() {
guard let floatingDate else { return }
// timelineItemsDictionary is ordered oldest-first; the first match is the earliest item for that day.
for uniqueID in timelineItemsDictionary.keys {
if let timestamp = timelineItemsDictionary[uniqueID]?.timestamp,
Calendar.current.isDate(timestamp, inSameDayAs: floatingDate),
let indexPath = dataSource?.indexPath(for: uniqueID) {
// The table view is flipped, so .bottom aligns the cell to the visual top.
tableView.scrollToRow(at: indexPath, at: .bottom, animated: true)
return
}
}
}
/// Returns the timestamp of the newest visible timeline item.
///
/// The table view is flipped (unless VoiceOver is running), so the "newest"
/// visible cell on screen is actually the *last* index path in `indexPathsForVisibleRows`
/// when the table is flipped, or the *first* when it is not.
private func newestVisibleDateText() -> String? {
private func newestVisibleDate() -> Date? {
guard let visibleIndexPaths = tableView.indexPathsForVisibleRows,
!visibleIndexPaths.isEmpty else {
return nil
@@ -562,11 +584,11 @@ extension TimelineTableViewController {
let isFlipped = scaleY == -1
let orderedPaths = isFlipped ? visibleIndexPaths.reversed() : visibleIndexPaths
// Walk from topmost downward and return the date of the first item that has one.
// Walk from topmost downward and return the timestamp of the first item that has one.
for indexPath in orderedPaths {
if let uniqueID = dataSource?.itemIdentifier(for: indexPath),
let timestamp = timelineItemsDictionary[uniqueID]?.timestamp {
return timestamp.formattedDateSeparator()
return timestamp
}
}

View File

@@ -178,6 +178,8 @@ class TimelineViewModel: TimelineViewModelType, TimelineViewModelProtocol {
paginateForwards()
case .scrollToBottom:
scrollToBottom()
case .scrollToFirstItemForCurrentDate:
state.timelineState.scrollToFirstItemForDatePublisher.send()
case .displayTimelineItemMenu(let itemID):
timelineInteractionHandler.displayTimelineItemActionMenu(for: itemID)
case .handleTimelineItemMenuAction(let itemID, let action):

View File

@@ -13,6 +13,7 @@ import SwiftUI
/// fades out shortly after scrolling stops.
struct FloatingDateBadge: View {
let dateText: String?
var onTap: (() -> Void)?
@Environment(\.colorScheme) private var colorScheme: ColorScheme
@@ -28,14 +29,18 @@ struct FloatingDateBadge: View {
var body: some View {
ZStack {
if let dateText {
Text(dateText)
.font(.compound.bodySMSemibold)
.foregroundColor(.compound.textPrimary)
.padding(.horizontal, 16)
.padding(.vertical, 8)
.background(backgroundColor, in: .capsule)
.shadow(color: Color(red: 0.11, green: 0.11, blue: 0.13).opacity(0.1), radius: 12, x: 0, y: 4)
.transition(.opacity)
Button { onTap?() } label: {
Text(dateText)
.font(.compound.bodySMSemibold)
.foregroundColor(.compound.textPrimary)
.padding(.horizontal, 16)
.padding(.vertical, 8)
.background(backgroundColor, in: .capsule)
.shadow(color: Color(red: 0.11, green: 0.11, blue: 0.13).opacity(0.1), radius: 12, x: 0, y: 4)
}
.buttonStyle(.plain)
.disabled(onTap == nil)
.transition(.opacity)
}
}
.animation(.easeInOut(duration: 0.15).disabledDuringTests(), value: dateText)

View File

@@ -83,8 +83,9 @@ struct TimelineViewRepresentable: UIViewControllerRepresentable {
func makeUIViewController(context: Context) -> TimelineTableViewController {
TimelineTableViewController(coordinator: context.coordinator,
isScrolledToBottom: $viewModelContext.isScrolledToBottom,
floatingDateText: $viewModelContext.floatingDateText,
scrollToBottomPublisher: viewModelContext.viewState.timelineState.scrollToBottomPublisher)
floatingDate: $viewModelContext.floatingDate,
scrollToBottomPublisher: viewModelContext.viewState.timelineState.scrollToBottomPublisher,
scrollToFirstItemForDatePublisher: viewModelContext.viewState.timelineState.scrollToFirstItemForDatePublisher)
}
func updateUIViewController(_ uiViewController: TimelineTableViewController, context: Context) {