Merge pull request #1433 from GetStream/fix/attachment-picker-mobile-web

This commit is contained in:
Sahil Kumar
2023-01-17 14:08:35 +05:30
committed by GitHub
2 changed files with 30 additions and 39 deletions
@@ -2,6 +2,7 @@
🐞 Fixed 🐞 Fixed
- [[#1424]](https://github.com/GetStream/stream-chat-flutter/issues/1424) Fixed a render issue when showing messages starting with 4 whitespaces. - [[#1424]](https://github.com/GetStream/stream-chat-flutter/issues/1424) Fixed a render issue when showing messages starting with 4 whitespaces.
- Fixed a bug where the `AttachmentPickerBottomSheet` was not able to identify the mobile browser.
## 5.2.0 ## 5.2.0
@@ -1,7 +1,7 @@
import 'dart:async'; import 'dart:async';
import 'package:flutter/foundation.dart' show kIsWeb, defaultTargetPlatform;
import 'package:flutter/material.dart'; import 'package:flutter/material.dart';
import 'package:stream_chat_flutter/platform_widget_builder/src/platform_widget.dart';
import 'package:stream_chat_flutter/stream_chat_flutter.dart'; import 'package:stream_chat_flutter/stream_chat_flutter.dart';
/// Shows a modal material design bottom sheet. /// Shows a modal material design bottom sheet.
@@ -107,8 +107,13 @@ Future<T?> showStreamAttachmentPickerModalBottomSheet<T>({
controller: controller, controller: controller,
initialAttachments: initialAttachments, initialAttachments: initialAttachments,
builder: (context, controller, child) { builder: (context, controller, child) {
return PlatformWidget( final currentPlatform = defaultTargetPlatform;
web: (context) { final isWebOrDesktop = kIsWeb ||
currentPlatform == TargetPlatform.macOS ||
currentPlatform == TargetPlatform.linux ||
currentPlatform == TargetPlatform.windows;
if (isWebOrDesktop) {
return webOrDesktopAttachmentPickerBuilder.call( return webOrDesktopAttachmentPickerBuilder.call(
context: context, context: context,
controller: controller, controller: controller,
@@ -120,8 +125,8 @@ Future<T?> showStreamAttachmentPickerModalBottomSheet<T>({
attachmentThumbnailQuality: attachmentThumbnailQuality, attachmentThumbnailQuality: attachmentThumbnailQuality,
attachmentThumbnailScale: attachmentThumbnailScale, attachmentThumbnailScale: attachmentThumbnailScale,
); );
}, }
mobile: (context) {
return mobileAttachmentPickerBuilder.call( return mobileAttachmentPickerBuilder.call(
context: context, context: context,
controller: controller, controller: controller,
@@ -132,21 +137,6 @@ Future<T?> showStreamAttachmentPickerModalBottomSheet<T>({
attachmentThumbnailScale: attachmentThumbnailScale, attachmentThumbnailScale: attachmentThumbnailScale,
); );
}, },
desktop: (context) {
return webOrDesktopAttachmentPickerBuilder.call(
context: context,
controller: controller,
customOptions: customOptions?.map(
WebOrDesktopAttachmentPickerOption.fromAttachmentPickerOption,
),
attachmentThumbnailSize: attachmentThumbnailSize,
attachmentThumbnailFormat: attachmentThumbnailFormat,
attachmentThumbnailQuality: attachmentThumbnailQuality,
attachmentThumbnailScale: attachmentThumbnailScale,
);
},
);
},
); );
}, },
); );