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

When going fullscreen, nav bar should be hidden (and show when mouse enters top of screen) #4371

Closed
srirambv opened this issue Sep 29, 2016 · 15 comments
Labels

Comments

@srirambv
Copy link
Collaborator

srirambv commented Sep 29, 2016

When going fullscreen, the navigation bar, URL bar, etc should be hidden. If you move the mouse in the area they used to occupy, we can have them slide down.

Since some folks may not want this functionality (instead preferring to always show the navigation controls), we may want to make this a preference.

Expected behavior:
Should show top bar every time there is mouse over at the top of the screen

@srirambv srirambv added this to the 0.12.3dev milestone Sep 29, 2016
@srirambv
Copy link
Collaborator Author

cc: @Sh1d0w

@bbondy bbondy removed this from the 0.12.3dev milestone Sep 29, 2016
@bbondy
Copy link
Member

bbondy commented Sep 29, 2016

Reverted here:
#4372

It can be re-added once we have a fix for Windows. Sorry about this but this is holding up RC3 from releasing and we need it out today so it is the best option.

@Sh1d0w
Copy link

Sh1d0w commented Oct 12, 2016

Unfortunately I do not own Windows machine nor I have time to install and debug it in VM. If anyone wants to pick up my work (which is working just fine under Linux) and fix the windows magnification issue it will be much appreciated, so we can all have this nice feature.

As a hint how this quickly can be fixed is that the top area which sets mounseInTitleBar to true has to be set from 1px to 2-3px probably (https://github.com/brave/browser-laptop/pull/4193/files#diff-303f0b6a297506f2cc18bf7b4cb370c5R522)

@bbondy bbondy modified the milestones: 0.12.6dev, 0.12.5dev Oct 13, 2016
@srirambv
Copy link
Collaborator Author

@Sh1d0w @bbondy @bsclifton

Can only the URL bar be shown in full screen mode hiding bookmarks toolbar and tabs. On windows it can become a bit confusing whether the browser is in full screen or maximized window with the current behavior( reverted to show top bar). There is no significant difference between full screen and maximized window, only the browser covers the taskbar in full screen mode.

@bbondy bbondy modified the milestones: 0.12.7dev, 0.12.6dev, 0.12.8dev Oct 18, 2016
@bbondy
Copy link
Member

bbondy commented Oct 24, 2016

Moving to 0.12.8

@Sh1d0w
Copy link

Sh1d0w commented Oct 25, 2016

Can I kindly ask @bradleyrichter for suggestions how it should behave?

Brad please see #4234

@bsclifton
Copy link
Member

Removing milestone for now

@bbondy @bradleyrichter please feel free to re-assign a milestone (also, see @Sh1d0w's question in last post)

@bsclifton bsclifton removed this from the 0.12.8dev milestone Oct 31, 2016
@luixxiul luixxiul added this to the Backlog milestone Jan 10, 2017
@Seferi
Copy link

Seferi commented Mar 17, 2017

Hi, i was wondering when it will be implemented back.. I think full screen is a very important feature especially for smaller screens. It should be a priority imho..

@bsclifton
Copy link
Member

@Seferi good question... priority-wise, this is likely something we'll get to after version 1.0. However, I agree: this would be a great one to have. I wonder if any of our contributors are interested?

Let me clean up the issue a bit (so the description of what we'd like is clear) and I'll tag with "Help wanted" 😄

@bsclifton bsclifton changed the title Top bar not always shown in full screen mode on mouse over Nav bar should be hidden and then show when mouse enters top of screen Mar 24, 2017
@bsclifton bsclifton added the help wanted The PR/issue opener needs help to complete/report the task. label Mar 24, 2017
@bsclifton
Copy link
Member

@Seferi please check the original issue above (which I edited). Anything I missed? 😄

@Seferi
Copy link

Seferi commented Mar 28, 2017

@bsclifton Thanks mate!

@bbondy bbondy removed this from the Backlog milestone Oct 19, 2017
@unnamed-user-account
Copy link

Hi,

I'm a new user to Brave and while playing some .io games and I am in F11 mode the entire nav-bar is shown as seen below:

image

Firefox & Chrome hide the nav-bar (although in different fashions) when in F11 mode. It would be really nice if Brave could also do the same thing. I would appreciate if something could be done about this :S

@bsclifton bsclifton added this to the Triage Backlog milestone Nov 27, 2017
@bsclifton bsclifton changed the title Nav bar should be hidden and then show when mouse enters top of screen When going fullscreen, nav bar should be hidden (and show when mouse enters top of screen) Dec 26, 2017
@bsclifton
Copy link
Member

+1 from @Kasmetski via #12399

Fullscreen option doesn't work. It hides only the menubar and topbar.
I'm expecting to hide everything like Chrome & Firefox. It should hide bookmarks, url bar and everything other except opened page.

@chrishobcroft
Copy link

For me, this is one of the main drawbacks of Brave over e.g. Chromium or Firefox on Ubuntu.

The beauty of surfing the web without any kind of navigation bar is really appealing, and having to "waste" about 13% (rough estimate) of the screen real-estate is disappointing.

Expected behaviour is that when you press F11, the full screen becomes dedicated to the webpage that you are viewing. And if you move the mouse to the top, or press Ctrl-T to open a new tab, then it should show the address bar (not the bookmarks bar / tabs bar).

@bsclifton bsclifton removed the help wanted The PR/issue opener needs help to complete/report the task. label Sep 1, 2018
@bsclifton
Copy link
Member

Closing as a duplicate of #11463 (which is marked as fixed-with-brave-core). You can download the developer channel release here:
https://brave.com/download-dev

@bsclifton bsclifton added the duplicate Issue has already been reported label Sep 21, 2018
@bsclifton bsclifton removed this from the Triage Backlog milestone Sep 21, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

9 participants