-
Notifications
You must be signed in to change notification settings - Fork 40
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
Fullscreen doesn't use the browser's fullscreen API #155
Comments
It's done like that by design to avoid losing the player controls (video.js' controls bound with the webcams media) when setting the content fullscreen. Although I'm not planning on changing this to use the browser native API anytime soon, I'll leave this open if someone is willing to make this happen. |
The latest BBB 2.4RC3 does not use the browser's fullscreen function either. |
Thanks for mentioning, @hiroshisuga . I did know about that client change. For the playback case it was purely a technical/human limitation that moved me to avoid using the browser API. I just couldn't make things work without breaking stuff or losing important UI elements. |
This option still uses the browser's fullscreen in 2.4RC3 (which seems to be intended: bigbluebutton/bigbluebutton#12667 (comment)): Whereas the presentation fullscreen button does not (and the same for webcams and screen shares): Toggling native fullscreen is a separate function from expanding the presentation or videos. What I like about this is that the user can have the chat/notes open while in native fullscreen. The only thing missing from bbb-playback compared to BBB 2.4 is a button that toggles native fullscreen for the whole UI. The current fullscreen button functions the same as BBB 2.4 (expanding the presentation/video area without toggling native fullscreen). Something like this: Though, it might be a bit confusing with the buttons being so close with the same icon. Another thing I find confusing (in BBB as well) is calling everything "fullscreen". In my opinion, "fullscreen" should only refer to browser native fullscreen and the other buttons should be "expand" or "maximize". |
@pedrobmarin please help me about [https://github.com/bigbluebutton/bigbluebutton/issues/18170] @hiroshisuga guided me using this [https://github.com//issues/155] but its already opened issue. |
Describe the bug
The fullscreen toggle isn't truly fullscreen. The content area is maximized by hiding the sidebar and slide navigation (as expected), but the browser's fullscreen API is not being used. Browser and OS toolbars are still visible while fullscreen is enabled.
For comparison, BBB uses the fullscreen API, as did the old version of bbb-playback (that shipped with BBB 2.2). Currently, the user experience for fullscreen is inconsistent between live classes and viewing recordings.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
Browser and OS toolbars should be hidden while fullscreen, allowing bbb-playback to fill the entire screen.
Actual behavior
Browser and OS toolbars are visible while fullscreen.
Desktop (please complete the following information):
The text was updated successfully, but these errors were encountered: