Closed
Bug 1168851
Opened 9 years ago
Closed 9 years ago
Text chat should be disabled when the other participant leaves the room
Categories
(Hello (Loop) :: Client, defect, P1)
Hello (Loop)
Client
Tracking
(firefox41 verified)
Tracking | Status | |
---|---|---|
firefox41 | --- | verified |
People
(Reporter: standard8, Assigned: standard8)
References
Details
(Whiteboard: [chat])
User Story
As a user, when I'm in a room and there is no active connection, I see: * any text chat history from the immediate past (i.e. that's leftover from the previous uses of the room in this page load, but not from previous page loads) * no text chat entry box, so I don't get frustrated by something I can't actually use.
Attachments
(1 file, 1 obsolete file)
16.84 KB,
patch
|
mikedeboer
:
review+
|
Details | Diff | Splinter Review |
When the room is left by either user, the text chat entry should be disabled. The display of the elements should remain the same.
Flags: qe-verify+
Flags: firefox-backlog+
Updated•9 years ago
|
Rank: 20
Whiteboard: [chat]
Updated•9 years ago
|
Assignee: nobody → dmose
Iteration: --- → 41.2 - Jun 8
Updated•9 years ago
|
Rank: 20 → 12
Priority: P2 → P1
Updated•9 years ago
|
Iteration: 41.2 - Jun 8 → 41.3 - Jun 29
Comment 1•9 years ago
|
||
It seems like the described user story sets the user up for frustration by disabling an input and giving no obvious way to re-enable it. What about, instead, leave the message list displayed (in case the user wants to see what was typed or click a link in it), but hide the text input?
Flags: needinfo?(standard8)
Flags: needinfo?(sfranks)
Comment 2•9 years ago
|
||
Sounds good. Just a note that this field will always remain present when chat history is persistent.
Flags: needinfo?(sfranks)
Assignee | ||
Updated•9 years ago
|
Flags: needinfo?(standard8)
Updated•9 years ago
|
User Story: (updated)
Assignee | ||
Comment 3•9 years ago
|
||
Gonna steal this, as I think we should take it for the next loop-client release, and the fix is fairly easy.
Assignee: dmose → standard8
Assignee | ||
Comment 4•9 years ago
|
||
This adds signalling to the sdk driver to tell the store when the streams are disconnected, and hence the data channel goes down. As the store now knows, the views get updated correctly. I've also added a follow-on fix for bug 1168829 where I realised the conversation toolbar had a border when it shouldn't have done.
Attachment #8624211 -
Flags: review?(mdeboer)
Comment 5•9 years ago
|
||
Comment on attachment 8624211 [details] [diff] [review] Text chat should be disabled when the other participant leaves the room. Review of attachment 8624211 [details] [diff] [review]: ----------------------------------------------------------------- This patch doesn't disable/ hide the textarea when the link clicker decides to leave the room.
Attachment #8624211 -
Flags: review?(mdeboer) → review-
Assignee | ||
Comment 6•9 years ago
|
||
Doh, I'd forgotton to test for that, and I had previously thought about needing to include it in disconnectSession (otherwise we drop the callbacks before they could happen). Also I just noticed an error in the console that actionData.urls was undefined in the textChatStore, so I've just fixed that whilst I'm in the vague area.
Attachment #8624211 -
Attachment is obsolete: true
Attachment #8624234 -
Flags: review?(mdeboer)
Comment 7•9 years ago
|
||
Comment on attachment 8624234 [details] [diff] [review] Text chat should be disabled when the other participant leaves the room. Review of attachment 8624234 [details] [diff] [review]: ----------------------------------------------------------------- Alright, works as expected. Thanks!
Attachment #8624234 -
Flags: review?(mdeboer) → review+
Comment 9•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/742f057dfbe2
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox41:
--- → fixed
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla41
Updated•9 years ago
|
QA Contact: bogdan.maris
Comment 10•9 years ago
|
||
Tested using latest Aurora 41.0a2 across platforms (Windows 7 64-bit, Mac OS X 10.10.4 and Ubuntu 14.04 32-bit), verified that the text chat is disabled if a user exits the conversation.
You need to log in
before you can comment on or make changes to this bug.
Description
•