From 006008f61d08d436d81deb5dd95b9a43f90781d8 Mon Sep 17 00:00:00 2001
From: Henry-Hiles <henry@henryhiles.com>
Date: Sat, 3 Oct 2026 12:25:22 -0400
Subject: [PATCH] rename AvatarOrHash.height to be more accurate

---
 lib/widgets/avatar_or_hash.dart              | 12 ++++++------
 lib/widgets/lazy_loading/message_avatar.dart |  4 ++--
 lib/widgets/member_list.dart                 |  2 +-
 lib/widgets/room_appbar.dart                 |  2 +-
 lib/widgets/room_list.dart                   |  4 ++--
 lib/widgets/room_summary_dialog.dart         |  2 +-
 lib/widgets/user_bottom_sheet.dart           |  2 +-
 7 files changed, 14 insertions(+), 14 deletions(-)

diff --git a/lib/widgets/avatar_or_hash.dart b/lib/widgets/avatar_or_hash.dart
index 791dd67..bdaf657 100644
--- a/lib/widgets/avatar_or_hash.dart
+++ b/lib/widgets/avatar_or_hash.dart
@@ -7,24 +7,24 @@ final class const AvatarOrHash(
   final Uri? avatar,
   final String title, {
   final Widget? fallback,
-  final double height = 24,
+  final double dimension = 24,
   super.key,
 }) extends ConsumerWidget {
   @override
   Widget build(BuildContext context, WidgetRef ref) {
     final box = ColoredBox(
       color: ColorHash(title).color,
-      child: Center(child: Icon(Icons.person, size: height / 2)),
+      child: Center(child: Icon(Icons.person, size: dimension / 2)),
     );
 
     return SizedBox(
-      width: height,
-      height: height,
+      width: dimension,
+      height: dimension,
       child: Center(
         child: ClipRRect(
-          borderRadius: .all(.circular((height - 8) / 2.5)),
+          borderRadius: .all(.circular((dimension - 8) / 2.5)),
           child: SizedBox.square(
-            dimension: height,
+            dimension: dimension,
             child: avatar == null
                 ? fallback ?? box
                 : Image(
diff --git a/lib/widgets/lazy_loading/message_avatar.dart b/lib/widgets/lazy_loading/message_avatar.dart
index 709d558..50b612a 100644
--- a/lib/widgets/lazy_loading/message_avatar.dart
+++ b/lib/widgets/lazy_loading/message_avatar.dart
@@ -21,9 +21,9 @@ class const MessageAvatar(
       child: AvatarOrHash(
         value.avatarUrl,
         value.displayName ?? event.sender.localpart,
-        height: height,
+        dimension: height,
       ),
     ),
-    _ => AvatarOrHash(null, event.sender.localpart, height: height),
+    _ => AvatarOrHash(null, event.sender.localpart, dimension: height),
   };
 }
diff --git a/lib/widgets/member_list.dart b/lib/widgets/member_list.dart
index 1639ea6..a3663be 100644
--- a/lib/widgets/member_list.dart
+++ b/lib/widgets/member_list.dart
@@ -125,7 +125,7 @@ class const MemberList(final String roomId, {super.key})
                                           ),
                                           leading: AvatarOrHash(
                                             avatarUrl,
-                                            height: 36,
+                                            dimension: 36,
                                             displayName ??
                                                 members[index]
                                                     .stateKey!
diff --git a/lib/widgets/room_appbar.dart b/lib/widgets/room_appbar.dart
index f9d9a06..4248177 100644
--- a/lib/widgets/room_appbar.dart
+++ b/lib/widgets/room_appbar.dart
@@ -46,7 +46,7 @@ final class const RoomAppbar({
                 : AvatarOrHash(
                     room.metadata?.avatar,
                     room.metadata?.name ?? "Unnamed Room",
-                    height: 24,
+                    dimension: 24,
                     fallback: Icon(Icons.numbers),
                   )
           : DrawerButton(onPressed: onOpenDrawer),
diff --git a/lib/widgets/room_list.dart b/lib/widgets/room_list.dart
index a9c5e67..552cd7b 100644
--- a/lib/widgets/room_list.dart
+++ b/lib/widgets/room_list.dart
@@ -159,7 +159,7 @@ class const RoomList({super.key})
                               },
                               short: true,
                               icon: AvatarOrHash(
-                                height: 28,
+                                dimension: 28,
                                 space.room?.metadata?.avatar,
                                 fallback: space.icon == null
                                     ? null
@@ -285,7 +285,7 @@ class const RoomList({super.key})
                                     subSpace.room.metadata?.avatar,
                                     subSpace.room.metadata?.name ??
                                         "Unnamed Room",
-                                    height: 16,
+                                    dimension: 16,
                                   ),
                                 Flexible(
                                   child: Text(
diff --git a/lib/widgets/room_summary_dialog.dart b/lib/widgets/room_summary_dialog.dart
index 5c7d302..b92f0d1 100644
--- a/lib/widgets/room_summary_dialog.dart
+++ b/lib/widgets/room_summary_dialog.dart
@@ -28,7 +28,7 @@ class const RoomSummaryDialog(
             child: AvatarOrHash(
               summary.avatarUrl,
               "",
-              height: 64,
+              dimension: 64,
               fallback: Icon(Icons.numbers),
             ),
           ),
diff --git a/lib/widgets/user_bottom_sheet.dart b/lib/widgets/user_bottom_sheet.dart
index 94cb50c..d7cc55b 100644
--- a/lib/widgets/user_bottom_sheet.dart
+++ b/lib/widgets/user_bottom_sheet.dart
@@ -101,7 +101,7 @@ final class const UserBottomSheet(
               child: AvatarOrHash(
                 member.avatarUrl,
                 member.displayName ?? userId.localpart,
-                height: 200,
+                dimension: 200,
               ),
             ),