fix: rounded button shadow and update assets (#604)

flutter · May 13, 2021 · 4fb1f43 · 4fb1f43
1 parent 5de6611
commit 4fb1f43
Show file tree

Hide file tree

Showing 7 changed files with 9 additions and 6 deletions.
diff --git a/assets/icons/delete_icon.png b/assets/icons/delete_icon.png
diff --git a/assets/icons/retake_button_icon.png b/assets/icons/retake_button_icon.png
diff --git a/assets/icons/stickers_button_icon.png b/assets/icons/stickers_button_icon.png
diff --git a/lib/share/view/share_page.dart b/lib/share/view/share_page.dart
@@ -85,8 +85,9 @@ class _ShareRetakeButton extends StatelessWidget {
             );
           }
         },
+        verticalOffset: 50,
         message: l10n.retakeButtonTooltip,
-        child: Image.asset('assets/icons/retake_button_icon.png', height: 50),
+        child: Image.asset('assets/icons/retake_button_icon.png', height: 100),
       ),
     );
   }

diff --git a/lib/stickers/view/stickers_page.dart b/lib/stickers/view/stickers_page.dart
@@ -66,7 +66,6 @@ class StickersView extends StatelessWidget {
                     child: Row(
                       children: [
                         const _RetakeButton(),
-                        const SizedBox(width: 15),
                         const ClearStickersButtonLayer(),
                       ],
                     ),
@@ -120,7 +119,6 @@ class _StickerReminderText extends StatelessWidget {
         key: const Key('stickersPage_propsReminder_appTooltip'),
         visible: true,
         message: context.l10n.propsReminderText,
-        padding: const EdgeInsets.all(24),
       ),
     );
   }
@@ -192,8 +190,9 @@ class _RetakeButton extends StatelessWidget {
           );
         }
       },
+      verticalOffset: 50,
       message: l10n.retakeButtonTooltip,
-      child: Image.asset('assets/icons/retake_button_icon.png', height: 50),
+      child: Image.asset('assets/icons/retake_button_icon.png', height: 100),
     );
   }
 }
@@ -444,7 +443,7 @@ class _OpenStickersButtonState extends State<OpenStickersButton> {
       message: l10n.openStickersTooltip,
       verticalOffset: 50,
       mode: TooltipMode.normal,
-      child: Image.asset('assets/icons/stickers_button_icon.png', height: 80),
+      child: Image.asset('assets/icons/stickers_button_icon.png', height: 100),
     );
     return _isAnimating ? AnimatedPulse(child: child) : child;
   }

diff --git a/lib/stickers/widgets/clear_stickers/clear_stickers_button_layer.dart b/lib/stickers/widgets/clear_stickers/clear_stickers_button_layer.dart
@@ -42,7 +42,8 @@ class ClearStickersButton extends StatelessWidget {
     return AppTooltipButton(
       onPressed: onPressed,
       message: l10n.clearStickersButtonTooltip,
-      child: Image.asset('assets/icons/delete_icon.png', height: 50),
+      verticalOffset: 50,
+      child: Image.asset('assets/icons/delete_icon.png', height: 100),
     );
   }
 }
diff --git a/packages/photobooth_ui/lib/src/widgets/app_tooltip_button.dart b/packages/photobooth_ui/lib/src/widgets/app_tooltip_button.dart
@@ -63,6 +63,8 @@ class _AppTooltipButtonState extends State<AppTooltipButton> {
     );
     return Material(
       color: PhotoboothColors.transparent,
+      shape: const CircleBorder(),
+      clipBehavior: Clip.hardEdge,
       child: AppTooltip.custom(
         visible: _isTooltipVisible,
         message: widget.message,