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

feat: enable window controls overlay on macOS (#29253) #30698

Merged
merged 2 commits into from Sep 16, 2021

Conversation

jkleinsc
Copy link
Contributor

Backport of #29253

See that PR for details.

Notes: Added support for Windows Control Overlay on macOS.

* feat: enable windows control overlay on macOS

* address review feedback

* chore: address review feedback

* Address review feedback

* update doc per review

* only enable WCO when titleBarStyle is overlay

* Revert "only enable WCO when titleBarStyle is overlay"

This reverts commit 1b58b5b.

* Add new titleBarOverlay property to manage feature

* spelling fix

* Update docs/api/frameless-window.md

Co-authored-by: Samuel Attard <sam@electronjs.org>

* Update shell/browser/api/electron_api_browser_window.cc

Co-authored-by: Samuel Attard <sam@electronjs.org>

* update per review feedback

Co-authored-by: Samuel Attard <sam@electronjs.org>
(cherry picked from commit 1f8a46c)
@electron-cation electron-cation bot added the new-pr 🌱 PR opened in the last 24 hours label Aug 25, 2021
@trop trop bot added 13-x-y backport This is a backport PR backport/requested 🗳 semver/minor backwards-compatible functionality labels Aug 25, 2021
@electron-cation electron-cation bot removed the new-pr 🌱 PR opened in the last 24 hours label Aug 25, 2021
Copy link
Member

@MarshallOfSound MarshallOfSound left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved by @electron/wg-releases

Please do not land until the equivalent Windows PR is up

@zcbenz
Copy link
Member

zcbenz commented Sep 16, 2021

Merging since the Windows PR is ready #30887.

@zcbenz zcbenz merged commit fa46c67 into 13-x-y Sep 16, 2021
@zcbenz zcbenz deleted the backport-macos-overlay branch September 16, 2021 00:34
@release-clerk
Copy link

release-clerk bot commented Sep 16, 2021

Release Notes Persisted

Added support for Windows Control Overlay on macOS.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
13-x-y backport/approved ✅ backport This is a backport PR semver/minor backwards-compatible functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants