views: Display users who've read the message in a sorted list.

Display users who've read the message in a sorted list.

Author: Sushmey

zulipterminal/ui_tools/views.py

@@ -2001,33 +2001,55 @@ def __init__(
 
         # Get read receipt user ids list
         response = controller.model.fetch_read_receipts(message["id"])
-        if response is None:
-            response = [
-                ("msg_bold", "RESPONSE CANNOT BE FETCHED AT THE MOMENT. "),
-                "\n\nPLEASE TRY AGAIN LATER",
-            ]
-            max_cols = 50
-        elif response == "unsupported_ZFL":
-            response = [
+        if response is None or response == "unsupported_ZFL":
+            if response is None:
+                response = [
+                    ("msg_bold", "RESPONSE CANNOT BE FETCHED AT THE MOMENT.\n\n"),
+                    (None, "PLEASE TRY AGAIN LATER"),
+                ]
+                max_cols = 50
+            elif response == "unsupported_ZFL":
+                response = [
+                    (
+                        "msg_bold",
+                        "YOUR ZT VERSION DOESN'T SUPPORT VIEWING READ RECEIPTS.\n\n",
+                    ),
+                    (None, "PLEASE UPDATE AND TRY AGAIN LATER"),
+                ]
+                max_cols = 69
+
+            body_list = [urwid.Text(response, "center", "space")]
+
+            super().__init__(
+                controller,
+                body_list,
+                "MSG_INFO",
+                max_cols,
+                title,
+            )
+        else:
+            user_names = [controller.model.user_name_from_id(id) for id in response]
+            sorted_user_names = sorted(user_names)
+            msg_read_list = [("", [(name, "") for name in sorted_user_names])]
+            msg_read_list.insert(
+                0,
                 (
-                    "msg_bold",
-                    "YOUR CURRENT ZT VERSION DOESN'T SUPPORT VIEWING READ RECEIPTS. ",
+                    "",
+                    [
+                        (
+                            f"This message has been read by {len(user_names)} people.",
+                            "",
+                        ),
+                        ("", ""),
+                    ],
                 ),
-                "\n\nPLEASE UPDATE AND TRY AGAIN LATER",
-            ]
-            max_cols = 70
-        else:
-            response = str(list(map(controller.model.user_name_from_id, response)))
-
-        body_list = [urwid.Text(response)]
+            )
+            popup_width, column_width = self.calculate_table_widths(
+                msg_read_list, len(title)
+            )
+            widgets = self.make_table_with_categories(msg_read_list, column_width)
 
-        super().__init__(
-            controller,
-            body_list,
-            "MSG_INFO",
-            max_cols,
-            title,
-        )
+            super().__init__(controller, widgets, "MSG_INFO", popup_width, title)
 
     def keypress(self, size: urwid_Size, key: str) -> str:
         if is_command_key("GO_BACK", key) or is_command_key("READ_RECEIPTS", key):