🐛 Fix view assets with only selected

fluttercandies · Feb 4, 2024 · 7609453 · 7609453
1 parent 9a8c4c1
commit 7609453
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 3 deletions.
diff --git a/example/lib/customs/pickers/insta_asset_picker.dart b/example/lib/customs/pickers/insta_asset_picker.dart
@@ -378,9 +378,12 @@ class InstaAssetPickerBuilder extends DefaultAssetPickerBuilderDelegate {
   @override
   Future<void> viewAsset(
     BuildContext context,
-    int index,
+    int? index,
     AssetEntity currentAsset,
   ) async {
+    if (index == null) {
+      return;
+    }
     // if is preview asset, unselect it
     if (provider.selectedAssets.isNotEmpty &&
         _previewAsset.value == currentAsset) {

diff --git a/lib/src/delegates/asset_picker_builder_delegate.dart b/lib/src/delegates/asset_picker_builder_delegate.dart
@@ -870,7 +870,7 @@ class DefaultAssetPickerBuilderDelegate
   @override
   Future<void> viewAsset(
     BuildContext context,
-    int index,
+    int? index,
     AssetEntity currentAsset,
   ) async {
     final DefaultAssetPickerProvider provider =
@@ -902,6 +902,10 @@ class DefaultAssetPickerBuilderDelegate
         selected = provider.selectedAssets;
         effectiveIndex = current.indexOf(currentAsset);
       }
+    } else if (index == null) {
+      current = provider.selectedAssets;
+      selected = provider.selectedAssets;
+      effectiveIndex = selected.indexOf(currentAsset);
     } else {
       current = provider.currentAssets;
       selected = provider.selectedAssets;
@@ -1936,7 +1940,7 @@ class DefaultAssetPickerBuilderDelegate
       child: Consumer<DefaultAssetPickerProvider>(
         builder: (context, DefaultAssetPickerProvider p, __) => GestureDetector(
           onTap: p.isSelectedNotEmpty
-              ? () => viewAsset(context, 0, p.selectedAssets.first)
+              ? () => viewAsset(context, null, p.selectedAssets.first)
               : null,
           child: Selector<DefaultAssetPickerProvider, String>(
             selector: (_, DefaultAssetPickerProvider p) =>