Bump build number and update changelog

This isn't great, but there's nowhere else for either to go and the
difference between the large/default scales wasn't doing much before.

Closes #449

CHANGELOG.md

@@ -1,5 +1,23 @@
 # Changelog
 
+## 2024.3 (129)
+Bugfixes:
+- Fix excessive network traffic on profile pages
+- Fix attachment gallery controls visibility not being synced between pages
+- Fix video attachments not restarting when play pressed while at ends
+- Fix profile field text being misaligned
+- Fix at sign in timeline statuses usernames sometimes clipping
+- Fix add hashtag/instance to Pinned Timelines sheets dismissing immediately when opened
+- Fix for display name being replaced with incorrect user in certain circumstances
+- Fix profile moved overlay view appearing behind avatar/header
+- Fix profile moved view accessibility with VoiceOver
+- Fix mention/status push notifications not showing content warning
+- Fix sensitive attachment thumbnails being shown in push notifications
+- Fix Dynamic Type not applying to status content
+- Fix expand all option in Conversation not transferring when opening ancestors
+- Fix not being able to resolve remote Mastodon status links in Conversation screen
+- Fix status indicator icons overlapping thread links when Dynamic Type is enabled
+
 ## 2024.3 (128)
 Bugfixes:
 - Fix selecting poll option playing too much haptic feedback

NotificationExtension/NotificationService.swift

@@ -122,7 +122,12 @@ class NotificationService: UNNotificationServiceExtension {
         
         let notificationContent: String?
         if let status = notification.status {
+            if notification.kind == .mention || notification.kind == .status,
+               !status.spoilerText.isEmpty {
+                notificationContent = "⚠️ \(status.spoilerText)"
+            } else {
                 notificationContent = NotificationService.textConverter.convert(html: status.content)
+            }
         } else if notification.kind == .follow || notification.kind == .followRequest {
             notificationContent = nil
         } else {
@@ -135,7 +140,9 @@ class NotificationService: UNNotificationServiceExtension {
         // We deliberately don't include attachments for other types of notifications that have statuses (favs, etc.)
         // because we risk just fetching the same thing a bunch of times for many senders.
         if notification.kind == .mention || notification.kind == .status || notification.kind == .update,
-           let attachment = notification.status?.attachments.first {
+           let status = notification.status,
+           !status.sensitive,
+           let attachment = status.attachments.first {
             let url = attachment.previewURL ?? attachment.url
             attachmentDataTask = Task {
                 do {

Packages/GalleryVC/Sources/GalleryVC/GalleryItemViewController.swift

@@ -44,6 +44,7 @@ class GalleryItemViewController: UIViewController {
     private(set) var scrollAndZoomEnabled = true
     
     private var scrollViewSizeForLastZoomScaleUpdate: CGSize?
+    
     override var prefersHomeIndicatorAutoHidden: Bool {
         return !controlsVisible
     }
@@ -227,6 +228,8 @@ class GalleryItemViewController: UIViewController {
             updateZoomScale(resetZoom: true)
         }
         centerContent()
+        // Ensure the transform is correct if the controls are hidden and their size changed.
+        setControlsVisible(controlsVisible, animated: false)
     }
     
     override func viewDidAppear(_ animated: Bool) {
@@ -289,10 +292,12 @@ class GalleryItemViewController: UIViewController {
     
     func setControlsVisible(_ visible: Bool, animated: Bool) {
         controlsVisible = visible
+        
         guard let topControlsView,
               let bottomControlsView else {
             return
         }
+        
         func updateControlsViews() {
             topControlsView.transform = CGAffineTransform(translationX: 0, y: visible ? 0 : -topControlsView.bounds.height)
             bottomControlsView.transform = CGAffineTransform(translationX: 0, y: visible ? 0 : bottomControlsView.bounds.height)

Packages/GalleryVC/Sources/GalleryVC/GalleryViewController.swift

@@ -125,6 +125,8 @@ extension GalleryViewController: UIPageViewControllerDataSource {
 extension GalleryViewController: UIPageViewControllerDelegate {
     public func pageViewController(_ pageViewController: UIPageViewController, willTransitionTo pendingViewControllers: [UIViewController]) {
         currentItemViewController.content.galleryContentWillDisappear()
+        let new = pendingViewControllers[0] as! GalleryItemViewController
+        new.setControlsVisible(currentItemViewController.controlsVisible, animated: false)
     }
     
     public func pageViewController(_ pageViewController: UIPageViewController, didFinishAnimating finished: Bool, previousViewControllers: [UIViewController], transitionCompleted completed: Bool) {

Tusker.xcodeproj/project.pbxproj

@@ -235,6 +235,7 @@
 		D698F46D2BD0B8310054DB14 /* AnnouncementsCollection.swift in Sources */ = {isa = PBXBuildFile; fileRef = D698F46C2BD0B8310054DB14 /* AnnouncementsCollection.swift */; };
 		D698F46F2BD0B8DF0054DB14 /* AddReactionView.swift in Sources */ = {isa = PBXBuildFile; fileRef = D698F46E2BD0B8DF0054DB14 /* AddReactionView.swift */; };
 		D698F4712BD0CBAA0054DB14 /* AnnouncementContentTextView.swift in Sources */ = {isa = PBXBuildFile; fileRef = D698F4702BD0CBAA0054DB14 /* AnnouncementContentTextView.swift */; };
+		D69F26342C4CDFD300FAF761 /* AccountDisplayAndUserNameLabel.swift in Sources */ = {isa = PBXBuildFile; fileRef = D69F26332C4CDFD300FAF761 /* AccountDisplayAndUserNameLabel.swift */; };
 		D6A00B1D26379FC900316AD4 /* PollOptionsView.swift in Sources */ = {isa = PBXBuildFile; fileRef = D6A00B1C26379FC900316AD4 /* PollOptionsView.swift */; };
 		D6A3A380295515550036B6EF /* ProfileHeaderButton.swift in Sources */ = {isa = PBXBuildFile; fileRef = D6A3A37F295515550036B6EF /* ProfileHeaderButton.swift */; };
 		D6A3A3822956123A0036B6EF /* TimelinePosition.swift in Sources */ = {isa = PBXBuildFile; fileRef = D6A3A3812956123A0036B6EF /* TimelinePosition.swift */; };
@@ -668,6 +669,7 @@
 		D698F46C2BD0B8310054DB14 /* AnnouncementsCollection.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = AnnouncementsCollection.swift; sourceTree = "<group>"; };
 		D698F46E2BD0B8DF0054DB14 /* AddReactionView.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = AddReactionView.swift; sourceTree = "<group>"; };
 		D698F4702BD0CBAA0054DB14 /* AnnouncementContentTextView.swift */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.swift; path = AnnouncementContentTextView.swift; sourceTree = "<group>"; };
+		D69F26332C4CDFD300FAF761 /* AccountDisplayAndUserNameLabel.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = AccountDisplayAndUserNameLabel.swift; sourceTree = "<group>"; };
 		D6A00B1C26379FC900316AD4 /* PollOptionsView.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = PollOptionsView.swift; sourceTree = "<group>"; };
 		D6A3A37F295515550036B6EF /* ProfileHeaderButton.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = ProfileHeaderButton.swift; sourceTree = "<group>"; };
 		D6A3A3812956123A0036B6EF /* TimelinePosition.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = TimelinePosition.swift; sourceTree = "<group>"; };
@@ -1480,7 +1482,9 @@
 		D6BED1722126661300F02DA0 /* Views */ = {
 			isa = PBXGroup;
 			children = (
+				D6D79F562A1160B800AB2315 /* AccountDisplayNameLabel.swift */,
 				D6B4A4FE2506B81A000C81C1 /* AccountDisplayNameView.swift */,
+				D69F26332C4CDFD300FAF761 /* AccountDisplayAndUserNameLabel.swift */,
 				D68E6F5E253C9B2D001A1B4C /* BaseEmojiLabel.swift */,
 				D6ADB6EF28ED1F25009924AB /* CachedImageView.swift */,
 				D6895DC328D65342006341DA /* ConfirmReblogStatusPreviewView.swift */,
@@ -1514,7 +1518,6 @@
 				D641C78B213DD92F004B4513 /* Profile Header */,
 				D641C78A213DD926004B4513 /* Status */,
 				D64AAE8F26C80DB600FC57FB /* Toast */,
-				D6D79F562A1160B800AB2315 /* AccountDisplayNameLabel.swift */,
 			);
 			path = Views;
 			sourceTree = "<group>";
@@ -2357,6 +2360,7 @@
 				D65B4B542971F71D00DABDFB /* EditedReport.swift in Sources */,
 				D62D67C52A97D8CD00167EE2 /* MultiColumnNavigationController.swift in Sources */,
 				D6A6C11525B62E9700298D0F /* CacheExpiry.swift in Sources */,
+				D69F26342C4CDFD300FAF761 /* AccountDisplayAndUserNameLabel.swift in Sources */,
 				D67C57AF21E28EAD00C3118B /* Array+Uniques.swift in Sources */,
 				D68A76F129539116001DA1B3 /* FlipView.swift in Sources */,
 				D6945C3223AC4D36005C403C /* HashtagTimelineViewController.swift in Sources */,
@@ -3256,7 +3260,7 @@
 			repositoryURL = "https://git.shadowfacts.net/shadowfacts/HTMLStreamer.git";
 			requirement = {
 				kind = exactVersion;
-				version = 0.2.5;
+				version = 0.3.0;
 			};
 		};
 		D63CC700290EC0B8000E19DE /* XCRemoteSwiftPackageReference "sentry-cocoa" */ = {

Tusker/HTMLConverter.swift

@@ -25,8 +25,14 @@ class HTMLConverter {
     
     private let converter: AttributedStringConverter<Callbacks>
     
-    init(font: UIFont, monospaceFont: UIFont, color: UIColor, paragraphStyle: NSParagraphStyle) {
-        let config = AttributedStringConverterConfiguration(font: font, monospaceFont: monospaceFont, color: color, paragraphStyle: paragraphStyle)
+    init(font: UIFont, monospaceFont: UIFont, fontMetrics: UIFontMetrics, color: UIColor, paragraphStyle: NSParagraphStyle) {
+        let config = AttributedStringConverterConfiguration(
+            font: font,
+            monospaceFont: monospaceFont,
+            fontMetrics: fontMetrics,
+            color: color,
+            paragraphStyle: paragraphStyle
+        )
         self.converter = AttributedStringConverter(configuration: config)
     }
    

Tusker/Screens/Conversation/ConversationCollectionViewController.swift

@@ -364,7 +364,9 @@ extension ConversationCollectionViewController: UICollectionViewDelegate {
                 conv.showStatusesAutomatically = showStatusesAutomatically
                 show(conv)
             } else {
-                selected(status: id, state: state.copy())
+                let conv = ConversationViewController(for: id, state: state.copy(), mastodonController: mastodonController)
+                conv.showStatusesAutomatically = showStatusesAutomatically
+                show(conv)
             }
         case .expandThread(childThreads: let childThreads, inline: _):
             let indexPathBeforeExpandThread = IndexPath(row: indexPath.row - 1, section: indexPath.section)

Tusker/Screens/Conversation/ConversationViewController.swift

@@ -221,20 +221,31 @@ class ConversationViewController: UIViewController {
                 completionHandler(nil)
             }
         }
-        if isLikelyMastodonRemoteStatus(url: url),
-           let (_, response) = try? await URLSession.appDefault.data(from: url, delegate: RedirectBlocker()),
+        if isLikelyMastodonRemoteStatus(url: url) {
+            var request = URLRequest(url: url)
+            // Mastodon uses an intermediate redirect page for browsers which requires user input that we don't want.
+            request.addValue("application/activity+json", forHTTPHeaderField: "accept")
+            if let (_, response) = try? await URLSession.appDefault.data(for: request, delegate: RedirectBlocker()),
               let location = (response as? HTTPURLResponse)?.value(forHTTPHeaderField: "location") {
                effectiveURL = location
            } else {
                effectiveURL = WebURL(url)!.serialized(excludingFragment: true)
            }
+        } else {
+            effectiveURL = WebURL(url)!.serialized(excludingFragment: true)
+        }
         
         let request = Client.search(query: effectiveURL, types: [.statuses], resolve: true)
         do {
             let (results, _) = try await mastodonController.run(request)
-            guard let status = results.statuses.compactMap(\.value).first(where: { $0.url?.serialized() == effectiveURL }) else {
+            let statuses = results.statuses.compactMap(\.value)
+            // Don't try to exactly match effective URL because the URL form Mastodon
+            // uses for the ActivityPub redirect doesn't match what's returned by the API.
+            // Instead we just assume that, if only one status was returned, it worked.
+            guard statuses.count == 1 else {
                 throw UnableToResolveError()
             }
+            let status = statuses[0]
             _ = mastodonController.persistentContainer.addOrUpdateOnViewContext(status: status)
             mode = .localID(status.id)
             return status.id

Tusker/Screens/Customize Timelines/CustomizeTimelinesView.swift

@@ -13,7 +13,7 @@ struct CustomizeTimelinesView: View {
     let mastodonController: MastodonController
     
     var body: some View {
-        CustomizeTimelinesList()
+        CustomizeTimelinesList(pinnedTimelines: mastodonController.accountPreferences.pinnedTimelines)
             .environmentObject(mastodonController)
             .environment(\.managedObjectContext, mastodonController.persistentContainer.viewContext)
     }
@@ -26,6 +26,14 @@ struct CustomizeTimelinesList: View {
     @FetchRequest(sortDescriptors: []) private var filters: FetchedResults<FilterMO>
     @Environment(\.dismiss) private var dismiss
     @State private var deletionError: (any Error)?
+    // store this separately from AccountPreferences in the view, b/c the @LazilyDecoding wrapper breaks animations
+    @State private var pinnedTimelines: [PinnedTimeline]
+    @State private var isShowingAddHashtagSheet = false
+    @State private var isShowingAddInstanceSheet = false
+    
+    init(pinnedTimelines: [PinnedTimeline]) {
+        self.pinnedTimelines = pinnedTimelines
+    }
 
     var body: some View {
         if #available(iOS 16.0, *) {
@@ -50,7 +58,11 @@ struct CustomizeTimelinesList: View {
     
     private var navigationBody: some View {
         List {
-            PinnedTimelinesView(accountPreferences: mastodonController.accountPreferences)
+            PinnedTimelinesView(
+                pinnedTimelines: $pinnedTimelines,
+                isShowingAddHashtagSheet: $isShowingAddHashtagSheet,
+                isShowingAddInstanceSheet: $isShowingAddInstanceSheet
+            )
             .appGroupedListRowBackground()
             
             Section {
@@ -99,6 +111,12 @@ struct CustomizeTimelinesList: View {
                 }
             }
         }
+        .modifier(PinnedTimelinesModifier(
+            accountPreferences: mastodonController.accountPreferences,
+            pinnedTimelines: $pinnedTimelines,
+            isShowingAddHashtagSheet: $isShowingAddHashtagSheet,
+            isShowingAddInstanceSheet: $isShowingAddInstanceSheet
+        ))
         .alertWithData("Error Deleting Filter", data: $deletionError, actions: { _ in
             Button("OK") {
                 self.deletionError = nil

Tusker/Screens/Customize Timelines/PinnedTimelinesView.swift

@@ -11,17 +11,10 @@ import Pachyderm
 
 struct PinnedTimelinesView: View {
     @EnvironmentObject private var mastodonController: MastodonController
-    @ObservedObject private var accountPreferences: AccountPreferences
     
-    @State private var isShowingAddHashtagSheet = false
-    @State private var isShowingAddInstanceSheet = false
-    // store this separately from AccountPreferences in the view, b/c the @LazilyDecoding wrapper breaks animations
-    @State private var pinnedTimelines: [PinnedTimeline]
-    
-    init(accountPreferences: AccountPreferences) {
-        self.accountPreferences = accountPreferences
-        self.pinnedTimelines = accountPreferences.pinnedTimelines
-    }
+    @Binding var pinnedTimelines: [PinnedTimeline]
+    @Binding var isShowingAddHashtagSheet: Bool
+    @Binding var isShowingAddInstanceSheet: Bool
     
     var body: some View {
         Section {
@@ -110,6 +103,17 @@ struct PinnedTimelinesView: View {
         } header: {
             Text("Pinned Timelines")
         }
+    }
+}
+
+struct PinnedTimelinesModifier: ViewModifier {
+    let accountPreferences: AccountPreferences
+    @Binding var pinnedTimelines: [PinnedTimeline]
+    @Binding var isShowingAddHashtagSheet: Bool
+    @Binding var isShowingAddInstanceSheet: Bool
+
+    func body(content: Content) -> some View {
+        content
             .sheet(isPresented: $isShowingAddHashtagSheet, content: {
                 #if os(visionOS)
                 AddHashtagPinnedTimelineView(pinnedTimelines: $pinnedTimelines)

Tusker/Screens/Gallery/VideoOverlayViewController.swift

@@ -106,6 +106,9 @@ class VideoOverlayViewController: UIViewController {
         if player.rate > 0 {
             player.rate = 0
         } else {
+            if player.currentTime() >= player.currentItem!.duration {
+                player.seek(to: .zero)
+            }
             #if os(visionOS)
             player.play()
             #else

Tusker/Screens/Preferences/Appearance/MockStatusView.swift

@@ -135,12 +135,11 @@ private struct MockStatusCardView: UIViewRepresentable {
     func makeUIView(context: Context) -> StatusCardView {
         let view = StatusCardView()
         view.isUserInteractionEnabled = false
-        let card = Card(
+        let card = StatusCardView.CardData(
             url: WebURL("https://vaccor.space/tusker")!,
-            title: "Tusker",
-            description: "Tusker is an iOS app for Mastodon",
             image: WebURL("https://vaccor.space/tusker/img/icon.png")!,
-            kind: .link
+            title: "Tusker",
+            description: "Tusker is an iOS app for Mastodon"
         )
         view.updateUI(card: card, sensitive: false)
         return view

Tusker/Screens/Profile/ProfileStatusesViewController.swift

@@ -9,12 +9,6 @@
 import UIKit
 import Pachyderm
 import Combine
-import OSLog
-#if canImport(Sentry)
-import Sentry
-#endif
-
-private let logger = Logger(subsystem: Bundle.main.bundleIdentifier!, category: "ProfileStatusesViewController")
 
 class ProfileStatusesViewController: UIViewController, TimelineLikeCollectionViewController, CollectionViewController {
     
@@ -255,20 +249,6 @@ class ProfileStatusesViewController: UIViewController, TimelineLikeCollectionVie
         
         state = .setupInitialSnapshot
         
-        Task {
-            do {
-                let (all, _) = try await mastodonController.run(Client.getRelationships(accounts: [accountID]))
-                if let relationship = all.first {
-                    self.mastodonController.persistentContainer.addOrUpdate(relationship: relationship)
-                }
-            } catch {
-                logger.error("Error fetching relationship: \(String(describing: error))")
-                #if canImport(Sentry)
-                SentrySDK.capture(error: error)
-                #endif
-            }
-        }
-        
         await controller.loadInitial()
         await tryLoadPinned()
         

Tusker/Screens/Profile/ProfileViewController.swift

@@ -9,6 +9,12 @@
 import UIKit
 import Pachyderm
 import Combine
+import OSLog
+#if canImport(Sentry)
+import Sentry
+#endif
+
+private let logger = Logger(subsystem: Bundle.main.bundleIdentifier!, category: "ProfileViewController")
 
 class ProfileViewController: UIViewController, StateRestorableViewController {
     
@@ -120,6 +126,19 @@ class ProfileViewController: UIViewController, StateRestorableViewController {
         guard let accountID else {
             return
         }
+        Task {
+            do {
+                let (all, _) = try await mastodonController.run(Client.getRelationships(accounts: [accountID]))
+                if let relationship = all.first {
+                    self.mastodonController.persistentContainer.addOrUpdate(relationship: relationship)
+                }
+            } catch {
+                logger.error("Error fetching relationship: \(String(describing: error))")
+                #if canImport(Sentry)
+                SentrySDK.capture(error: error)
+                #endif
+            }
+        }
         if let account = mastodonController.persistentContainer.account(for: accountID) {
             updateAccountUI(account: account)
         } else {

Tusker/Screens/Report/ReportStatusView.swift

@@ -12,6 +12,7 @@ import SwiftUI
 private var converter = HTMLConverter(
     font: .preferredFont(forTextStyle: .body),
     monospaceFont: UIFontMetrics.default.scaledFont(for: .monospacedSystemFont(ofSize: 17, weight: .regular)),
+    fontMetrics: .default,
     color: .label,
     paragraphStyle: .default
 )

Tusker/Views/AccountDisplayAndUserNameLabel.swift

@@ -0,0 +1,57 @@
+//
+//  AccountDisplayAndUserNameLabel.swift
+//  Tusker
+//
+//  Created by Shadowfacts on 7/20/24.
+//  Copyright © 2024 Shadowfacts. All rights reserved.
+//
+
+import UIKit
+import Pachyderm
+
+class AccountDisplayAndUserNameLabel: EmojiLabel {
+    var baseFont: UIFontDescriptor = .preferredFontDescriptor(withTextStyle: .body)
+    
+    private var state: State?
+    
+    func updateUI(account: some AccountProtocol) {
+        let state = State(accountID: account.id, displayName: account.displayName, acct: account.acct)
+        guard state != self.state || Preferences.shared.hideCustomEmojiInUsernames != hasEmojis else {
+            return
+        }
+        self.state = state
+        self.attributedText = makeAttributedText(state: state)
+        if Preferences.shared.hideCustomEmojiInUsernames {
+            self.removeEmojis()
+        } else {
+            self.setEmojis(account.emojis, identifier: state.accountID)
+        }
+    }
+    
+    private func makeAttributedText(state: State) -> NSAttributedString {
+        let s = NSMutableAttributedString()
+        s.append(NSAttributedString(string: state.displayName, attributes: [
+            .font: UIFont(descriptor: baseFont.addingAttributes([
+                .traits: [
+                    UIFontDescriptor.TraitKey.weight: UIFont.Weight.semibold.rawValue,
+                ]
+            ]), size: 0),
+        ]))
+        s.append(NSAttributedString(string: " "))
+        s.append(NSAttributedString(string: "@\(state.acct)", attributes: [
+            .font: UIFont(descriptor: baseFont.addingAttributes([
+                .traits: [
+                    UIFontDescriptor.TraitKey.weight: UIFont.Weight.light.rawValue,
+                ]
+            ]), size: 0),
+            .foregroundColor: UIColor.secondaryLabel,
+        ]))
+        return s
+    }
+    
+    private struct State: Equatable {
+        var accountID: String
+        var displayName: String
+        var acct: String
+    }
+}

Tusker/Views/AccountDisplayNameLabel.swift

@@ -16,7 +16,7 @@ class AccountDisplayNameLabel: EmojiLabel {
     private var accountDisplayName: String?
 
     func updateForAccountDisplayName(account: some AccountProtocol) {
-        guard accountID != account.id || accountDisplayName != account.displayName || Preferences.shared.hideCustomEmojiInUsernames == hasEmojis else {
+        guard accountID != account.id || accountDisplayName != account.displayName || Preferences.shared.hideCustomEmojiInUsernames != hasEmojis else {
             return
         }
         accountID = account.id

Tusker/Views/ConfirmReblogStatusPreviewView.swift

@@ -13,6 +13,7 @@ class ConfirmReblogStatusPreviewView: UIView {
     private static let htmlConverter = HTMLConverter(
         font: .preferredFont(forTextStyle: .caption2),
         monospaceFont: UIFontMetrics(forTextStyle: .caption2).scaledFont(for: .monospacedSystemFont(ofSize: 17, weight: .regular)),
+        fontMetrics: UIFontMetrics(forTextStyle: .caption2),
         color: .label,
         paragraphStyle: .default
     )

Tusker/Views/ContentTextView.swift

@@ -25,6 +25,7 @@ class ContentTextView: LinkTextView, BaseEmojiLabel {
     private static let defaultBodyHTMLConverter = HTMLConverter(
         font: .preferredFont(forTextStyle: .body),
         monospaceFont: UIFontMetrics.default.scaledFont(for: .monospacedSystemFont(ofSize: 17, weight: .regular)),
+        fontMetrics: .default,
         color: .label,
         paragraphStyle: .default
     )

Tusker/Views/Profile Header/ProfileFieldValueView.swift

@@ -21,6 +21,7 @@ class ProfileFieldValueView: UIView {
     private static let converter = HTMLConverter(
         font: .preferredFont(forTextStyle: .body),
         monospaceFont: UIFontMetrics.default.scaledFont(for: .monospacedSystemFont(ofSize: 17, weight: .regular)),
+        fontMetrics: .default,
         color: .label,
         paragraphStyle: .default
     )
@@ -54,8 +55,8 @@ class ProfileFieldValueView: UIView {
         textView.isScrollEnabled = false
         textView.isSelectable = false
         textView.isEditable = false
-        textView.textContainerInset = .zero
         textView.font = .preferredFont(forTextStyle: .body)
+        updateTextContainerInset()
         textView.adjustsFontForContentSizeCategory = true
         textView.attributedText = converted
         textView.setEmojis(account.emojis, identifier: account.id)
@@ -108,6 +109,27 @@ class ProfileFieldValueView: UIView {
         return size
     }
     
+    override func traitCollectionDidChange(_ previousTraitCollection: UITraitCollection?) {
+        super.traitCollectionDidChange(previousTraitCollection)
+        if traitCollection.preferredContentSizeCategory != previousTraitCollection?.preferredContentSizeCategory {
+            updateTextContainerInset()
+        }
+    }
+    
+    private func updateTextContainerInset() {
+        // blergh
+        switch traitCollection.preferredContentSizeCategory {
+        case .extraSmall:
+            textView.textContainerInset = UIEdgeInsets(top: 4, left: 0, bottom: 0, right: 0)
+        case .small:
+            textView.textContainerInset = UIEdgeInsets(top: 3, left: 0, bottom: 0, right: 0)
+        case .medium, .large:
+            textView.textContainerInset = UIEdgeInsets(top: 2, left: 0, bottom: 0, right: 0)
+        default:
+            textView.textContainerInset = .zero
+        }
+    }
+    
     func setTextAlignment(_ alignment: NSTextAlignment) {
         textView.textAlignment = alignment
     }

Tusker/Views/Profile Header/ProfileHeaderMovedOverlayView.swift

@@ -14,6 +14,7 @@ class ProfileHeaderMovedOverlayView: UIView {
     weak var delegate: TuskerNavigationDelegate?
     
     var collapse: (() -> Void)?
+    var hide: (() -> Void)?
 
     private var avatarImageView: CachedImageView!
     private var displayNameLabel: EmojiLabel!
@@ -145,6 +146,45 @@ class ProfileHeaderMovedOverlayView: UIView {
         delegate?.selected(account: movedToID)
     }
     
+    // MARK: Accessibility
+    
+    override var isAccessibilityElement: Bool {
+        get { true }
+        set {}
+    }
+    
+    override var accessibilityLabel: String? {
+        get {
+            guard let movedToID,
+                  let account = delegate?.apiController?.persistentContainer.account(for: movedToID) else {
+                return "This account has moved"
+            }
+            return "This account has moved to @\(account.acct)"
+        }
+        set {}
+    }
+    
+    override func accessibilityActivate() -> Bool {
+        guard let movedToID,
+              let delegate else {
+            return false
+        }
+        delegate.selected(account: movedToID)
+        return true
+    }
+    
+    override var accessibilityCustomActions: [UIAccessibilityCustomAction]? {
+        get {
+            [
+                UIAccessibilityCustomAction(name: "Hide banner", actionHandler: { [unowned self] _ in
+                    self.hide?()
+                    return true
+                })
+            ]
+        }
+        set {}
+    }
+    
 }
 
 extension ProfileHeaderMovedOverlayView: UIPointerInteractionDelegate {

Tusker/Views/Profile Header/ProfileHeaderView.swift

@@ -41,6 +41,7 @@ class ProfileHeaderView: UIView {
     @IBOutlet weak var followersCountButton: UIButton!
     private(set) var pagesSegmentedControl: ScrollingSegmentedControl<ProfileViewController.Page>!
     private var movedOverlayView: ProfileHeaderMovedOverlayView?
+    private var hideMovedOverlayView = false
     
     var accountID: String!
     
@@ -178,7 +179,8 @@ class ProfileHeaderView: UIView {
         followersCountButton.setAttributedTitle(followersCountTitle, for: .normal)
         followersCountButton.accessibilityLabel = "\(followersSpelledOut) followers"
         
-        if let movedTo = account.movedTo {
+        if let movedTo = account.movedTo,
+           !hideMovedOverlayView {
             if let movedOverlayView {
                 movedOverlayView.updateUI(movedTo: movedTo)
             } else {
@@ -207,6 +209,7 @@ class ProfileHeaderView: UIView {
     
     private func createMovedOverlayView(movedTo: AccountMO) -> ProfileHeaderMovedOverlayView {
         let overlay = ProfileHeaderMovedOverlayView()
+        overlay.layer.zPosition = 1000
         overlay.delegate = delegate
         overlay.updateUI(movedTo: movedTo)
         overlay.translatesAutoresizingMaskIntoConstraints = false
@@ -234,6 +237,12 @@ class ProfileHeaderView: UIView {
             }
             animator.startAnimation()
         }
+        overlay.hide = { [weak self] in
+            guard let self else { return }
+            self.hideMovedOverlayView = true
+            self.updateUI(for: self.accountID)
+            UIAccessibility.post(notification: .layoutChanged, argument: self)
+        }
         
         return overlay
     }

Tusker/Views/Status/ConversationMainStatusCollectionViewCell.swift

@@ -20,6 +20,7 @@ class ConversationMainStatusCollectionViewCell: UICollectionViewListCell, Status
     private static let htmlConverter = HTMLConverter(
         font: ConversationMainStatusCollectionViewCell.contentFont,
         monospaceFont: ConversationMainStatusCollectionViewCell.monospaceFont,
+        fontMetrics: .default,
         color: .label,
         paragraphStyle: ConversationMainStatusCollectionViewCell.contentParagraphStyle
     )
@@ -514,8 +515,15 @@ class ConversationMainStatusCollectionViewCell: UICollectionViewListCell, Status
         return contentContainer.estimateVisibleSubviewHeight(effectiveWidth: width)
     }
     
+    func updateAccountUI(account: AccountMO) {
+        baseUpdateAccountUI(account: account)
+        displayNameLabel.updateForAccountDisplayName(account: account)
+        usernameLabel.text = "@\(account.acct)"
+    }
+    
     func updateUIForPreferences(status: StatusMO) {
         baseUpdateUIForPreferences(status: status)
+        displayNameLabel.updateForAccountDisplayName(account: status.account)
     }
     
     @objc private func preferencesChanged() {

Tusker/Views/Status/StatusCardView.swift

@@ -9,6 +9,7 @@
 import UIKit
 import Pachyderm
 import SafariServices
+import WebURL
 import WebURLFoundationExtras
 import HTMLStreamer
 
@@ -18,7 +19,7 @@ class StatusCardView: UIView {
     weak var actionProvider: MenuActionProvider?
     
     private var statusID: String?
-    private(set) var card: Card?
+    private(set) var card: CardData?
     
     private static let activeBackgroundColor = UIColor.secondarySystemFill
     private static let inactiveBackgroundColor = UIColor.secondarySystemBackground
@@ -163,20 +164,22 @@ class StatusCardView: UIView {
     }
     
     func updateUI(status: StatusMO) {
-        guard status.id != statusID else {
+        let newData = status.card.map { CardData(card: $0) }
+        guard self.card != newData else {
             return
         }
-        self.card = status.card
+        self.card = newData
         self.statusID = status.id
         
-        guard let card = status.card else {
+        guard let newData else {
             return
         }
         
-        updateUI(card: card, sensitive: status.sensitive)
+        updateUI(card: newData, sensitive: status.sensitive)
     }
     
-    func updateUI(card: Card, sensitive: Bool) {
+    // This method is internal for use by MockStatusView
+    func updateUI(card: CardData, sensitive: Bool) {
         if let image = card.image {
             if sensitive {
                 if let blurhash = card.blurhash {
@@ -244,6 +247,30 @@ class StatusCardView: UIView {
         setNeedsDisplay()
     }
     
+    struct CardData: Equatable {
+        let url: WebURL
+        let image: WebURL?
+        let title: String
+        let description: String
+        let blurhash: String?
+        
+        init(card: Card) {
+            self.url = card.url
+            self.image = card.image
+            self.title = card.title
+            self.description = card.description
+            self.blurhash = card.blurhash
+        }
+        
+        init(url: WebURL, image: WebURL? = nil, title: String, description: String, blurhash: String? = nil) {
+            self.url = url
+            self.image = image
+            self.title = title
+            self.description = description
+            self.blurhash = blurhash
+        }
+    }
+
 }
 
 extension StatusCardView: UIContextMenuInteractionDelegate {

Tusker/Views/Status/StatusCollectionViewCell.swift

@@ -21,8 +21,6 @@ protocol StatusCollectionViewCellDelegate: AnyObject, TuskerNavigationDelegate,
 protocol StatusCollectionViewCell: UICollectionViewCell, AttachmentViewDelegate {
     // MARK: Subviews
     var avatarImageView: CachedImageView { get }
-    var displayNameLabel: AccountDisplayNameLabel { get }
-    var usernameLabel: UILabel { get }
     var contentWarningLabel: EmojiLabel { get }
     var collapseButton: StatusCollapseButton { get }
     var contentContainer: StatusContentContainer { get }
@@ -49,6 +47,7 @@ protocol StatusCollectionViewCell: UICollectionViewCell, AttachmentViewDelegate
     var isGrayscale: Bool { get set }
     var cancellables: Set<AnyCancellable> { get set }
     
+    func updateAccountUI(account: AccountMO)
     func updateAttachmentsUI(status: StatusMO)
     func updateUIForPreferences(status: StatusMO)
     func updateStatusState(status: StatusMO)
@@ -177,10 +176,8 @@ extension StatusCollectionViewCell {
         attachmentsView.updateUI(attachments: status.attachments)
     }
  
-    func updateAccountUI(account: AccountMO) {
+    func baseUpdateAccountUI(account: AccountMO) {
         avatarImageView.update(for: account.avatar)
-        displayNameLabel.updateForAccountDisplayName(account: account)
-        usernameLabel.text = "@\(account.acct)"
     }
     
     func baseUpdateUIForPreferences(status: StatusMO) {
@@ -219,7 +216,6 @@ extension StatusCollectionViewCell {
         if contentTextView.hasEmojis {
             contentTextView.setEmojis(status.emojis, identifier: status.id)
         }
-        displayNameLabel.updateForAccountDisplayName(account: status.account)
     }
     
     func baseUpdateStatusState(status: StatusMO) {

Tusker/Views/Status/StatusMetaIndicatorsView.swift

@@ -70,8 +70,7 @@ class StatusMetaIndicatorsView: UIView {
     
     private func configureImageView(_ imageView: UIImageView) {
         let weight: UIImage.SymbolWeight = UIAccessibility.isBoldTextEnabled ? .regular : traitCollection.preferredContentSizeCategory > .large ? .light : .thin
-        let scale: UIImage.SymbolScale = traitCollection.preferredContentSizeCategory > .extraLarge ? .large : .default
-        imageView.preferredSymbolConfiguration = .init(pointSize: 0, weight: weight, scale: scale)
+        imageView.preferredSymbolConfiguration = .init(pointSize: 0, weight: weight, scale: .default)
     }
     
     func updateUI(status: StatusMO) {

Tusker/Views/Status/TimelineStatusCollectionViewCell.swift

@@ -23,6 +23,7 @@ class TimelineStatusCollectionViewCell: UICollectionViewListCell, StatusCollecti
     static let htmlConverter = HTMLConverter(
         font: TimelineStatusCollectionViewCell.contentFont,
         monospaceFont: TimelineStatusCollectionViewCell.monospaceFont,
+        fontMetrics: .default,
         color: .label,
         paragraphStyle: TimelineStatusCollectionViewCell.contentParagraphStyle
     )
@@ -121,8 +122,7 @@ class TimelineStatusCollectionViewCell: UICollectionViewListCell, StatusCollecti
     }
     
     private lazy var nameHStack = UIStackView(arrangedSubviews: [
-        displayNameLabel,
-        usernameLabel,
+        displayAndUserNameLabel,
         pinImageView,
         timestampLabel,
     ]).configure {
@@ -130,27 +130,10 @@ class TimelineStatusCollectionViewCell: UICollectionViewListCell, StatusCollecti
         $0.spacing = 4
     }
     
-    let displayNameLabel = AccountDisplayNameLabel().configure {
-        $0.font = UIFont(descriptor: .preferredFontDescriptor(withTextStyle: .body).addingAttributes([
-            .traits: [
-                UIFontDescriptor.TraitKey.weight: UIFont.Weight.semibold.rawValue,
-            ]
-        ]), size: 0)
-        $0.adjustsFontForContentSizeCategory = true
-        $0.setContentHuggingPriority(.init(251), for: .horizontal)
-        $0.setContentCompressionResistancePriority(.init(749), for: .horizontal)
-    }
-    
-    let usernameLabel = UILabel().configure {
-        $0.textColor = .secondaryLabel
-        $0.font = UIFont(descriptor: .preferredFontDescriptor(withTextStyle: .body).addingAttributes([
-            .traits: [
-                UIFontDescriptor.TraitKey.weight: UIFont.Weight.light.rawValue,
-            ]
-        ]), size: 0)
+    let displayAndUserNameLabel = AccountDisplayAndUserNameLabel().configure {
         $0.adjustsFontForContentSizeCategory = true
         $0.setContentHuggingPriority(.init(249), for: .horizontal)
-        $0.setContentCompressionResistancePriority(.init(748), for: .horizontal)
+        $0.setContentCompressionResistancePriority(.init(749), for: .horizontal)
     }
     
     private let pinImageView = UIImageView(image: UIImage(systemName: "pin.fill")).configure {
@@ -693,6 +676,11 @@ class TimelineStatusCollectionViewCell: UICollectionViewListCell, StatusCollecti
             .store(in: &cancellables)
     }
     
+    func updateAccountUI(account: AccountMO) {
+        baseUpdateAccountUI(account: account)
+        displayAndUserNameLabel.updateUI(account: account)
+    }
+    
     func updateUIForPreferences(status: StatusMO) {
         baseUpdateUIForPreferences(status: status)
         
@@ -704,6 +692,8 @@ class TimelineStatusCollectionViewCell: UICollectionViewListCell, StatusCollecti
         metaIndicatorsView.updateUI(status: status)
         
         timelineReasonIcon.layer.cornerRadius = Preferences.shared.avatarStyle.cornerRadiusFraction * Self.timelineReasonIconSize
+        
+        displayAndUserNameLabel.updateUI(account: status.account)
     }
     
     func updateStatusState(status: StatusMO) {

Version.xcconfig

@@ -10,7 +10,7 @@
 // https://help.apple.com/xcode/#/dev745c5c974
 
 MARKETING_VERSION = 2024.3
-CURRENT_PROJECT_VERSION = 128
+CURRENT_PROJECT_VERSION = 129
 CURRENT_PROJECT_VERSION = $(inherited)$(CURRENT_PROJECT_VERSION_BUILD_SUFFIX_$(CONFIGURATION))
 
 CURRENT_PROJECT_VERSION_BUILD_SUFFIX_Debug=-dev