Skip to content

Commit f103128

Browse files
committed
action_sheet [nfc]: Tighten button constructors onto one line
This is a nice small bonus from renaming the parameter that was `messageListContext` to the shorter name `pageContext`. Some of these don't quite fit in 80 columns. But the details of the parameter list are boring (because they're all the same), so it's OK that some of them go a little over.
1 parent 2cf1f6d commit f103128

File tree

1 file changed

+7
-35
lines changed

1 file changed

+7
-35
lines changed

lib/widgets/action_sheet.dart

Lines changed: 7 additions & 35 deletions
Original file line numberDiff line numberDiff line change
@@ -183,11 +183,7 @@ class MessageActionSheetCancelButton extends StatelessWidget {
183183
// This button is very temporary, to complete #125 before we have a way to
184184
// choose an arbitrary reaction (#388). So, skipping i18n.
185185
class AddThumbsUpButton extends MessageActionSheetMenuItemButton {
186-
AddThumbsUpButton({
187-
super.key,
188-
required super.message,
189-
required super.pageContext,
190-
});
186+
AddThumbsUpButton({super.key, required super.message, required super.pageContext});
191187

192188
@override IconData get icon => ZulipIcons.smile;
193189

@@ -223,11 +219,7 @@ class AddThumbsUpButton extends MessageActionSheetMenuItemButton {
223219
}
224220

225221
class StarButton extends MessageActionSheetMenuItemButton {
226-
StarButton({
227-
super.key,
228-
required super.message,
229-
required super.pageContext,
230-
});
222+
StarButton({super.key, required super.message, required super.pageContext});
231223

232224
@override IconData get icon => _isStarred ? ZulipIcons.star_filled : ZulipIcons.star;
233225

@@ -320,11 +312,7 @@ Future<String?> fetchRawContentWithFeedback({
320312
}
321313

322314
class QuoteAndReplyButton extends MessageActionSheetMenuItemButton {
323-
QuoteAndReplyButton({
324-
super.key,
325-
required super.message,
326-
required super.pageContext,
327-
});
315+
QuoteAndReplyButton({super.key, required super.message, required super.pageContext});
328316

329317
@override IconData get icon => ZulipIcons.format_quote;
330318

@@ -380,11 +368,7 @@ class QuoteAndReplyButton extends MessageActionSheetMenuItemButton {
380368
}
381369

382370
class MarkAsUnreadButton extends MessageActionSheetMenuItemButton {
383-
MarkAsUnreadButton({
384-
super.key,
385-
required super.message,
386-
required super.pageContext,
387-
});
371+
MarkAsUnreadButton({super.key, required super.message, required super.pageContext});
388372

389373
@override IconData get icon => Icons.mark_chat_unread_outlined;
390374

@@ -400,11 +384,7 @@ class MarkAsUnreadButton extends MessageActionSheetMenuItemButton {
400384
}
401385

402386
class CopyMessageTextButton extends MessageActionSheetMenuItemButton {
403-
CopyMessageTextButton({
404-
super.key,
405-
required super.message,
406-
required super.pageContext,
407-
});
387+
CopyMessageTextButton({super.key, required super.message, required super.pageContext});
408388

409389
@override IconData get icon => ZulipIcons.copy;
410390

@@ -437,11 +417,7 @@ class CopyMessageTextButton extends MessageActionSheetMenuItemButton {
437417
}
438418

439419
class CopyMessageLinkButton extends MessageActionSheetMenuItemButton {
440-
CopyMessageLinkButton({
441-
super.key,
442-
required super.message,
443-
required super.pageContext,
444-
});
420+
CopyMessageLinkButton({super.key, required super.message, required super.pageContext});
445421

446422
@override IconData get icon => Icons.link;
447423

@@ -467,11 +443,7 @@ class CopyMessageLinkButton extends MessageActionSheetMenuItemButton {
467443
}
468444

469445
class ShareButton extends MessageActionSheetMenuItemButton {
470-
ShareButton({
471-
super.key,
472-
required super.message,
473-
required super.pageContext,
474-
});
446+
ShareButton({super.key, required super.message, required super.pageContext});
475447

476448
@override
477449
IconData get icon => defaultTargetPlatform == TargetPlatform.iOS

0 commit comments

Comments
 (0)