Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Hide the menu bar by default stopped working #5086

Closed
Sh1d0w opened this issue Oct 24, 2016 · 47 comments
Closed

Hide the menu bar by default stopped working #5086

Sh1d0w opened this issue Oct 24, 2016 · 47 comments
Assignees
Labels
fixed-with-brave-core This issue will automatically resolved with the replacement of Muon with Brave Core. muon OS/unix-like/linux priority/P3 Major loss of function. settings wontfix

Comments

@Sh1d0w
Copy link

Sh1d0w commented Oct 24, 2016

Did you search for similar issues before submitting this one?

No

Describe the issue you encountered:

Enabling "Hide the menu bar by default" from settings has no effect when you launch new Brave instance.

Expected behavior:

Should happen what the setting name suggests.

  • Platform (Win7, 8, 10? macOS? Linux distro?): Fedora 24
  • Brave Version: 8b00d06
  • Steps to reproduce:
    1. Enable "Hide the menu bar by default"
    2. Close Brave
    3. Open Brave
  • Screenshot if needed:
    http://webm.land/media/u1Na.webm
  • Any related issues:
@bsclifton
Copy link
Member

bsclifton commented Oct 24, 2016

Works as-expected on Windows; let me try it out on my Ubuntu VM...

@bsclifton
Copy link
Member

I tried on Ubuntu and menu works... but I had forgotten that Unity always shows a system menu (instead of the per-window menu):
screen shot 2016-10-24 at 5 17 45 pm

I'll have to setup a Fedora VM later tonite and give it a shot

@bsclifton
Copy link
Member

bsclifton commented Oct 25, 2016

@Sh1d0w can the issue be with the theme you're using? Do you have more information on that?

So you're running Fedora 24, which uses Gnome 3 by default... Gnome should be showing the menu at the top of each Window, not at the top of the window manager (ex: where macOS and Ubuntu show it)

@Sh1d0w
Copy link
Author

Sh1d0w commented Oct 25, 2016

@bsclifton No it is not from the theme the same applies if I switch back to the default one for Gnome 3.

The setting is working as you can see in the video if I flip the switch off/on again. The problem is that it is not working during first launch (also same applies if I open a new window with CTRL + N).

Please take a look at the video I have captured http://webm.land/media/u1Na.webm

I guess there is a race condition between initial window initialization and the loading of the settings.

@Sh1d0w
Copy link
Author

Sh1d0w commented Oct 25, 2016

After short investigation it seems the problem is not in the settings. It correctly sets the setting value to true or false here

I think the problem is on Electron side. From the docs:

autoHideMenuBar Boolean - Auto hide the menu bar unless the Alt key is pressed. Default is false.

If it is true then the menu should be hidden, but it is not.

@bbondy bbondy removed this from the 0.12.7dev milestone Oct 25, 2016
@bsclifton
Copy link
Member

Thanks for the research, @Sh1d0w 😄 I'm going to remove the regression tag and add upstream tag. Also going to remove from the milestone for the time being. However, I would be up for helping if you (or anyone else) wanted to grab this 😄 It's possible it's already been fixed upstream and we could pull in a commit that fixes it into our brave/electron fork

@ghost
Copy link

ghost commented Dec 22, 2016

I've tested Brave on several different distros and found that this problem persists regardless the desktop environment on Linux. Two things I have found while testing this. One, if you open another window in front of brave after it starts then switch back to brave it will then hide the menu bar. Two, it may have been mentioned but toggling the "hide menu" option does also work.

I'm no dev but two crude ways to fix this issue would be to either have an invisible window open then close in front of brave after it starts, or have brave start with the hide menu option off then set a script to toggle this option on after startup.

@srirambv
Copy link
Collaborator

image

This happens on any of the Linux distros which doesn't run Unity. This is on Gnome Debian

@ChillyHellion
Copy link

Seems to happen on any version of Linux running Gnome Shell. I can reproduce this in Fedora 25 running Gnome 3.22.2. Going into Brave Browser preferences, turning "hide the menu bar by default" off and then on again will fix the issue. Also pressing alt will successfully hide the menu bar. Both of these fixes only work until the next time Brave Browser is launched.

It's annoying because like Firefox, Brave Browser uses the default GTK system bar at the top of its window which already takes up a lot of screen space. I don't know how Google Chrome avoids this, but I wonder if Chrome's solution could be replicated in Brave.

@bsclifton
Copy link
Member

+1 from #8293

@bbondy bbondy added this to the Prioritized Backlog milestone Nov 2, 2017
@thinkjanis
Copy link

Same issue for me. Using Ubuntu 17.10(Gnome) and Ubuntu Budgie 17.10.

Also, when a tab gets pulled out from a Brave instance where the menu bar is hidden, the new window session with the pulled tab shows the menu bar again.

Brave | 0.19.80
rev | 7d07299
Muon | 4.5.13
libchromiumcontent | 62.0.3202.75
V8 | 6.2.414.36
Node.js | 7.9.0
Update Channel | Release
OS Platform | Linux
OS Release | 4.13.0-16-generic
OS Architecture | x64

@thinkjanis
Copy link

Noticed that when Brave is launched, the menu is shown as described above but when the Brave app window is not active (e. g. switching to a different app), the menu bar disappears.

@BlackRabbit22
Copy link

Same thing is happening to me and I'm on ubuntu. Please fix this.

@Redrobo66
Copy link

Same here using Linux Lite 3.6 (XFCE).
I don't mind the menu so much but I use a dark theme and therefore it displays very dark gray text against a back background - looks ugly so hiding it is pretty much essential.

@OldW1zzrd
Copy link

OldW1zzrd commented Dec 13, 2017

I'm also experiencing this problem. Whenever Brave is reopened, menu bar will be open by default. Even when you go into settings and the "Hide menu bar by default" is already toggled. Toggling it will fix it for the session, but menu bar will be open again when Brave is restarted.

My system info:

OS _ Linux Mint 18.3 Cinnamon 64-bit
Cinnamon Version _ 3.6.6
Linux Kernel _ 4.10.0-40-generic

Brave: 0.19.105
rev: 51f8ba5
Muon: 4.5.16
libchromiumcontent: 62.0.3202.94
V8: 6.2.414.42
Node.js: 7.9.0
Update Channel: Release
OS Platform: Linux
OS Release: 4.10.0-40-generic
OS Architecture: x64

Glad the issue is open. Hope is gets resolved!

Thanks,
OldW1zzrd

@eljuno
Copy link
Contributor

eljuno commented Jan 1, 2018

@jorkki000
Copy link

Same here on Archlinux with openbox. As a workaround I start brave with:
brave & xte 'sleep 3' 'key Alt_L'

@bsclifton
Copy link
Member

+1 from @insanerwayner

When I open the Browser it shows the menu bar. I have to right click and put the checkmark on to active the menu bar then right click again to to turn it off, then the menu bar disappears.

and +1 from @hyph3n

I'm seeing the same behavior on 0.22.0.

@Mando75
Copy link

Mando75 commented Feb 7, 2018

+1 on Arch Linux running Gnome 3.26.2
It is very annoying

No problem when running on the latest build of Windows 10

@eljuno
Copy link
Contributor

eljuno commented Feb 18, 2018

+1 from community https://community.brave.com/t/menu-bar-showing-on-restart-crash-to-desktop-when-clicking-it/16347?u=eljuno

Brave 0.20.42
V8 6.4.388.41
rev 096c7cb
OS Release 4.15.3-2-ARCH
Update Channel Release
OS Architecture x64
OS Platform Linux

@srirambv
Copy link
Collaborator

+1 from @ryanmusante via #13179

@sergiofteixeira
Copy link

1 year and half after still no solution?

@Jacalz
Copy link
Contributor

Jacalz commented Mar 2, 2018

Yeah, this is getting terribly annoying 😐

@ho3ein-mola
Copy link

same problem here . installed by snapy.

@dietbrand
Copy link

+1
Still happens in Ubuntu Bionic Mate. Installed as a snap

@eljuno
Copy link
Contributor

eljuno commented Jun 3, 2018

@srirambv
Copy link
Collaborator

@srirambv
Copy link
Collaborator

Also this issue would be fixed on brave-core. @bsclifton safe to close this issue?

@srirambv srirambv added the fixed-with-brave-core This issue will automatically resolved with the replacement of Muon with Brave Core. label Jun 18, 2018
@bsclifton
Copy link
Member

@srirambv I may be able to deliver a fix for this. Let's keep it open

@bsclifton bsclifton removed this from the Backlog (Prioritized) milestone Aug 13, 2018
@bsclifton
Copy link
Member

This is a super annoying bug, but unfortunately I don't think we'll be able to address it before moving to brave-core

@bsclifton
Copy link
Member

For folks that are willing to try an experimental build, you can grab a copy of brave-core (where this is no longer an issue) from here:
https://github.com/brave/brave-browser/releases

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
fixed-with-brave-core This issue will automatically resolved with the replacement of Muon with Brave Core. muon OS/unix-like/linux priority/P3 Major loss of function. settings wontfix
Projects
None yet
Development

No branches or pull requests