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

Hide menu bar in full screen #51

Closed
rom1504 opened this issue Mar 6, 2016 · 6 comments
Closed

Hide menu bar in full screen #51

rom1504 opened this issue Mar 6, 2016 · 6 comments

Comments

@rom1504
Copy link
Member

rom1504 commented Mar 6, 2016

full-screen-webtorrent-app

On ubuntu with kde. I don't know if it happens with other desktop environments

@rom1504 rom1504 mentioned this issue Mar 6, 2016
14 tasks
@giuscri
Copy link
Contributor

giuscri commented Mar 6, 2016

Same on LXDE

@dcposch
Copy link
Contributor

dcposch commented Mar 7, 2016

Looks fine on Ubuntu with Unity

image

Ubuntu is the standard desktop Linux, the most common by far.

The rabbit hole of alternative setups -- various distributions using LXDE, KDE, older versions of Gnome, xmonad, etc etc--goes pretty deep. I don't know if we want to support all that.

(If it can be done without hacks, I'm cool with it. However, we're calling the built-in Electron BrowserWindow.setFullScreen() function. This Just Works on the three standard desktops: Windows, OSX, and unmodified Ubuntu.)

@ungoldman
Copy link
Contributor

It would be nice to support edge cases but @dcposch makes a very good point about the variety of linux distros. @rom1504 @giuscri I think PRs are welcome if you can come up with a solution but priority will probably be given to making sure it works as intended on the big three.

@rom1504
Copy link
Member Author

rom1504 commented Mar 7, 2016

playback works just fine on kubuntu. I'll try to figure out what is going wrong here. Maybe this can help https://github.com/mafintosh/playback/blob/67b9d034094b3fd4954fafdd5e156e8e44c39163/index.js#L131

@giuscri
Copy link
Contributor

giuscri commented Mar 7, 2016

@ngoldman, to my understanding this might be a problem that has to be delegated to Electron's source code. I'll try to see if I have to skill to solve it and try a PR...

@feross feross closed this as completed in 251eff8 Mar 8, 2016
feross added a commit that referenced this issue Mar 8, 2016
@dcposch
Copy link
Contributor

dcposch commented Mar 8, 2016

👍

@lock lock bot locked as resolved and limited conversation to collaborators May 11, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants