From bbd64092e7e2a2521c2772f8c53a4de7f2afe312 Mon Sep 17 00:00:00 2001 From: Doug <6060466+pixlwave@users.noreply.github.com> Date: Mon, 20 Mar 2023 10:29:49 +0000 Subject: [PATCH] Room list avatar accessibility. (#707) * Hide the avatar when the Dynamic Text is large --------- Co-authored-by: sempruijs --- .../HomeScreen/View/HomeScreenRoomCell.swift | 16 ++++++++++------ changelog.d/pr-690.bugfix | 1 + 2 files changed, 11 insertions(+), 6 deletions(-) create mode 100644 changelog.d/pr-690.bugfix diff --git a/ElementX/Sources/Screens/HomeScreen/View/HomeScreenRoomCell.swift b/ElementX/Sources/Screens/HomeScreen/View/HomeScreenRoomCell.swift index 9a15ad17e..350b4e707 100644 --- a/ElementX/Sources/Screens/HomeScreen/View/HomeScreenRoomCell.swift +++ b/ElementX/Sources/Screens/HomeScreen/View/HomeScreenRoomCell.swift @@ -17,6 +17,7 @@ import SwiftUI struct HomeScreenRoomCell: View { + @Environment(\.dynamicTypeSize) var dynamicTypeSize @Environment(\.redactionReasons) private var redactionReasons let room: HomeScreenRoom @@ -52,12 +53,15 @@ struct HomeScreenRoomCell: View { @ViewBuilder var avatar: some View { - LoadableAvatarImage(url: room.avatarURL, - name: room.name, - contentID: room.roomId, - avatarSize: .room(on: .home), - imageProvider: context.imageProvider) - .accessibilityHidden(true) + if dynamicTypeSize < .accessibility3 { + LoadableAvatarImage(url: room.avatarURL, + name: room.name, + contentID: room.roomId, + avatarSize: .room(on: .home), + imageProvider: context.imageProvider) + .dynamicTypeSize(dynamicTypeSize < .accessibility1 ? dynamicTypeSize : .accessibility1) + .accessibilityHidden(true) + } } var content: some View { diff --git a/changelog.d/pr-690.bugfix b/changelog.d/pr-690.bugfix new file mode 100644 index 000000000..0f13c9a81 --- /dev/null +++ b/changelog.d/pr-690.bugfix @@ -0,0 +1 @@ +Hide the avatars when the users has larger font on by Sem Pruijs