New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 868398 link

Starred by 122 users

Issue metadata

Status: Assigned
Merged: issue 831219
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 1
Type: Bug

Blocking:
issue 831219
issue 890073

Restricted
  • Only users with EditIssue permission may comment.


Show other hotlists

Hotlists containing this issue:
MacViewsBrowser-RS


Sign in to add a comment

[MacViews] In Fullscreen Mode, and the Hide the Toolbar option is enabled, Tabstrip/Omnibox/BMB don't appear when you move the mouse to the top of the screen

Project Member Reported by meh...@chromium.org, Jul 27

Issue description

Chrome Version: Canary 70.0.3504.0
OS: macOS 10.13.6

What steps will reproduce the problem?
(1) Switch into Fullscreen Mode via CTRL-CMD-F
(2) Hide the Toolbar in Fullscreen Mode via SHIFT-CMD-F
(3) Move the Mouse cursor to the top of the screen

What is the expected result?
Tabstrip/Omnibox/BMB should appear, so you can navigate easily.

What happens instead?
Tabstrip/Omnibox/BMB don't appear. Only Menubar and the grey bar with the window controls appear.

That was the way it worked before in Cocoa.

Thanks
Mehmet


 
Labels: M-70 Target-70
Owner: sdy@chromium.org
Status: Assigned (was: Untriaged)
Routing to sdy@ for initial assessment.

Comment 2 Deleted

FYI this behavior has reached the stable build of Chrome on MacOS. It renders navigation impossible in fullscreen mode.
 Issue 881312  has been merged into this issue.
Yeah, this is in stable now, Chrome 69. Very annoying.
I'm actively working on bringing this back. For now, a workaround is to use cmd+shift+F to show and hide the toolbar in fullscreen.
 Issue 881947  has been merged into this issue.
I'm feeling cautiously good about hooking into macOS's native toolbar handling.
toolbar_transplant.mp4
198 KB View Download
@sdy nice! thanks a lot for working on this. 

two observations:

1. the window buttons appear to overlap the new tab, rather than what appears to be current behavior (attached screenshot)
2. it would be interesting to see if the correct behavior still preserved when the address / tab bar is hidden via: ⌘ + Shift + f


Screenshot 2018-09-07 18.21.04.png
12.6 KB View Download
Mergedinto: 831219
Status: Duplicate (was: Assigned)
Re. #9, yep, that's just a PoC :-). I'm planning to fix both of those things! Let me also merge this into another bug that happened to be opened first.
Status: Assigned (was: Duplicate)
Oh, I just realized that 831219 is serving more as a tracking bug. Un-merged.
Labels: -Pri-2 Pri-1
Kicking this up to Pri-1 based on star count and other reports.
Issue 882640 has been merged into this issue.
Progress…
toolbar_now_with_sliding.mp4
1.3 MB View Download
Looks great. Thanks!!!
What about SHIFT-CMD-F mode? The issue is mainly about this exact mode. The video demonstrates some improvements in CTRL-CMD-F mode, but that is not related to the issue.
Cc: robliao@chromium.org
 Issue 883863  has been merged into this issue.
 Issue 884292  has been merged into this issue.
 Issue 884324  has been merged into this issue.
yeah, I'd like to know that the SHIFT-CMD-F mode is being fixed as pressing that whenever I need to change tabs or type in a new website is annoying. the browser is 99% unusable in that mode in this state
 Issue 884504  has been merged into this issue.
 Issue 884657  has been merged into this issue.
Yes, the change covers both modes even though I've only posted recordings of the toolbar visible mode.
Labels: Hotlist-ConOps
 Issue 886929  has been merged into this issue.
Thanks for working on bringing the shift-cmd-F reveal/autohide functionality back. It is a huge inconvenience to have to hit the shortcut every time. When do you think we'll see an update fixing this? 
Labels: Restrict-AddIssueComment-EditIssue
 Issue 889762  has been merged into this issue.
Cc: nyerramilli@chromium.org rbasuvula@chromium.org
 Issue 894431  has been merged into this issue.
Cc: abdulsyed@chromium.org
 Issue 894481  has been merged into this issue.
Issue 894088 has been merged into this issue.
 Issue 896896  has been merged into this issue.
Blocking: 890073
Issue 899061 has been merged into this issue.
 Issue 898364  has been merged into this issue.
 Issue 901347  has been merged into this issue.
 Issue 905038  has been merged into this issue.
Cc: susan.boorgula@chromium.org
 Issue 905346  has been merged into this issue.
 Issue 905776  has been merged into this issue.
 Issue 905417  has been merged into this issue.
Labels: Hotlist-DesktopUIChecked
Status: WontFix (was: Assigned)
**UI mass Triage**

We were unable to reproduce this bug. If this bug still reproduces for you, 
please reopen or file a new issue. 

Thanks!
Cc: krajshree@chromium.org
Status: Assigned (was: WontFix)
This is still an issue. 
Project Member

Comment 44 by bugdroid1@chromium.org, Nov 19

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/9bfd10b3bcbc4af6b9a3132937fb4c48401bbe15

commit 9bfd10b3bcbc4af6b9a3132937fb4c48401bbe15
Author: Sidney San Martín <sdy@chromium.org>
Date: Mon Nov 19 23:27:44 2018

Mac: Delete the OSX.Fullscreen.ToolbarStyle histogram.

This'll make it easier for me to delete surrounding code, and I can't
find any references to it so I suspect that it's safe to prune.

Bug: 868398
Change-Id: Ib62d77f87df261200c6e6c1b821f99ed707041b9
Reviewed-on: https://chromium-review.googlesource.com/c/1343058
Commit-Queue: Sidney San Martín <sdy@chromium.org>
Reviewed-by: Elly Fong-Jones <ellyjones@chromium.org>
Reviewed-by: Ilya Sherman <isherman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#609503}
[modify] https://crrev.com/9bfd10b3bcbc4af6b9a3132937fb4c48401bbe15/chrome/browser/ui/cocoa/fullscreen/fullscreen_toolbar_controller.h
[modify] https://crrev.com/9bfd10b3bcbc4af6b9a3132937fb4c48401bbe15/chrome/browser/ui/cocoa/fullscreen/fullscreen_toolbar_controller.mm
[modify] https://crrev.com/9bfd10b3bcbc4af6b9a3132937fb4c48401bbe15/chrome/browser/ui/views/frame/browser_non_client_frame_view_mac.mm
[modify] https://crrev.com/9bfd10b3bcbc4af6b9a3132937fb4c48401bbe15/tools/metrics/histograms/histograms.xml

 Issue 904824  has been merged into this issue.
Cc: phanindra.mandapaka@chromium.org
 Issue 921336  has been merged into this issue.

Sign in to add a comment