Ticket #6036 (closed enhancement: fixed)

Opened 7 years ago

Last modified 6 years ago

Chat should provide a visual cue between past and present conversations

Reported by: HoboPrimate Owned by: joe
Priority: normal Milestone: 8.2.0 (was Update.2)
Component: chat-activity Version:
Keywords: joyride-2263:+ chat-45:+ Cc: Eben, morgs
Action Needed: finalize Verified: yes
Deployments affected: Blocked By:
Blocking:

Description

I resumed a Chat from neighbourhood, and since I've been in it before, all previous conversation appears. It becomes confusing to know who is in the Chat and who isn't by only looking at the "HoboPrimate is here" messages, because some of them belong to the previous conversations. Chat should have some divider line or other cue to differentiate between logged conversations and the new messages.

Change History

  Changed 7 years ago by jg

  • milestone changed from Never Assigned to FutureFeatures

  Changed 7 years ago by jg

  • type changed from defect to enhancement

  Changed 6 years ago by Eben

This is a really nice idea, actually. As a first pass, I'd probably suggest some whitespace with a horizontal rule dividing the sections, and a date/timestamp beneath the rule to indicate when the next session started.

As a future addition to this, it would be pretty fantastic if each section had a disclosure triangle, such that they could be expanded and collapsed as desired. When resuming a chat, previous sections might all be collapsed by default, with only their dates and perhaps a snippet of the first message (and maybe total number of messages; also, perhaps the participants of the chat if there are more than two in the whole activity...) visible within the collapsed bar.

  Changed 6 years ago by morgs

  • owner changed from Collabora to morgs

  Changed 6 years ago by morgs

  • status changed from new to assigned

Chat-41 includes a first pass at the whitespace with timestamp between historic conversations.

  Changed 6 years ago by gregorio

  • milestone deleted

Milestone FutureFeatures deleted

  Changed 6 years ago by morgs

  • next_action set to test in build

Chat-42 has some improvements - in particular the timestamp is shown as an elapsed time (1 day ago) instead of a date/time.

  Changed 6 years ago by morgs

  • keywords joyride-2153:+ added
  • next_action changed from test in build to qa signoff

Tested Chat-42 in joyride-2153.

  Changed 6 years ago by morgs

  • milestone set to 8.2.0 (was Update.2)

  Changed 6 years ago by morgs

  • cc morgs added
  • owner changed from morgs to joe
  • status changed from assigned to new

  Changed 6 years ago by joe

  • owner changed from joe to Charlie

Charlie,

Please check that enhancement.

Joe

  Changed 6 years ago by Charlie

The whitespace appears properly, but the chat does not seem to be separated correctly.

I tested with two XOs both running Chat-43 on joyride-2230. Here's what happened:

G02 hosted the chat, and W08 joined from the Neighborhood view. The two laptops chatted for a few lines, then W08 left the chat.

A minute or two later, W08 rejoined the chat, and while the whitespace did appear properly with the text "1 minute ago", the chat up to the point where W08 not only appeared above the whitespace (correctly so), but it also appeared below the whitespace, where further conversation was concatenated to the old chat messages.

This behavior continued over several iterations of W08 leaving the chat and rejoining it; each time it rejoined, the entirety of the chat is added to a new "past chats" area above a whitespace, so the amount of space used in the scrollback buffer potentially could grow exponentially.

Here's how my chat history looked at the end on G02 (the host machine):

W08 joined the chat
G02: Ichi.
W08: Dos.
G02: San.
W08: Cuatro.
W08 left the chat
W08 joined the chat
G02: hm...
W08: beep
W08 left the chat
W08 joined the chat
W08: trsjhf
G02: dfjsg
     ghsfgh
W08 left the chat
W08 joined the chat

Here's how it looks on W08:

G02 está aquí
G02: Ichi.
W08: Dos.
G02: San.
W08: Cuatro.
            14 minutes ago
G02 está aquí
G02: Ichi.
W08: Dos.
G02: San.
W08: Cuatro.
G02: hm...
W08: beep
            12 minutes ago
G02 está aquí
G02: Ichi.
W08: Dos.
G02: San.
W08: Cuatro.
G02: hm...
W08: beep
     trsjhf
G02: dfjsg
     ghsfgh
            1 minute ago
G02 está aquí
G02: Ichi.
W08: Dos.
G02: San.
W08: Cuatro.
G02: hm...
W08: beep
     trsjhf
G02: dfjsg
     ghsfgh

I apologize for the lack of a real log file, I couldn't find it.

I'm not sure whether I should mark this as 'verified' and file a new ticket for the issue I found, or to leave it un-verified and bump it back to morgs. Thoughts?

follow-up: ↓ 14   Changed 6 years ago by morgs

Please retest with Chat-44.

The fix for #7692 stopped replaying the old messages again when you connect, which is what resulted in the duplication of messages.

Then you should be able to mark this one as verified... :)

in reply to: ↑ 13   Changed 6 years ago by Charlie

  • keywords joyride-2263:+ chat-45:+ added; joyride-2153:+ removed
  • next_action changed from qa signoff to finalize
  • verified set

Tested with Chat-45 on joyride-2263. Working as expected, so I'll mark it verified.

  Changed 6 years ago by Charlie

  • owner changed from Charlie to joe

  Changed 6 years ago by gregorio

  • status changed from new to closed
  • resolution set to fixed
Note: See TracTickets for help on using tickets.