compose_box: Disable the compose box in DMs with deactivated users

Fixes: #675

Author: sm-sayedi

assets/l10n/app_en.arb

@@ -203,6 +203,13 @@
       "user": {"type": "String", "example": "channel name"}
     }
   },
+  "composeBoxDeactivatedDmContentHint": "You cannot send messages to {persons, plural, =1{a deactivated user} other{deactivated users}}.",
+  "@composeBoxDeactivatedDmContentHint": {
+    "description": "Hint text for content input when sending a message to one or multiple deactivated persons.",
+    "placeholders": {
+      "persons": {"type": "int", "example": "1"}
+    }
+  },
   "composeBoxGroupDmContentHint": "Message group",
   "@composeBoxGroupDmContentHint": {
     "description": "Hint text for content input when sending a message to a group."

lib/widgets/compose_box.dart

@@ -271,16 +271,21 @@ class _ContentInput extends StatelessWidget {
     required this.controller,
     required this.focusNode,
     required this.hintText,
+    this.enabled = true,
   });
 
   final Narrow narrow;
   final ComposeContentController controller;
   final FocusNode focusNode;
   final String hintText;
+  final bool enabled;
 
   @override
   Widget build(BuildContext context) {
     ColorScheme colorScheme = Theme.of(context).colorScheme;
+    if (!enabled) {
+      controller.clear();
+    }
 
     return InputDecorator(
       decoration: const InputDecoration(),
@@ -303,6 +308,7 @@ class _ContentInput extends StatelessWidget {
               decoration: InputDecoration.collapsed(hintText: hintText),
               maxLines: null,
               textCapitalization: TextCapitalization.sentences,
+              enabled: enabled,
             );
           }),
         ));
@@ -377,31 +383,36 @@ class _FixedDestinationContentInput extends StatelessWidget {
     required this.narrow,
     required this.controller,
     required this.focusNode,
+    required this.enabled,
   });
 
   final SendableNarrow narrow;
   final ComposeContentController controller;
   final FocusNode focusNode;
+  final bool enabled;
 
   String _hintText(BuildContext context) {
     final zulipLocalizations = ZulipLocalizations.of(context);
-    switch (narrow) {
-      case TopicNarrow(:final streamId, :final topic):
+    switch ((narrow, enabled)) {
+      case (TopicNarrow(:final streamId, :final topic), _):
         final store = PerAccountStoreWidget.of(context);
         final streamName = store.streams[streamId]?.name
           ?? zulipLocalizations.composeBoxUnknownChannelName;
         return zulipLocalizations.composeBoxChannelContentHint(streamName, topic);
 
-      case DmNarrow(otherRecipientIds: []): // The self-1:1 thread.
+      case (DmNarrow(otherRecipientIds: []), _): // The self-1:1 thread.
         return zulipLocalizations.composeBoxSelfDmContentHint;
 
-      case DmNarrow(otherRecipientIds: [final otherUserId]):
+      case (DmNarrow(otherRecipientIds: [final otherUserId]), true):
         final store = PerAccountStoreWidget.of(context);
         final fullName = store.users[otherUserId]?.fullName;
         if (fullName == null) return zulipLocalizations.composeBoxGenericContentHint;
         return zulipLocalizations.composeBoxDmContentHint(fullName);
 
-      case DmNarrow(): // A group DM thread.
+      case (DmNarrow(:final otherRecipientIds), false):
+        return zulipLocalizations.composeBoxDeactivatedDmContentHint(otherRecipientIds.length);
+
+      case (DmNarrow(), true): // A group DM thread.
         return zulipLocalizations.composeBoxGroupDmContentHint;
     }
   }
@@ -412,7 +423,8 @@ class _FixedDestinationContentInput extends StatelessWidget {
       narrow: narrow,
       controller: controller,
       focusNode: focusNode,
-      hintText: _hintText(context));
+      hintText: _hintText(context),
+      enabled: enabled);
   }
 }
 
@@ -492,10 +504,15 @@ Future<void> _uploadFiles({
 }
 
 abstract class _AttachUploadsButton extends StatelessWidget {
-  const _AttachUploadsButton({required this.contentController, required this.contentFocusNode});
+  const _AttachUploadsButton({
+    required this.contentController,
+    required this.contentFocusNode,
+    required this.enabled,
+  });
 
   final ComposeContentController contentController;
   final FocusNode contentFocusNode;
+  final bool enabled;
 
   IconData get icon;
   String tooltip(ZulipLocalizations zulipLocalizations);
@@ -534,7 +551,7 @@ abstract class _AttachUploadsButton extends StatelessWidget {
     return IconButton(
       icon: Icon(icon),
       tooltip: tooltip(zulipLocalizations),
-      onPressed: () => _handlePress(context));
+      onPressed: enabled ? () => _handlePress(context) : null);
   }
 }
 
@@ -578,7 +595,11 @@ Future<Iterable<_File>> _getFilePickerFiles(BuildContext context, FileType type)
 }
 
 class _AttachFileButton extends _AttachUploadsButton {
-  const _AttachFileButton({required super.contentController, required super.contentFocusNode});
+  const _AttachFileButton({
+    required super.contentController,
+    required super.contentFocusNode,
+    required super.enabled,
+  });
 
   @override
   IconData get icon => Icons.attach_file;
@@ -594,7 +615,11 @@ class _AttachFileButton extends _AttachUploadsButton {
 }
 
 class _AttachMediaButton extends _AttachUploadsButton {
-  const _AttachMediaButton({required super.contentController, required super.contentFocusNode});
+  const _AttachMediaButton({
+    required super.contentController,
+    required super.contentFocusNode,
+    required super.enabled,
+  });
 
   @override
   IconData get icon => Icons.image;
@@ -611,7 +636,11 @@ class _AttachMediaButton extends _AttachUploadsButton {
 }
 
 class _AttachFromCameraButton extends _AttachUploadsButton {
-  const _AttachFromCameraButton({required super.contentController, required super.contentFocusNode});
+  const _AttachFromCameraButton({
+    required super.contentController,
+    required super.contentFocusNode,
+    required super.enabled,
+  });
 
   @override
   IconData get icon => Icons.camera_alt;
@@ -667,11 +696,13 @@ class _SendButton extends StatefulWidget {
     required this.topicController,
     required this.contentController,
     required this.getDestination,
+    this.enabled = true,
   });
 
   final ComposeTopicController? topicController;
   final ComposeContentController contentController;
   final MessageDestination Function() getDestination;
+  final bool enabled;
 
   @override
   State<_SendButton> createState() => _SendButtonState();
@@ -776,7 +807,7 @@ class _SendButtonState extends State<_SendButton> {
         ),
         color: foregroundColor,
         icon: const Icon(Icons.send),
-        onPressed: _send));
+        onPressed: widget.enabled ? _send : null));
   }
 }
 
@@ -787,13 +818,15 @@ class _ComposeBoxLayout extends StatelessWidget {
     required this.sendButton,
     required this.contentController,
     required this.contentFocusNode,
+    this.enabled = true,
   });
 
   final Widget? topicInput;
   final Widget contentInput;
   final Widget sendButton;
   final ComposeContentController contentController;
   final FocusNode contentFocusNode;
+  final bool enabled;
 
   @override
   Widget build(BuildContext context) {
@@ -837,9 +870,21 @@ class _ComposeBoxLayout extends StatelessWidget {
               data: themeData.copyWith(
                 iconTheme: themeData.iconTheme.copyWith(color: colorScheme.onSurfaceVariant)),
               child: Row(children: [
-                _AttachFileButton(contentController: contentController, contentFocusNode: contentFocusNode),
-                _AttachMediaButton(contentController: contentController, contentFocusNode: contentFocusNode),
-                _AttachFromCameraButton(contentController: contentController, contentFocusNode: contentFocusNode),
+                        _AttachFileButton(
+                          contentController: contentController,
+                          contentFocusNode: contentFocusNode,
+                          enabled: enabled,
+                        ),
+                        _AttachMediaButton(
+                          contentController: contentController,
+                          contentFocusNode: contentFocusNode,
+                          enabled: enabled,
+                        ),
+                        _AttachFromCameraButton(
+                          contentController: contentController,
+                          contentFocusNode: contentFocusNode,
+                          enabled: enabled,
+                        ),
               ])),
           ]))));  }
 }
@@ -937,19 +982,28 @@ class _FixedDestinationComposeBoxState extends State<_FixedDestinationComposeBox
 
   @override
   Widget build(BuildContext context) {
+    final store = PerAccountStoreWidget.of(context);
+    final bool enabled = switch (widget.narrow) {
+      DmNarrow(:final otherRecipientIds) => otherRecipientIds.every((id) =>
+        store.users[id]?.isActive ?? true),
+      TopicNarrow() => true,
+    };
     return _ComposeBoxLayout(
       contentController: _contentController,
       contentFocusNode: _contentFocusNode,
       topicInput: null,
+      enabled: enabled,
       contentInput: _FixedDestinationContentInput(
         narrow: widget.narrow,
         controller: _contentController,
         focusNode: _contentFocusNode,
+        enabled: enabled,
       ),
       sendButton: _SendButton(
         topicController: null,
         contentController: _contentController,
         getDestination: () => widget.narrow.destination,
+        enabled: enabled,
       ));
   }
 }