@droppii-org/chat-mobile 0.2.82 → 0.2.84

package/src/screens/chat-detail/legend/LegendChatMessage.tsx

@@ -47,6 +47,7 @@ interface LegendChatMessageProps {
   onLongPress?: () => void;
   customMessageItemFactory?: (m: MessageItem) => ReactNode;
   isHighlighted?: boolean;
+  isTimeRevealed?: boolean;
   /** Sender name above the bubble — first message of an incoming cluster. */
   showName?: boolean;
   /** Sender avatar beside the bubble — last message of an incoming cluster. */
@@ -87,6 +88,14 @@ const MEDIA_TYPES: number[] = [
 
 const SYSTEM_MESSAGE_TYPES: number[] = GROUP_NOTIFICATION_TYPES;
 
+// Tapping these types must keep opening the media/file as today — they never
+// participate in tap-to-reveal-time, so no Tap gesture is attached at all for
+// them (avoids racing their own internal touchables for the same tap).
+const TIME_REVEAL_EXCLUDED_TYPES: number[] = [
+  ...MEDIA_TYPES,
+  MessageType.FileMessage,
+];
+
 export const LegendChatMessage = memo(
   ({
     message,
@@ -99,6 +108,7 @@ export const LegendChatMessage = memo(
     onLongPress,
     customMessageItemFactory,
     isHighlighted,
+    isTimeRevealed,
     showName = false,
     showAvatar = false,
     showSenderGutter = false,
@@ -145,6 +155,9 @@ export const LegendChatMessage = memo(
       onPressUrl &&
       (message?.contentType === MessageType.UrlTextMessage ||
         message?.contentType === MessageType.AtTextMessage);
+    const canRevealTime = !TIME_REVEAL_EXCLUDED_TYPES.includes(
+      message?.contentType
+    );
 
     return (
       <KContainer.View style={messageStyles.wrapper} row alignItems>
@@ -176,7 +189,10 @@ export const LegendChatMessage = memo(
               {message.senderNickname}
             </KLabel.Text>
           )}
-          <LongPressWrapper onPress={handlePress} onLongPress={handleLongPress}>
+          <LongPressWrapper
+            onPress={canRevealTime ? handlePress : undefined}
+            onLongPress={handleLongPress}
+          >
             <MessageComponent
               message={message}
               isOutgoing={isOutgoing}
@@ -184,6 +200,7 @@ export const LegendChatMessage = memo(
               onMediaPress={hasMediaPress ? onMediaPress : undefined}
               onPressUrl={hasUrlPress ? onPressUrl : undefined}
               isHighlighted={isHighlighted}
+              isTimeRevealed={canRevealTime && isTimeRevealed}
               groupPosition={groupPosition}
             />
           </LongPressWrapper>

package/src/utils/legendListMessage.ts

@@ -48,9 +48,10 @@ export interface PrecomputedMessageData {
   messageId: string;
   dayStart: boolean;
   /**
-   * More than GROUP_TIME_GAP has passed since the previous message. The list
-   * marks these with a time label: the cluster break is real but, unlike a day
-   * change, nothing else on screen would show why the run stopped.
+   * More than DIVIDER_TIME_GAP has passed since the previous message on the
+   * same day. The list marks these with a time-only divider so a long pause
+   * is visible even when nothing else on screen (day change, sender change)
+   * would show it.
    */
   timeBreak: boolean;
   createdAt: number;
@@ -60,6 +61,14 @@ export interface PrecomputedMessageData {
 /** Two messages further apart than this never share a cluster. */
 const GROUP_TIME_GAP = 300_000; // 5 minutes
 
+/**
+ * Same-day gap that earns a standalone time divider (BR: 30 minutes) —
+ * intentionally separate from GROUP_TIME_GAP: a cluster can visually break on
+ * the shorter gap without a divider explaining it, since the spec's divider
+ * threshold and the grouping threshold are independent rules.
+ */
+const DIVIDER_TIME_GAP = 1_800_000; // 30 minutes
+
 /** Virtual sender for bot conversations — see getGroupKey. */
 const BOT_GROUP_KEY = '__bot__';
 
@@ -145,7 +154,7 @@ export const precomputeLegendListData = (
       dayStart: dayStarts[index]!,
       timeBreak:
         prevTimestamp !== undefined &&
-        createdAt - prevTimestamp > GROUP_TIME_GAP,
+        createdAt - prevTimestamp > DIVIDER_TIME_GAP,
       createdAt,
       groupPosition: 'solo',
     });