From 77c9cfd0e71e343f55446f9c94693bc736fad56a Mon Sep 17 00:00:00 2001
From: Fabian Freund <github@fabi.online>
Date: Wed, 8 Jul 2026 07:18:16 +0200
Subject: [PATCH] move favicon closer to url bar

---
 .../widgets/browser_modules/app_bar_title.dart            | 3 +++
 .../browser/presentation/widgets/toolbar_button.dart      | 8 +++++++-
 2 files changed, 10 insertions(+), 1 deletion(-)

diff --git a/apps/weblibre/lib/features/geckoview/features/browser/presentation/widgets/browser_modules/app_bar_title.dart b/apps/weblibre/lib/features/geckoview/features/browser/presentation/widgets/browser_modules/app_bar_title.dart
index eb4a8f2f..a633187b 100644
--- a/apps/weblibre/lib/features/geckoview/features/browser/presentation/widgets/browser_modules/app_bar_title.dart
+++ b/apps/weblibre/lib/features/geckoview/features/browser/presentation/widgets/browser_modules/app_bar_title.dart
@@ -611,6 +611,9 @@ class RailAppBarTitleView extends StatelessWidget {
       children: [
         ToolbarButton(
           onTap: onSiteSettingsTap,
+          // Match the pinned extension icons' vertical spacing so the favicon
+          // sits equally close to the address field on both sides.
+          padding: const EdgeInsets.symmetric(horizontal: 8.0, vertical: 8.0),
           child: Stack(
             clipBehavior: Clip.none,
             children: [
diff --git a/apps/weblibre/lib/features/geckoview/features/browser/presentation/widgets/toolbar_button.dart b/apps/weblibre/lib/features/geckoview/features/browser/presentation/widgets/toolbar_button.dart
index 1e0396d2..8323f208 100644
--- a/apps/weblibre/lib/features/geckoview/features/browser/presentation/widgets/toolbar_button.dart
+++ b/apps/weblibre/lib/features/geckoview/features/browser/presentation/widgets/toolbar_button.dart
@@ -25,12 +25,18 @@ class ToolbarButton extends StatelessWidget {
   final VoidCallback? onDoubleTap;
   final VoidCallback? onLongPress;
 
+  /// Padding around the icon. Defaults to the horizontal-bar spacing; the
+  /// vertical rail passes a tighter value so the favicon sits as close to the
+  /// address field as the pinned extension icons do.
+  final EdgeInsetsGeometry padding;
+
   const ToolbarButton({
     super.key,
     required this.child,
     this.onTap,
     this.onDoubleTap,
     this.onLongPress,
+    this.padding = const EdgeInsets.symmetric(horizontal: 8.0, vertical: 15.0),
   });
 
   @override
@@ -43,7 +49,7 @@ class ToolbarButton extends StatelessWidget {
       onDoubleTap: onDoubleTap,
       onLongPress: onLongPress,
       child: Padding(
-        padding: const EdgeInsets.symmetric(horizontal: 8.0, vertical: 15.0),
+        padding: padding,
         child: IconTheme(
           data: IconThemeData(color: iconColor),
           child: child,