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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

macOS: windows no longer open in fullscreen properly anymore #125122

Closed
bpasero opened this issue Jun 1, 2021 · 8 comments
Closed

macOS: windows no longer open in fullscreen properly anymore #125122

bpasero opened this issue Jun 1, 2021 · 8 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug electron Issues and items related to Electron electron-12-update Issues and items related to electron 12 update insiders-released Patch has been released in VS Code Insiders macos Issues with VS Code on MAC/OS X regression Something that used to work is now broken verified Verification succeeded workbench-window Window management
Milestone

Comments

@bpasero
Copy link
Member

bpasero commented Jun 1, 2021

Steps to Reproduce:

  1. be in a fullscreen window
  2. Ctrl+R
  3. select an entry and hold Cmd key

=> 馃悰 a new window opens but not fullscreen

//cc @sandy081

@bpasero bpasero added the workbench-window Window management label Jun 1, 2021
@bpasero bpasero added this to the May 2021 milestone Jun 1, 2021
@bpasero bpasero self-assigned this Jun 1, 2021
@bpasero bpasero added bug Issue identified by VS Code Team member as probable bug regression Something that used to work is now broken labels Jun 1, 2021
@bpasero
Copy link
Member Author

bpasero commented Jun 1, 2021

@deepak1556 I cannot reproduce with Electron 12.0.4 so this seems like a regression with our Electron updates. Can you please advise?

@bpasero
Copy link
Member Author

bpasero commented Jun 1, 2021

I cannot reproduce in 12.0.8 so it can only be electron/electron#29235 or electron/electron#29226

@bpasero bpasero added electron Issues and items related to Electron macos Issues with VS Code on MAC/OS X labels Jun 1, 2021
@bpasero bpasero modified the milestones: May 2021, June 2021 Jun 2, 2021
@bpasero bpasero removed their assignment Jun 2, 2021
@bpasero
Copy link
Member Author

bpasero commented Jun 2, 2021

Pushed a workaround to open in fullscreen but there is still an issue that the window does not seem to create a space on the desktop.

intellild pushed a commit to intellild/vscode that referenced this issue Jun 2, 2021
@deepak1556 deepak1556 added the electron-12-update Issues and items related to electron 12 update label Jun 2, 2021
@intellild
Copy link

Seems I hit github's bug
I synced the main branch to my fork and it shows that I pushed a commit referenced this issue
Please ignore it

@bpasero bpasero changed the title macOS: windows do not open fullscreen from recently opened macOS: windows no longer open in fullscreen properly anymore Jun 11, 2021
@ryenus
Copy link
Contributor

ryenus commented Jun 11, 2021

One more case with native tabs on macOS:

	"window.nativeTabs": true,
	"window.titleBarStyle": "native",
  1. Open vscode as a normal window on Desktop
  2. Press Cmd + Option + F to make it fullscreen with its own Space
  3. Press Cmd + Option + N to open a new window
  4. With 1.56, the new window would just open as a native tab, merged with existing vscode window, but with 1.57.0 there're several issues:
    a. the native tab disappears, leaving only the black background
    b. a separate blank Space is created without anything there, just the black background.

If the step 2 above is skipped, the new window opens just fine on the Desktop, merged with the existing one.

bpasero added a commit that referenced this issue Jun 11, 2021
bpasero added a commit that referenced this issue Jun 14, 2021
* Revert "chore: bump electron@12.0.9"

This reverts commit 286b643.

* Revert "macOS - workaround fullscreen window regression (#125122)"

This reverts commit a286059.
@bpasero bpasero moved this from New to To do in Electron Integration Jun 21, 2021
@deepak1556 deepak1556 moved this from To do to Done / Deferred in Electron Integration Jun 24, 2021
@ryenus
Copy link
Contributor

ryenus commented Jun 25, 2021

One more case with native tabs on macOS:

	"window.nativeTabs": true,
	"window.titleBarStyle": "native",
  1. Open vscode as a normal window on Desktop
  2. Press Cmd + Option + F to make it fullscreen with its own Space
  3. Press Cmd + Option + N to open a new window
  4. With 1.56, the new window would just open as a native tab, merged with existing vscode window, but with 1.57.0 there're several issues:
    a. the native tab disappears, leaving only the black background
    b. a separate blank Space is created without anything there, just the black background.

If the step 2 above is skipped, the new window opens just fine on the Desktop, merged with the existing one.

@deepak1556 The above issue still exists after 3caebff

And the same with the latest main branch at time of writing this.

@deepak1556
Copy link
Contributor

@ryenus thanks for checking, can you extract it to a new issue. The original issue reported here is fixed with the latest insiders.

@deepak1556
Copy link
Contributor

@ryenus I just tried yours steps and unable to reproduce the issue. Can you check if I am missing anything.

recording

@bpasero bpasero added the verified Verification succeeded label Jun 30, 2021
@github-actions github-actions bot locked and limited conversation to collaborators Aug 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug electron Issues and items related to Electron electron-12-update Issues and items related to electron 12 update insiders-released Patch has been released in VS Code Insiders macos Issues with VS Code on MAC/OS X regression Something that used to work is now broken verified Verification succeeded workbench-window Window management
Projects
Archived in project
Electron Integration
  
鉁旓笍 Done / Deferred
Development

No branches or pull requests

5 participants
@ryenus @bpasero @deepak1556 @intellild and others