Skip to content

Commit 3b9366e

Browse files
committed
compose [nfc]: Remove obsoleted TODO(#720)s
Issue #720 is superseded by #1441, and these don't apply... I guess with the exception of a note on how a generic "x" button could be laid out, so we leave that.
1 parent 72ffbd8 commit 3b9366e

File tree

1 file changed

+4
-13
lines changed

1 file changed

+4
-13
lines changed

lib/widgets/compose_box.dart

Lines changed: 4 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -1290,9 +1290,6 @@ class _SendButtonState extends State<_SendButton> {
12901290
controller.content.clear();
12911291

12921292
try {
1293-
// TODO(#720) clear content input only on success response;
1294-
// while waiting, put input(s) and send button into a disabled
1295-
// "working on it" state (letting input text be selected for copying).
12961293
await store.sendMessage(destination: widget.getDestination(), content: content);
12971294
} on ApiRequestException catch (e) {
12981295
if (!mounted) return;
@@ -1384,7 +1381,6 @@ class _ComposeBoxContainer extends StatelessWidget {
13841381
border: Border(top: BorderSide(color: designVariables.borderBar)),
13851382
boxShadow: ComposeBoxTheme.of(context).boxShadow,
13861383
),
1387-
// TODO(#720) try a Stack for the overlaid linear progress indicator
13881384
child: Material(
13891385
color: designVariables.composeBoxBg,
13901386
child: Column(
@@ -1720,10 +1716,10 @@ class _ErrorBanner extends _Banner {
17201716

17211717
@override
17221718
Widget? buildTrailing(context) {
1723-
// TODO(#720) "x" button goes here.
1724-
// 24px square with 8px touchable padding in all directions?
1725-
// and `bool get padEnd => false`; see Figma:
1726-
// https://www.figma.com/design/1JTNtYo9memgW7vV6d0ygq/Zulip-Mobile?node-id=4031-17029&m=dev
1719+
// An "x" button can go here.
1720+
// 24px square with 8px touchable padding in all directions?
1721+
// and `bool get padEnd => false`; see Figma:
1722+
// https://www.figma.com/design/1JTNtYo9memgW7vV6d0ygq/Zulip-Mobile?node-id=4031-17029&m=dev
17271723
return null;
17281724
}
17291725
}
@@ -2061,11 +2057,6 @@ class _ComposeBoxState extends State<ComposeBox> with PerAccountStoreAwareStateM
20612057
}
20622058
}
20632059

2064-
// TODO(#720) dismissable message-send error, maybe something like:
2065-
// if (controller.sendMessageError.value != null) {
2066-
// errorBanner = _ErrorBanner(label:
2067-
// ZulipLocalizations.of(context).errorSendMessageTimeout);
2068-
// }
20692060
return ComposeBoxInheritedWidget.fromComposeBoxState(this,
20702061
child: _ComposeBoxContainer(body: body, banner: banner));
20712062
}

0 commit comments

Comments
 (0)