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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Simple fullscreen support #5344

Closed
subhaze opened this issue Apr 15, 2016 · 55 comments · Fixed by #55267
Closed

Simple fullscreen support #5344

subhaze opened this issue Apr 15, 2016 · 55 comments · Fixed by #55267
Assignees
Labels
feature-request Request for new features or functionality layout General VS Code workbench layout issues on-testplan
Milestone

Comments

@subhaze
Copy link

subhaze commented Apr 15, 2016

  • VSCode Version: 1.0
  • OS Version: OS X 10.10

I could be missing this in an option somewhere, but if not, it would be awesome if there was an option for "use_simple_full_screen" (like in Sublime Text) which just full screens VS Code in the current window and not create a new space like it does currently.

@egamma
Copy link
Member

egamma commented Apr 17, 2016

Did you try View>Toggle Full Screen

@bpasero bpasero added feature-request Request for new features or functionality workbench labels Apr 17, 2016
@bpasero bpasero added this to the Backlog milestone Apr 17, 2016
@bpasero bpasero changed the title [Feature Request] OS X - Simple fullscreen support Simple fullscreen support Apr 17, 2016
@subhaze
Copy link
Author

subhaze commented Apr 17, 2016

@egamma yes, this works for full screening VS Code into a new space, but it's not the same a 'simple full screen' where VS Code would full screen without entering a new space on the desktop.

@subhaze subhaze changed the title Simple fullscreen support Simple fullscreen support OS X Apr 17, 2016
@subhaze subhaze changed the title Simple fullscreen support OS X Simple fullscreen support for OS X Apr 17, 2016
@subhaze subhaze changed the title Simple fullscreen support for OS X Simple fullscreen support Apr 17, 2016
@bpasero bpasero removed their assignment Aug 17, 2016
@dynamix
Copy link

dynamix commented Sep 29, 2016

that is one of the things I really miss compared to Sublime as well. 👍

@mattrc
Copy link

mattrc commented Nov 4, 2016

+1

2 similar comments
@rogeriopvl
Copy link

+1

@rands0n
Copy link

rands0n commented May 14, 2017

+1

@richarddavenport
Copy link

richarddavenport commented May 14, 2017

@subhaze All you have to do is hold down the option key when you click the full screen button and it changes to the old plus button. Is that what you are looking for?

@subhaze
Copy link
Author

subhaze commented May 14, 2017

@richarddavenport nope, that does maximize the editor and is what I currently do. But, it doesn't get rid of the menu/window chrome and actually put the app into "full-screen" mode :(

@bijanx
Copy link

bijanx commented Aug 5, 2017

+1

@kibiz0r
Copy link

kibiz0r commented Aug 16, 2017

+1

@zachflower
Copy link

I thought I'd post an update to this thread, since it has had some eyes on it for a while now. Simple (pre-Lion) fullscreen support has just been added in electron/electron#10254 and will (probably) be released in Electron 1.8.x. While I'm sure it will be a little while before vscode fully tests and upgrades to that version of Electron (1.8.0 is still in beta, after all), some of you will be happy to hear that there is at least a light at the end of the tunnel ☺️

@calebporzio
Copy link

+1

@bpasero bpasero added layout General VS Code workbench layout issues and removed workbench labels Nov 16, 2017
@bpasero bpasero removed this from the Backlog milestone Nov 17, 2017
@SHND
Copy link

SHND commented Dec 9, 2017

Any news on this one guys? This feature is the only reason I'm not switching to vscode from sublime.

@clankill3r
Copy link

Yeah if you have iTerm for example they have the option:

screen shot 2018-01-01 at 23 44 10

Which is really great cause it allows to have a window on top.
The thing I hate the most about the native fullscreen with vscode is that when I run my code it goes away to another window. Therefor I can not test my app and see the console at the same time.

@richarddavenport
Copy link

@clankill3r, if you are on a later version of macOS (not sure which one) you can long press the green fullscreen button and it will let you have two windows split in full screen mode.

I know that still doesn't solve the overall goal of this issue, but it sounded like you do what I like to do and have my code on one side of the screen and the app running on another.

@bertolo1988
Copy link

On 1.28.2this issue still exists.

@bpasero bpasero modified the milestones: On Deck, October 2018 Oct 25, 2018
@bpasero
Copy link
Member

bpasero commented Oct 25, 2018

This landed on master. To enable, configure window.nativeFullscreen: false

@mesqueeb
Copy link

@bpasero I get "Unknown configuration setting" error when I try to add that to my config file.

@bpasero
Copy link
Member

bpasero commented Oct 25, 2018

@mesqueeb you will have to wait until tomorrow and then get our insiders build: https://code.visualstudio.com/insiders/

@a-eid
Copy link

a-eid commented Oct 27, 2018

@bpasero it doesn't work for the insider build as well, Unknown configuration setting
version 1.29.0-insider

@ignu
Copy link
Contributor

ignu commented Oct 27, 2018

@a-eid it's not in the insider build yet. looks like the current commit is 4a29bd3 and this is a few commits after that. hopefully tomorrow.

@bpasero
Copy link
Member

bpasero commented Oct 27, 2018

We had to stop the insiders build due to a bug, so you will probably only see this next week Monday.

@ignu
Copy link
Contributor

ignu commented Oct 29, 2018

it's in insiders now!

@bpasero there's a 1 or 2 second pause when i switch to simpleFullScreen now that wasn't on my custom build. is that from 9667519#diff-7c4f2973d00eb042eb19c3c3260ae078R523 ? also i don't quite understand what that was fixing.

@bpasero
Copy link
Member

bpasero commented Oct 29, 2018

@ignu seems quite fast for me, I do not think my change has an impact. That was to fix an issue (that is still there actually) where Electron would return a 0px width for the document while entering or leaving fullscreen. But if you think your delay is caused by it, just comment it out and try it.

@bpasero
Copy link
Member

bpasero commented Oct 30, 2018

Fyi the setting got renamed from window.nativeFullscreen to window.nativeFullScreen

@kibiz0r
Copy link

kibiz0r commented Oct 31, 2018

I am seeing some weird behavior when using multiple windows and switching between them with cmd+`. Windows no longer rendering but still appearing in the Window menu, or windows completely closing without warning.

I'll see if I can get some solid repro steps, but if it's not something with my specific setup then I bet you can repro easily regardless. I had no extensions enabled, and my settings file isn't too crazy.

settings.json.txt (Github won't upload JSON files?)

@bpasero
Copy link
Member

bpasero commented Oct 31, 2018

@kibiz0r if you can distill steps, file a separate issue please. if this feature is too broken in Electron, I will consider to take it out again. I will only tolerate it if it does not cause too many new bugs.

@ignu
Copy link
Contributor

ignu commented Oct 31, 2018

@kibiz0r @bpasero cmd+` is fine for me on multiple monitors. i even put instances on three monitors and even tried a combination of some in fullscreen and some windowed and can't reproduce. I'm on High Sierra (10.13.2)

@kibiz0r
Copy link

kibiz0r commented Nov 5, 2018

I'm just on one monitor. I suppose it could potentially be affected by multiple monitors. I'll do a repro today and try it with two monitors as well.

@kibiz0r
Copy link

kibiz0r commented Nov 6, 2018

  1. Install VSCode Insiders (bdfe4fd)
  2. Save this as ~/Library/Application Support/Code - Insiders/User/settings.json:
{
    "window.nativeFullScreen": false,
    "window.restoreFullscreen": true
}
  1. Open VSCode Insiders
  2. cmd+ctrl+f to fullscreen the window
  3. cmd+n to create a new file, type some stuff in it
  4. cmd+shift+n to open a new window
  5. cmd+ctrl+f to fullscreen this new window
  6. cmd+n to create a new file, type some other stuff in it
  7. cmd+` to switch between windows; see that this works fine
  8. Quit VSCode Insiders
  9. Re-open VSCode Insiders
  10. See that only one window from the last session is re-opened
  11. cmd+shift+n to open a new window
  12. cmd+ctrl+f to fullscreen this new window
  13. cmd+n to create a new file, type some more stuff in it
  14. cmd+` to switch between windows
  15. See that the app alternates between switching windows immediately after pressing cmd+`, then the next time requiring the user to press cmd+` twice before it will switch
  16. Open the Window menu; see that there is only one entry, "Welcome", despite having two windows open
  17. cmd+shift+w to close the first window; see that the "Do you want to save?" dialog slides down from the top of the screen and the window closes [upon clicking "Don't save"]
  18. cmd+shift+w to close some invisible second window; see that the "Do you want to save?" dialog slides up from the bottom of the screen; see that this didn't close any visible window [upon clicking "Don't save"]
  19. cmd+shift+w to close a third window (the second visible window); see that the "Do you want to save?" dialog slides down from the top of the screen and the window closes [upon clicking "Don't save"]

@bpasero
Copy link
Member

bpasero commented Nov 18, 2018

@ignu I (and others: #63291) are seeing a strange behaviour where when entering simple fullscreen, the mouse acts up: it no longer changes its cursor when moving over UI elements, such as the explorer divider to resize it. I also see no hover showing up in the editor when hovering over code.

Given these issues (also #5344 (comment)) I think I will have to disable this feature until the underlying issues (possibly in Electron?) are fixed.

@calebporzio
Copy link

Because it's not enabled by default and at least for me - it works well - can we keep the feature? This has been one of my most anticipated features yet.

@bpasero
Copy link
Member

bpasero commented Nov 26, 2018

I found a way around it and the feature will remain, however we have to disable the custom titlebar on macOS if this feature is enabled due to issues with the mouse interaction.

@clankill3r
Copy link

For me it is bugged but I still love it.
The bugged fullscreen is still better then apple's horrible native fullscreen.

  • when opening vscode the green plus sign does not work, only the shortcut does.
  • when I use a second monitor and have my laptop closed and then unplug the monitor and open my laptop the window is in the wrong size (too large). Maybe other apps have this as well, I need to check that when I'm in the office.
  • the red orange green buttons dissapear after exiting fullscreen, but who uses those anyway...

@vivacenontroppo
Copy link

But you all can just double click on title bar on VSCode and the result is the same. So why even bother to add bugged feature.

@clankill3r
Copy link

clankill3r commented Nov 27, 2018

@vivacenontroppo
How is the result the same?

@ignu
Copy link
Contributor

ignu commented Nov 27, 2018

It's not the same. I mean, it might seem silly to obsess over the few pixels the menu bar takes up "maximized" in macOS, but I find the aesthetics awful and I'm in my editor too many hours a day.

Anyway @clankill3r I'm curious why that happens to you. I fixed that very issue and Simple Fullscreen VS Code should resize itself whenever the resolution changes. (I also seem to have the same flow as you when I take my MacBook home for the night.)

@clankill3r
Copy link

clankill3r commented Nov 28, 2018 via email

@vscodebot vscodebot bot locked and limited conversation to collaborators Dec 10, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature-request Request for new features or functionality layout General VS Code workbench layout issues on-testplan
Projects
None yet
Development

Successfully merging a pull request may close this issue.