Skip to content

Commit 4844404

Browse files
sm-sayedism-sayedi
committed
action-sheet: Move the "Share" option to the bottom of options
This is because in the next commit(s) the "Copy link to message" button will be added and we want to make it next to the "Share" button at the bottom of the options.
1 parent b832f0c commit 4844404

File tree

2 files changed

+122
-122
lines changed

2 files changed

+122
-122
lines changed

‎lib/widgets/action_sheet.dart‎

Lines changed: 58 additions & 58 deletions
Original file line numberDiff line numberDiff line change
@@ -39,12 +39,12 @@ void showMessageActionSheet({required BuildContext context, required Message mes
3939
return Column(children: [
4040
if (!hasThumbsUpReactionVote) AddThumbsUpButton(message: message, messageListContext: context),
4141
StarButton(message: message, messageListContext: context),
42-
ShareButton(message: message, messageListContext: context),
4342
if (isComposeBoxOffered) QuoteAndReplyButton(
4443
message: message,
4544
messageListContext: context,
4645
),
4746
CopyMessageTextButton(message: message, messageListContext: context),
47+
ShareButton(message: message, messageListContext: context),
4848
]);
4949
});
5050
}
@@ -164,63 +164,6 @@ class StarButton extends MessageActionSheetMenuItemButton {
164164
}
165165
}
166166

167-
class ShareButton extends MessageActionSheetMenuItemButton {
168-
ShareButton({
169-
super.key,
170-
required super.message,
171-
required super.messageListContext,
172-
});
173-
174-
@override IconData get icon => Icons.adaptive.share;
175-
176-
@override
177-
String label(ZulipLocalizations zulipLocalizations) {
178-
return zulipLocalizations.actionSheetOptionShare;
179-
}
180-
181-
@override void onPressed(BuildContext context) async {
182-
// Close the message action sheet; we're about to show the share
183-
// sheet. (We could do this after the sharing Future settles
184-
// with [ShareResultStatus.success], but on iOS I get impatient with
185-
// how slowly our action sheet dismisses in that case.)
186-
// TODO(#24): Fix iOS bug where this call causes the keyboard to
187-
// reopen (if it was open at the time of this
188-
// `showMessageActionSheet` call) and cover a large part of the
189-
// share sheet.
190-
Navigator.of(context).pop();
191-
final zulipLocalizations = ZulipLocalizations.of(messageListContext);
192-
193-
final rawContent = await fetchRawContentWithFeedback(
194-
context: messageListContext,
195-
messageId: message.id,
196-
errorDialogTitle: zulipLocalizations.errorSharingFailed,
197-
);
198-
199-
if (rawContent == null) return;
200-
201-
if (!messageListContext.mounted) return;
202-
203-
// TODO: to support iPads, we're asked to give a
204-
// `sharePositionOrigin` param, or risk crashing / hanging:
205-
// https://pub.dev/packages/share_plus#ipad
206-
// Perhaps a wart in the API; discussion:
207-
// https://github.com/zulip/zulip-flutter/pull/12#discussion_r1130146231
208-
final result = await Share.share(rawContent);
209-
210-
switch (result.status) {
211-
// The plugin isn't very helpful: "The status can not be determined".
212-
// Until we learn otherwise, assume something wrong happened.
213-
case ShareResultStatus.unavailable:
214-
if (!messageListContext.mounted) return;
215-
await showErrorDialog(context: messageListContext,
216-
title: zulipLocalizations.errorSharingFailed);
217-
case ShareResultStatus.success:
218-
case ShareResultStatus.dismissed:
219-
// nothing to do
220-
}
221-
}
222-
}
223-
224167
/// Fetch and return the raw Markdown content for [messageId],
225168
/// showing an error dialog on failure.
226169
Future<String?> fetchRawContentWithFeedback({
@@ -366,3 +309,60 @@ class CopyMessageTextButton extends MessageActionSheetMenuItemButton {
366309
data: ClipboardData(text: rawContent));
367310
}
368311
}
312+
313+
class ShareButton extends MessageActionSheetMenuItemButton {
314+
ShareButton({
315+
super.key,
316+
required super.message,
317+
required super.messageListContext,
318+
});
319+
320+
@override IconData get icon => Icons.adaptive.share;
321+
322+
@override
323+
String label(ZulipLocalizations zulipLocalizations) {
324+
return zulipLocalizations.actionSheetOptionShare;
325+
}
326+
327+
@override void onPressed(BuildContext context) async {
328+
// Close the message action sheet; we're about to show the share
329+
// sheet. (We could do this after the sharing Future settles
330+
// with [ShareResultStatus.success], but on iOS I get impatient with
331+
// how slowly our action sheet dismisses in that case.)
332+
// TODO(#24): Fix iOS bug where this call causes the keyboard to
333+
// reopen (if it was open at the time of this
334+
// `showMessageActionSheet` call) and cover a large part of the
335+
// share sheet.
336+
Navigator.of(context).pop();
337+
final zulipLocalizations = ZulipLocalizations.of(messageListContext);
338+
339+
final rawContent = await fetchRawContentWithFeedback(
340+
context: messageListContext,
341+
messageId: message.id,
342+
errorDialogTitle: zulipLocalizations.errorSharingFailed,
343+
);
344+
345+
if (rawContent == null) return;
346+
347+
if (!messageListContext.mounted) return;
348+
349+
// TODO: to support iPads, we're asked to give a
350+
// `sharePositionOrigin` param, or risk crashing / hanging:
351+
// https://pub.dev/packages/share_plus#ipad
352+
// Perhaps a wart in the API; discussion:
353+
// https://github.com/zulip/zulip-flutter/pull/12#discussion_r1130146231
354+
final result = await Share.share(rawContent);
355+
356+
switch (result.status) {
357+
// The plugin isn't very helpful: "The status can not be determined".
358+
// Until we learn otherwise, assume something wrong happened.
359+
case ShareResultStatus.unavailable:
360+
if (!messageListContext.mounted) return;
361+
await showErrorDialog(context: messageListContext,
362+
title: zulipLocalizations.errorSharingFailed);
363+
case ShareResultStatus.success:
364+
case ShareResultStatus.dismissed:
365+
// nothing to do
366+
}
367+
}
368+
}

‎test/widgets/action_sheet_test.dart‎

Lines changed: 64 additions & 64 deletions
Original file line numberDiff line numberDiff line change
@@ -246,70 +246,6 @@ void main() {
246246
});
247247
});
248248

249-
group('ShareButton', () {
250-
// Tests should call this.
251-
MockSharePlus setupMockSharePlus() {
252-
final mock = MockSharePlus();
253-
TestDefaultBinaryMessengerBinding.instance.defaultBinaryMessenger.setMockMethodCallHandler(
254-
MethodChannelShare.channel,
255-
mock.handleMethodCall,
256-
);
257-
return mock;
258-
}
259-
260-
Future<void> tapShareButton(WidgetTester tester) async {
261-
await tester.ensureVisible(find.byIcon(Icons.adaptive.share, skipOffstage: false));
262-
await tester.tap(find.byIcon(Icons.adaptive.share));
263-
await tester.pump(); // [MenuItemButton.onPressed] called in a post-frame callback: flutter/flutter@e4a39fa2e
264-
}
265-
266-
testWidgets('request succeeds; sharing succeeds', (WidgetTester tester) async {
267-
final mockSharePlus = setupMockSharePlus();
268-
final message = eg.streamMessage();
269-
await setupToMessageActionSheet(tester, message: message, narrow: TopicNarrow.ofMessage(message));
270-
final store = await testBinding.globalStore.perAccount(eg.selfAccount.id);
271-
272-
prepareRawContentResponseSuccess(store, message: message, rawContent: 'Hello world');
273-
await tapShareButton(tester);
274-
await tester.pump(Duration.zero);
275-
check(mockSharePlus.sharedString).equals('Hello world');
276-
});
277-
278-
testWidgets('request succeeds; sharing fails', (WidgetTester tester) async {
279-
final mockSharePlus = setupMockSharePlus();
280-
final message = eg.streamMessage();
281-
await setupToMessageActionSheet(tester, message: message, narrow: TopicNarrow.ofMessage(message));
282-
final store = await testBinding.globalStore.perAccount(eg.selfAccount.id);
283-
284-
prepareRawContentResponseSuccess(store, message: message, rawContent: 'Hello world');
285-
mockSharePlus.resultString = 'dev.fluttercommunity.plus/share/unavailable';
286-
await tapShareButton(tester);
287-
await tester.pump(Duration.zero);
288-
check(mockSharePlus.sharedString).equals('Hello world');
289-
await tester.pump();
290-
await tester.tap(find.byWidget(checkErrorDialog(tester,
291-
expectedTitle: 'Sharing failed')));
292-
});
293-
294-
testWidgets('request has an error', (WidgetTester tester) async {
295-
final mockSharePlus = setupMockSharePlus();
296-
final message = eg.streamMessage();
297-
await setupToMessageActionSheet(tester, message: message, narrow: TopicNarrow.ofMessage(message));
298-
final store = await testBinding.globalStore.perAccount(eg.selfAccount.id);
299-
300-
prepareRawContentResponseError(store);
301-
await tapShareButton(tester);
302-
await tester.pump(Duration.zero); // error arrives; error dialog shows
303-
304-
await tester.tap(find.byWidget(checkErrorDialog(tester,
305-
expectedTitle: 'Sharing failed',
306-
expectedMessage: 'That message does not seem to exist.',
307-
)));
308-
309-
check(mockSharePlus.sharedString).isNull();
310-
});
311-
});
312-
313249
group('QuoteAndReplyButton', () {
314250
ComposeBoxController? findComposeBoxController(WidgetTester tester) {
315251
return tester.widget<ComposeBox>(find.byType(ComposeBox))
@@ -502,4 +438,68 @@ void main() {
502438
check(await Clipboard.getData('text/plain')).isNull();
503439
});
504440
});
441+
442+
group('ShareButton', () {
443+
// Tests should call this.
444+
MockSharePlus setupMockSharePlus() {
445+
final mock = MockSharePlus();
446+
TestDefaultBinaryMessengerBinding.instance.defaultBinaryMessenger.setMockMethodCallHandler(
447+
MethodChannelShare.channel,
448+
mock.handleMethodCall,
449+
);
450+
return mock;
451+
}
452+
453+
Future<void> tapShareButton(WidgetTester tester) async {
454+
await tester.ensureVisible(find.byIcon(Icons.adaptive.share, skipOffstage: false));
455+
await tester.tap(find.byIcon(Icons.adaptive.share));
456+
await tester.pump(); // [MenuItemButton.onPressed] called in a post-frame callback: flutter/flutter@e4a39fa2e
457+
}
458+
459+
testWidgets('request succeeds; sharing succeeds', (WidgetTester tester) async {
460+
final mockSharePlus = setupMockSharePlus();
461+
final message = eg.streamMessage();
462+
await setupToMessageActionSheet(tester, message: message, narrow: TopicNarrow.ofMessage(message));
463+
final store = await testBinding.globalStore.perAccount(eg.selfAccount.id);
464+
465+
prepareRawContentResponseSuccess(store, message: message, rawContent: 'Hello world');
466+
await tapShareButton(tester);
467+
await tester.pump(Duration.zero);
468+
check(mockSharePlus.sharedString).equals('Hello world');
469+
});
470+
471+
testWidgets('request succeeds; sharing fails', (WidgetTester tester) async {
472+
final mockSharePlus = setupMockSharePlus();
473+
final message = eg.streamMessage();
474+
await setupToMessageActionSheet(tester, message: message, narrow: TopicNarrow.ofMessage(message));
475+
final store = await testBinding.globalStore.perAccount(eg.selfAccount.id);
476+
477+
prepareRawContentResponseSuccess(store, message: message, rawContent: 'Hello world');
478+
mockSharePlus.resultString = 'dev.fluttercommunity.plus/share/unavailable';
479+
await tapShareButton(tester);
480+
await tester.pump(Duration.zero);
481+
check(mockSharePlus.sharedString).equals('Hello world');
482+
await tester.pump();
483+
await tester.tap(find.byWidget(checkErrorDialog(tester,
484+
expectedTitle: 'Sharing failed')));
485+
});
486+
487+
testWidgets('request has an error', (WidgetTester tester) async {
488+
final mockSharePlus = setupMockSharePlus();
489+
final message = eg.streamMessage();
490+
await setupToMessageActionSheet(tester, message: message, narrow: TopicNarrow.ofMessage(message));
491+
final store = await testBinding.globalStore.perAccount(eg.selfAccount.id);
492+
493+
prepareRawContentResponseError(store);
494+
await tapShareButton(tester);
495+
await tester.pump(Duration.zero); // error arrives; error dialog shows
496+
497+
await tester.tap(find.byWidget(checkErrorDialog(tester,
498+
expectedTitle: 'Sharing failed',
499+
expectedMessage: 'That message does not seem to exist.',
500+
)));
501+
502+
check(mockSharePlus.sharedString).isNull();
503+
});
504+
});
505505
}

0 commit comments

Comments
 (0)