@@ -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