Use CupertinoButton for thumbnails

moffatman · Nov 19, 2022 · a6594f5 · a6594f5
1 parent ea43bbc
commit a6594f5
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 7 deletions.
diff --git a/lib/widgets/post_row.dart b/lib/widgets/post_row.dart
@@ -172,7 +172,9 @@ class PostRow extends StatelessWidget {
 						mainAxisSize: MainAxisSize.min,
 						children: latestPost.attachments.map((attachment) => PopupAttachment(
 							attachment: attachment,
-							child: GestureDetector(
+							child: CupertinoButton(
+								padding: EdgeInsets.zero,
+								minSize: 0,
 								child: Stack(
 									alignment: Alignment.center,
 									fit: StackFit.loose,
@@ -211,7 +213,7 @@ class PostRow extends StatelessWidget {
 										)
 									]
 								),
-								onTap: () {
+								onPressed: () {
 									onThumbnailTap?.call(attachment);
 								}
 							)
@@ -222,9 +224,8 @@ class PostRow extends StatelessWidget {
 					child: SizedBox(
 						width: 75,
 						height: 75,
-						child: GestureDetector(
-							behavior: HitTestBehavior.opaque,
-							onTap: onRequestArchive,
+						child: CupertinoButton(
+							onPressed: onRequestArchive,
 							child: const Icon(CupertinoIcons.question_square, size: 36)
 						)
 					)

diff --git a/lib/widgets/thread_row.dart b/lib/widgets/thread_row.dart
@@ -272,7 +272,9 @@ class ThreadRow extends StatelessWidget {
 						mainAxisSize: MainAxisSize.min,
 						children: latestThread.attachments.map((attachment) => PopupAttachment(
 							attachment: attachment,
-							child: GestureDetector(
+							child: CupertinoButton(
+								padding: EdgeInsets.zero,
+								minSize: 0,
 								child: Stack(
 									alignment: Alignment.center,
 									fit: StackFit.loose,
@@ -311,7 +313,7 @@ class ThreadRow extends StatelessWidget {
 										)
 									]
 								),
-								onTap: () => onThumbnailTap?.call(attachment)
+								onPressed: () => onThumbnailTap?.call(attachment)
 							)
 						)).expand((x) => [const SizedBox(height: 8), x]).skip(1).toList()
 					)