compose_box: Disable the compose box in DMs with deactivated users

Fixes: #675

Co-authored-by: Rajesh Malviya <[email protected]>

Author: sm-sayedi

assets/l10n/app_en.arb

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

lib/widgets/compose_box.dart

@@ -272,17 +272,18 @@ 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;
-
     return InputDecorator(
       decoration: const InputDecoration(),
       child: ConstrainedBox(
@@ -304,6 +305,7 @@ class _ContentInput extends StatelessWidget {
               decoration: InputDecoration.collapsed(hintText: hintText),
               maxLines: null,
               textCapitalization: TextCapitalization.sentences,
+              enabled: enabled,
             );
           }),
         ));
@@ -378,32 +380,37 @@ 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(), true): // A group DM thread.
         return zulipLocalizations.composeBoxGroupDmContentHint;
+
+      case (DmNarrow(), false):
+        return zulipLocalizations.composeBoxDeactivatedDmContentHint;
     }
   }
 
@@ -413,7 +420,8 @@ class _FixedDestinationContentInput extends StatelessWidget {
       narrow: narrow,
       controller: controller,
       focusNode: focusNode,
-      hintText: _hintText(context));
+      hintText: _hintText(context),
+      enabled: enabled);
   }
 }
 
@@ -493,10 +501,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);
@@ -535,7 +548,7 @@ abstract class _AttachUploadsButton extends StatelessWidget {
     return IconButton(
       icon: Icon(icon),
       tooltip: tooltip(zulipLocalizations),
-      onPressed: () => _handlePress(context));
+      onPressed: enabled ? () => _handlePress(context) : null);
   }
 }
 
@@ -579,7 +592,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;
@@ -595,7 +612,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;
@@ -612,7 +633,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;
@@ -668,11 +693,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();
@@ -792,7 +819,7 @@ class _SendButtonState extends State<_SendButton> {
         ),
         color: foregroundColor,
         icon: const Icon(Icons.send),
-        onPressed: _send));
+        onPressed: widget.enabled ? _send : null));
   }
 }
 
@@ -803,13 +830,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) {
@@ -853,9 +882,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,
+                ),
               ])),
           ]))));  }
 }
@@ -943,6 +984,20 @@ class _FixedDestinationComposeBoxState extends State<_FixedDestinationComposeBox
 
   @override FocusNode get contentFocusNode => _contentFocusNode;
   final _contentFocusNode = FocusNode();
+  late bool enabled;
+
+  @override
+  void didChangeDependencies() {
+    super.didChangeDependencies();
+    final store = PerAccountStoreWidget.of(context);
+    enabled = switch (widget.narrow) {
+      DmNarrow(:final otherRecipientIds) => otherRecipientIds.every((id) =>
+        store.users[id]?.isActive ?? true),
+      TopicNarrow() => true,
+    };
+
+    if (!enabled) _contentController.clear();
+  }
 
   @override
   void dispose() {
@@ -957,15 +1012,18 @@ class _FixedDestinationComposeBoxState extends State<_FixedDestinationComposeBox
       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,
       ));
   }
 }