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

fix: titlebar and window buttons under simple fullscreen #30671

Merged
merged 1 commit into from Aug 25, 2021

Conversation

zcbenz
Copy link
Member

@zcbenz zcbenz commented Aug 24, 2021

Description of Change

Fix 2 problems:

  • After entering simple fullscreen, the titlebar should be hidden.
  • After leaving simple fullscreen, the window buttons should be hidden for normal frameless window.

Close #30536.

Release Notes

Notes: Fix titlebar showing under simple fullscreen mode.

@zcbenz zcbenz added semver/patch backwards-compatible bug fixes target/13-x-y labels Aug 24, 2021
@electron-cation electron-cation bot added the new-pr 🌱 PR opened in the last 24 hours label Aug 24, 2021
@electron-cation electron-cation bot removed the new-pr 🌱 PR opened in the last 24 hours label Aug 25, 2021
@zcbenz zcbenz merged commit 00d65eb into main Aug 25, 2021
@zcbenz zcbenz deleted the fix-simple-fullscreen branch August 25, 2021 23:29
@release-clerk
Copy link

release-clerk bot commented Aug 25, 2021

Release Notes Persisted

Fix titlebar showing under simple fullscreen mode.

@trop
Copy link
Contributor

trop bot commented Aug 25, 2021

I have automatically backported this PR to "13-x-y", please check out #30706

@trop
Copy link
Contributor

trop bot commented Aug 25, 2021

I have automatically backported this PR to "14-x-y", please check out #30707

@trop
Copy link
Contributor

trop bot commented Aug 25, 2021

I have automatically backported this PR to "15-x-y", please check out #30708

BlackHole1 pushed a commit to BlackHole1/electron that referenced this pull request Aug 30, 2021
igorgladkoborodov added a commit to vv-vim/vv that referenced this pull request Jan 24, 2022
* Upgrade to Electron 16.
* Remove the simple fullscreen title bar fix (#105). Looks like it was fixed in electron/electron#30671
* `second-window` event adds Chromium flags to `args` and it breaks the app. Add temporary fix for this just cleaning exact flags. Need to come up with more permanent and reliable fix.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver/patch backwards-compatible bug fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: setSimpleFullscreen does not hide native titlebar
3 participants