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

URL and tab bar do not hide in full screen mode #11463

Closed
satnatantas opened this issue Oct 11, 2017 · 39 comments
Closed

URL and tab bar do not hide in full screen mode #11463

satnatantas opened this issue Oct 11, 2017 · 39 comments
Labels
feature/tabsbar feature/URLbar fixed-with-brave-core This issue will automatically resolved with the replacement of Muon with Brave Core. misc/fullscreen-option priority/P5 Cosmetic. Spelling, copy, layout. New features (which should also be part of an initiative). wontfix

Comments

@satnatantas
Copy link

satnatantas commented Oct 11, 2017

Requires #2775 to be fixed

Description

URL and tab bar are not hidden in full screen mode while Windows' task bar gets hidden (i.e. browser is indeed going into full screen mode).

Steps to Reproduce

I cannot get rid of this bug.

Actual result:
Bars are present.

Expected result:
They should not.

Reproduces how often:
Reproducible on my machine 100% of cases.

Brave Version

Brave | 0.19.44
rev | e391af1
Muon | 4.4.28
libchromiumcontent | 61.0.3163.100
V8 | 6.1.534.41
Node.js | 7.9.0
Update Channel | Beta
OS Platform | Microsoft Windows
OS Release | 6.1.7601
OS Architecture | x64

Same on 0.18.36.

Reproducible on current live release:
Yes.

Additional Information

https://imgur.com/a/jbLSt
It worked for few days first. Restart, reboot or installing Brave for different user changes nothing. It is very interesting how can there be any state saved between users which causes this bug. I can provide more information and try to debug this if there are guides about that. Only ever worked if site itself has full screen support and own full screen button, I did not pay attention to that.

@hugobuddel
Copy link
Contributor

How do you trigger full screen? I'm trying F11, but it will just maximize the window.

What I'm looking for is a way to just show the page on the screen, nothing else, no tab-bar, title-bar, any-other-bar, as well as no bars/panels from the desktop/window manager.

@luixxiul
Copy link
Contributor

@hugobuddel

What I'm looking for is a way to just show the page on the screen, nothing else, no tab-bar, title-bar, any-other-bar, as well as no bars/panels from the desktop/window manager.

How about enhancing these options on #11547 (comment) to achieve #6928 (comment)?

clipboard01

the menubar will also be able to be hidden (to achieve that we need to figure out how and where to display the menu bar)

@luixxiul
Copy link
Contributor

luixxiul commented Oct 17, 2017

that issue on about:brave is a duplicate of #10711

please close this issue if the issue is covered with that. thanks!

@luixxiul luixxiul added the duplicate Issue has already been reported label Oct 17, 2017
@hugobuddel
Copy link
Contributor

That seems to be almost like fullscreen @luixxiul, although it still shows the menu bar in your example. And I suppose it would also show the windows taskbar and such?

Also, it seems that what is described in #11547 would require quite some manual steps to remove everything and put everything back, but perhaps I don't understand it.

What I would like is something similar to the F11 key in most other browsers (firefox, chrome, edge). That is, a quick way to maximizes the viewport of the page (to the entire screen) and to minimize distractions. E.g. when reading a long read, or when composing text.

Currently F11 maximizes/'demaximizes' the Brave window (at least in Windows). I don't think I ever needed a keyboard shortcut for that, but apparently there is one already in Windows: winkey + arrow up (or arrow down), so F11 for that is superfluous.

@luixxiul
Copy link
Contributor

Also, it seems that what is described in #11547 would require quite some manual steps to remove everything and put everything back, but perhaps I don't understand it.

What I would like is something similar to the F11 key in most other browsers (firefox, chrome, edge)

Yes, I understand what you mean. Pushing F11 on Fx hides everything but the webview. The actual blocker for that seems to me is #2775:

Display menubar on hover in fullscreen mode. Unlike Mac, Windows does not show you a menubar when you hover the top part of the window. You cannot escape from fullscreen mode UNLESS either you push F11 or kill the program.

That seems to be almost like fullscreen @luixxiul, although it still shows the menu bar in your example. And I suppose it would also show the windows taskbar and such?

I follow the Fx's practice to customize the toolbar. which I now think is irrelevant to the fullscreen option.

@luixxiul luixxiul removed the duplicate Issue has already been reported label Oct 17, 2017
@hugobuddel
Copy link
Contributor

Could you tell me what Fx is @luixxiul?

The behavior of my Brave installation (0.19.54, Windows 10) is very different from what is described in #2775, and also very different from the screenshot you posted. Almost nothing happens when I press F11. The window maximizes or turns back to a normal window and that's it. But it is hard to spot the difference since the windowed version is almost the same size as the maximized version. See the below two screenshots:

Before hitting F11, Brave is not maximized: you can see the black of my desktop wallpaper, and the windowing button (next to the close button on the top right) shows that I can maximize the window.
screenshot 161

After hitting F11, the windows is now maximized: no wallpaper visible and the windowing button now shows that I can turn the application back to windowed mode.
screenshot 162

I can go back and forth between those two modes by either: a) hitting F11, b) clicking the window-icon next to the close-icon on the top right, or c) pressing windows-key + up-arrow and windows-key + down-arrow. These all three do almost the same thing (the difference is the size of the window when not maximized). I'd like F11 to be different: not show anything except the page.

So where #2775 talks about difficulty of leaving full screen, I have a problem entering full screen. From the screenshot from @satnatantas it seems that they have the same problem.

@luixxiul
Copy link
Contributor

luixxiul commented Oct 17, 2017

I see, let me explain: what I mean is that it is quite easy to code to hide everything on F11 mode, but the issue here is that we often forget which key you have to push to go out of the fullscreen mode until you know the browser well.

On Firefox the toolbar is displayed when you move your mouse cursor on the top of the monitor, and you can click the caption button to go out of the issue.

Without that hint, for new users, sometimes it becomes hard to do so.

We could easily imagine that they might think "how to go out the fullscreen mode?" (whether they actually would say so or not). Without solving the usability issue, hiding everything from the fullscreen mode can be an issue for them (not for you; I know you're an advanced user). I myself have noticed that issue. Neither ESC nor Windows key worked and I almost gave up to use the browser at all.

Please forget about my screenshot. I found It was irrelevant with the fullscreen option.

@hugobuddel
Copy link
Contributor

Ah thank you @luixxiul, I misunderstood you. I thought that there were all these people using Brave full screen with only minor problems 😃. Other issues seem to indicate that such a full screen mode has indeed existed in the past.

E.g. #11558 has a checkbox next to "Test that entering full screen window works View -> Toggle Full Screen. And exit back (Not Esc)." This lead me to conclude that it should be possible to go to full screen mode. Actually toggling full screen mode through the view menu does nothing.

You are right, it should be clear how to leave full screen mode. If that's not clear, then it is better to not have a full screen mode at all. I've also been through that, putting something full screen by accident with no clue how to get back. But then we should not check that checkbox in #11558 until it works. And perhaps remove the option from the menu until then.

Made me wonder what other browsers do, perhaps we can do something similar:

  • Firefox shows the bars at the top again when you move the mouse cursor to the top of the screen.
  • Chromium/chrome has a message 'Press F11 to leave full screen'. (Nothing happens when moving the mouse to the top of the screen.)
  • Edge shows the top right window buttons when you move the mouse to the end of the screen.

@hugobuddel
Copy link
Contributor

Ah, learned something. Pressing F11 or using view->full screen does do something: it does not show the windows task bar.

I've always got the windows taskbar to automatically hide (on the right side of the screen), so there was no real visible change for me when going to full screen mode. I only noticed that something was different because the taskbar doesn't unhide when moving the mouse cursor to it in full screen mode.

Hence the confusion. So perhaps the checkbox in #11558 is then technically correct, even though the full screen option (as it is now) is irrelevant for me.

@satnatantas
Copy link
Author

I am very afraid to be hallucinating but I was very sure that full screen mode hid all bars when I activated it with F11. If there are good reasons to suppose that I am wrong do not hesitate to do it.

If it the case then treat this issue like request for implementing full screen mode without bars (which appear upon hovering the border).

@satnatantas
Copy link
Author

I now understand my eff-up. I did not activate full screen mode on Periscope (the only site which I used in full screen mode) with F11, I used deicated page button instead (for whatever reason). Sorry for confusion.

Would still like to see better full screen mode.

@hugobuddel
Copy link
Contributor

hugobuddel commented Oct 18, 2017

Hmm yes, that button on periscope does exactly what I'd want F11 to do. Brave even asks whether it allows the site to go full screen and shows 'press Esc to exit' on top. Youtube can also go full screen, but that'll just show the movie, unlike periscope. So apparently all the necessary functionality for full screen is already in Brave.

So it is totally unclear to me why F11 (on any page) does not do the same as that full screen button on periscope. @luixxiul do you know?

@NumDeP
Copy link

NumDeP commented Oct 28, 2017

+1 I'd also like a feature for tab bar to hide under the url bar and either appears when scrolling upwards or clicking something on the browser.

Slightly different to 'Automatically hide the taskbar in desktop mode', perhaps you could allow a fun feature of when it is allowed to hide under the url bar moving the mouse upwards doesn't have to always display the entire bar but also allowing it to be displayed like piano keys, so moving the mouse upwards shows the tab that would originate to wherever you're pointing the mouse and moving right to left displays the tabs like a piano.

The reason why I'd like a feature like this would be because there are some occasions where I'd like to take screenshots/vids and I would prefer to keep the numerous tabs that are open private but have no choice but to close every tab and then reopening every single tab and I don't think I have to mention the fact that the bookmaking feature isn't appreciated just yet.

Another reason for this feature would be for better screen maximization not just for small screen issues but also the fact it sometimes looks neater so you can concentrate on just the page rather than having numerous titles and different colours presented along a line which can cause distraction.

I suppose this could be an added benefit as well given over time more user extensions will be presented and it may be best to have two or three rows, one, for pinned/bookmarked items, one for the tabs and the other for extensions, unless the team have decided to opt for something better. from a design perspective.

@bsclifton bsclifton added this to the Triage Backlog milestone Nov 27, 2017
@ftruzzi
Copy link

ftruzzi commented Dec 7, 2017

+1 to this, it's a very important feature to be able to focus without getting distracted and finding yourself on another website

@eljuno
Copy link
Contributor

eljuno commented Dec 14, 2017

+1 from community https://community.brave.com/t/what-about-a-real-fullscreen-mode/12096?u=eljuno

when we enter in full screen (F11), it’s not a real full screen because we can still see the URL and all open tabs…

@bradleyrichter
Copy link
Contributor

@bsclifton If this is an easy enough change, let's do it soon.

@bradleyrichter
Copy link
Contributor

This user really likes true full screen: ; )

https://www.youtube.com/watch?v=-iP9-Et5mv4

@bradleyrichter
Copy link
Contributor

So the desire, to summarize, is to change our current implementation of full-screen (not to be confused with a FS video view) to hide all UI elements with an expanded view of the content area.

(no toolbars visible)

@bsclifton bsclifton modified the milestones: Triage Backlog, 0.22.x (Nightly Channel) Jan 25, 2018
@hugobuddel
Copy link
Contributor

I'm using Brave 0.19.147 on windows with the windows taskbar hidden. I just noted that full screen currently results in a smaller viewport than non-full screen.

Normal, maximized. Notice that on the right the 'window' seems to extend a bit out of the screen, as can be seen from the arrows on the scrollbar that touch the edge of the screen. The taskbar is hidden on that side of the screen, not sure whether that matters.
screenshot 190

'Full screen', now the viewport is slightly smaller. The arrows on the scrollbar are now fully visible.
screenshot 191

Brave is the very first program where full screen is smaller than non full screen :-).

@satnatantas
Copy link
Author

@hugobuddel looks very much like erratic window size calculation because the scrollbar is slightly cropped on first shot.

@bbondy bbondy modified the milestones: 0.22.x (Developer Channel), 0.23.x (Nightly Channel) Feb 25, 2018
@alexwykoff alexwykoff added the priority/P5 Cosmetic. Spelling, copy, layout. New features (which should also be part of an initiative). label Mar 13, 2018
@alexwykoff alexwykoff modified the milestones: 0.23.x (Nightly Channel), Backlog (Prioritized) Mar 13, 2018
@mduliszewski
Copy link

This problem is probably the only thing keeping me from using Brave, thanks everyone for working on issue.

@bsclifton bsclifton removed their assignment Mar 20, 2018
@peeech
Copy link

peeech commented Mar 25, 2018

+1 from Mac user.

@DesertDwarf
Copy link

DesertDwarf commented Apr 1, 2018

+1 from me (Windows 10 user). This is the only thing keeping me from using this as my default browser. Sadly, this issue doesn't seem to have anybody assigned to it.

@Dendystale
Copy link

+1 from Mac use. Though I use it as default now. I really like it. Fingers crossed on this thing getting fixed! Cheers everyone

@camelloman
Copy link

+1 Mac user. I love brave, but writing in it is a pain. Without full screen I can't concentrate.

@ghost
Copy link

ghost commented May 8, 2018

I have the same problem on latest release. Key F11 for fullscreen doesn't remove url bar, rest is fullscreen. (Meaning the Windows taskbar gets hidden, meaning fullscreen started)
OS: Windows 10

Brave: 0.22.669
V8: 6.6.346.24
rev: 6cda2e8
Muon: 5.2.7
OS Release: 10.0.16299
Update Channel: Release
OS Architecture: x64
OS Platform: Microsoft Windows
Node.js: 7.9.0
Brave Sync: v1.4.2
libchromiumcontent: 66.0.3359.117

@MikeJWms
Copy link

+1 from a Mac user. Really like Brave so far but this is a problem that might cause me to jump back to another browser... Thanks for all the hard work on this project!

@elmarkruitwagen
Copy link

+1 from a Mac User

@mangomadness
Copy link

+1 from another Mac user. Brave seems great but not having 'true' full screen (URL and tabs hidden) is almost a (trivial) deal breaker

@droumis
Copy link

droumis commented Jun 6, 2018

+1 mac

@WuerfelDev
Copy link

+1 Ubuntu&Windows&Android

@mangomadness
Copy link

+1 ubuntu

@bsclifton
Copy link
Member

@srirambv is this fixed with Brave Core?

@betegon
Copy link

betegon commented Jun 18, 2018

+1 Ubuntu

@srirambv
Copy link
Collaborator

This will be fixed in Brave-core. Closing as wont fix.

@srirambv srirambv removed this from the Backlog (Prioritized) milestone Jun 18, 2018
@srirambv srirambv added wontfix fixed-with-brave-core This issue will automatically resolved with the replacement of Muon with Brave Core. labels Jun 18, 2018
@hugobuddel
Copy link
Contributor

Can we follow the progress of the full screen feature somewhere? Or the process of replacing Muon with Brave Core?

This issue is now closed, but full screen does not work yet. Brave-core does not seem to have an issue tracker of its own, so it is not clear what the status of the full screen feature is, or where to discuss it.

@bbondy
Copy link
Member

bbondy commented Jun 18, 2018

We're tracking issues for brave-core in the parent repo:
https://github.com/brave/brave-browser

While most work goes here:
https://github.com/brave/brave-core

I believe this issue in particular will be covered with brave-core and isn't planned to be fixed before that is released which is this year within the next few months. You can compare for example to what Chrome does and if you think that behaviour isn't right then you can post an issue on the brave/brave-browser repo. Thanks!

For cases like this, we're going with the process of adding a fixed-with-brave-core label and closing the issue. Thanks for your understanding.

@hugobuddel
Copy link
Contributor

OK, so brave-core will behave more 'Chromium-like' than muon?

It's not directly clear from the project descriptions: "Brave Core is a set of changes, APIs, and scripts used for customizing Chromium [..]" and "Muon is a framework that leverages the full power of Chromium [..]".

Seeing all the issues this will fix, it seems like a good change, even though you're probably giving something up in the process too.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature/tabsbar feature/URLbar fixed-with-brave-core This issue will automatically resolved with the replacement of Muon with Brave Core. misc/fullscreen-option priority/P5 Cosmetic. Spelling, copy, layout. New features (which should also be part of an initiative). wontfix
Projects
None yet
Development

No branches or pull requests