fix: Top app bar padding (#215)

ZebraDevs · Nov 20, 2024 · 7b374b7 · 7b374b7
1 parent fd9aada
commit 7b374b7
Show file tree

Hide file tree

Showing 8 changed files with 20 additions and 17 deletions.
diff --git a/lib/src/components/top_app_bar/top_app_bar.dart b/lib/src/components/top_app_bar/top_app_bar.dart
@@ -329,23 +329,26 @@ class _ZetaTopAppBarState extends State<ZetaTopAppBar> {
 
     return ZetaRoundedScope(
       rounded: context.rounded,
-      child: Padding(
-        padding: EdgeInsets.symmetric(horizontal: spacing.minimum),
-        child: AppBar(
-          elevation: 0,
-          scrolledUnderElevation: 0,
-          backgroundColor: colors.surfacePrimary,
-          iconTheme: IconThemeData(color: colors.iconDefault),
-          leading: leading,
-          toolbarHeight: spacing.xl_9,
-          automaticallyImplyLeading: widget.automaticallyImplyLeading,
-          surfaceTintColor: Colors.transparent,
-          centerTitle: widget.type == ZetaTopAppBarType.centered,
-          titleTextStyle: widget.titleTextStyle == null
-              ? ZetaTextStyles.bodyLarge.copyWith(color: colors.textDefault)
-              : widget.titleTextStyle!.copyWith(color: colors.textDefault),
-          title: title,
-          actions: actions,
+      child: ColoredBox(
+        color: colors.surfaceDefault,
+        child: Padding(
+          padding: EdgeInsets.symmetric(horizontal: spacing.minimum),
+          child: AppBar(
+            elevation: 0,
+            scrolledUnderElevation: 0,
+            backgroundColor: colors.surfacePrimary,
+            iconTheme: IconThemeData(color: colors.iconDefault),
+            leading: leading,
+            toolbarHeight: spacing.xl_9,
+            automaticallyImplyLeading: widget.automaticallyImplyLeading,
+            surfaceTintColor: Colors.transparent,
+            centerTitle: widget.type == ZetaTopAppBarType.centered,
+            titleTextStyle: widget.titleTextStyle == null
+                ? ZetaTextStyles.bodyLarge.copyWith(color: colors.textDefault)
+                : widget.titleTextStyle!.copyWith(color: colors.textDefault),
+            title: title,
+            actions: actions,
+          ),
         ),
       ),
     );

diff --git a/test/src/components/top_app_bar/golden/top_app_bar_centered.png b/test/src/components/top_app_bar/golden/top_app_bar_centered.png
diff --git a/test/src/components/top_app_bar/golden/top_app_bar_centered_actions.png b/test/src/components/top_app_bar/golden/top_app_bar_centered_actions.png
diff --git a/test/src/components/top_app_bar/golden/top_app_bar_default.png b/test/src/components/top_app_bar/golden/top_app_bar_default.png
diff --git a/test/src/components/top_app_bar/golden/top_app_bar_default_actions.png b/test/src/components/top_app_bar/golden/top_app_bar_default_actions.png
diff --git a/test/src/components/top_app_bar/golden/top_app_bar_search.png b/test/src/components/top_app_bar/golden/top_app_bar_search.png
diff --git a/test/src/components/top_app_bar/golden/top_app_bar_search_active.png b/test/src/components/top_app_bar/golden/top_app_bar_search_active.png
diff --git a/test/src/components/top_app_bar/golden/top_app_bar_search_centered.png b/test/src/components/top_app_bar/golden/top_app_bar_search_centered.png