Fix chat view when leaving current room #871
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: Daniel Calviño Sánchez <[email protected]>
When explicitly leaving the current room through the room list view the chat view was detached. However, as the internal state of the Application object (which manages the placement of the chat view) was not updated the chat view was always hidden from that point on until the page was reloaded. Moreover, as the chat view was detached only when explicitly leaving the current room the chat view was not hidden for other users in the room if it was deleted by a moderator. As the "leaveCurrentRoom" event is triggered in the room channel when the current room is left, either explicitly or implicitly, now the Application object listens to that event and hides the chat view as needed when it is received. Signed-off-by: Daniel Calviño Sánchez <[email protected]>
In order to check if certain conversation is the active conversation the element for the active conversation list item with the given name was found and then it was verified that it was visible; waiting for the element to be visible using the implicit wait made to find the element is a valid approach if the element is always visible when it exists in the DOM. However, during the lapse between removing/deleting the conversation and getting the updated conversation list from the server the element may be hidden but exist in the DOM. Due to that it has to be explicitly waited for the element to be visible. Signed-off-by: Daniel Calviño Sánchez <[email protected]>
Signed-off-by: Daniel Calviño Sánchez <[email protected]>
Signed-off-by: Daniel Calviño Sánchez <[email protected]>
danxuliu
added
3. to review
backport-request
bug
feature: chat 💬
This was referenced May 9, 2018
Ivansss
approved these changes
May 9, 2018
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes #857