Skip to content

set fullscreen goes to black after update 8.1.560 #804

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

Closed
dimitriacosta opened this issue Dec 8, 2018 · 2 comments · Fixed by #811
Closed

set fullscreen goes to black after update 8.1.560 #804

dimitriacosta opened this issue Dec 8, 2018 · 2 comments · Fixed by #811
Labels
Renderer Text rendering issues, including CoreText renderer
Milestone

Comments

@dimitriacosta
Copy link

Before this update I used to have in my ~/.gvimrc file the seting set fullscreen to start MacVim full screen when it starts, but after the update it goes all black.

If I remove this setting from my ~/.gvimrc file and type set fullscreen within MacVim from the command bar it works fine, but not at start up.

@ychin
Copy link
Member

ychin commented Dec 8, 2018

I can repro. Will work on fixing that. I only saw this happening under non-native fullscreen. Just to confirm that's your setting as well?

@ychin ychin added this to the snapshot-153 milestone Dec 8, 2018
@dimitriacosta
Copy link
Author

Oh yes, I forgot to mention that, I'm using the non-native fullscreen as well

ychin added a commit to ychin/macvim that referenced this issue Dec 10, 2018
Fix issues related to starting up in fullscreen mode (by setting
`set fullscreen` in gvimrc).

Fix black screen when starting in non-native fullscreen:
- Fix macvim-dev#804
- Non-native fullscreen has an old behavior where it sets CGLayer mode
  on core text renderer to address rendering artifacts in pre-Mojave,
  and we have an insufficient block against it (by checking cgContext).
  In Mojave, we use buffered draws instead and prevent setting CGLayer
  by checking cgContext, but it is not yet initialized yet in this
  situation (since it's lazily initialized). Block against
  cgBufferDrawEnabled instead.
- The CGLayer renderer is really deprecated at this point so ideally we
  could clean up the code a little bit in the future.

Setting MMUseCGLayerAlways no longer results in black screen:
- Fix macvim-dev#801
- CGLayer renderer is basically deprecated doesn't actually work in
  Mojave. Make sure setting MMUseCGLayerAlways won't trigger it if we
  are already using buffered draw (which is automatically the case in
  Mojave).

Fix starting up in fullscreen resulting in a small Vim window:
- When starting MacVim in fullscreen mode, the presentation is delayed
  until the window is made. need to make sure to set shouldResizeVimView
  in this code path too. Before the guioption-k fixes, there were a lot
  of random calls to recalculate sizes which was why this happened
  implicitly. Now need to make sure we do the right thing and call it.
@ychin ychin added the Renderer Text rendering issues, including CoreText renderer label Dec 12, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Renderer Text rendering issues, including CoreText renderer
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants