fix(YaruMasterTile): let onTap override scope ontap (#738)

ubuntu · Jul 27, 2023 · 9ac8623 · 9ac8623
1 parent 0cc96d8
commit 9ac8623
Showing 1 changed file with 8 additions and 3 deletions.
diff --git a/lib/src/widgets/master_detail/yaru_master_tile.dart b/lib/src/widgets/master_detail/yaru_master_tile.dart
@@ -35,13 +35,15 @@ class YaruMasterTile extends StatelessWidget {
   /// See [ListTile.trailing].
   final Widget? trailing;
 
-  /// See [ListTile.onTap].
+  /// An optional [VoidCallback] forwarded to the internal [ListTile]
+  /// If not provided [YaruMasterTileScope] `onTap` will be called.
   final VoidCallback? onTap;
 
   @override
   Widget build(BuildContext context) {
     final theme = Theme.of(context);
     final scope = YaruMasterTileScope.maybeOf(context);
+
     final isSelected = selected ?? scope?.selected ?? false;
     final scrollbarThicknessWithTrack =
         _calcScrollbarThicknessWithTrack(context);
@@ -76,8 +78,11 @@ class YaruMasterTile extends StatelessWidget {
           trailing: trailing,
           selected: isSelected,
           onTap: () {
-            scope?.onTap();
-            onTap?.call();
+            if (onTap != null) {
+              onTap!.call();
+            } else {
+              scope?.onTap();
+            }
           },
         ),
       ),