Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FIX] Livestream muted when audio only option was enabled #11267

Merged
merged 3 commits into from
Jul 3, 2018

Conversation

gdelavald
Copy link
Contributor

Due to a new flag on Chrome, when the video player was hidden (or open in another tab) it would not start automatically, this was causing a bug when audio only mode was enabled for the popout window on the Livestream feature.

The fix simply remove the visibility hidden and forces a 0px on the player height with overflow being hidden, it looks the same as before but the video is not considered hidden anymore.

@gdelavald gdelavald requested a review from a team June 26, 2018 22:37
@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-11267 June 26, 2018 22:37 Inactive
@gdelavald gdelavald temporarily deployed to rocket-chat-pr-11267 June 27, 2018 20:09 Inactive
@ggazzo ggazzo added this to the 0.66.1 milestone Jul 3, 2018
@ggazzo ggazzo added this to Review/QA in June/2018 via automation Jul 3, 2018
@sampaiodiego sampaiodiego merged commit 8b908ba into develop Jul 3, 2018
June/2018 automation moved this from Review/QA to Closed Jul 3, 2018
@sampaiodiego sampaiodiego deleted the livestream-muted branch July 3, 2018 12:03
@rodrigok rodrigok mentioned this pull request Jul 4, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
No open projects
June/2018
  
Closed
Development

Successfully merging this pull request may close these issues.

None yet

5 participants