Skip to content

Conversation

@MarshallOfSound
Copy link
Member

As per the plan in #23506 and in the breaking changes document, lets enable context isolation by default in Electron 12.

Notes: Changed the default value of contextIsolation to true.

@itsananderson
Copy link
Member

Should there be any docs changes included along with this?

@electron-cation electron-cation bot removed the new-pr 🌱 PR opened recently label Dec 15, 2020
@MarshallOfSound MarshallOfSound force-pushed the ctx-isolation-by-default branch from 47d399d to 07234f6 Compare February 16, 2021 20:43
codebytere
codebytere previously approved these changes Feb 26, 2021
@codebytere codebytere dismissed their stale review February 26, 2021 17:26

oops meant to comment

@codebytere
Copy link
Member

Appears the visibility state tests are struggling on macOS

@jkleinsc jkleinsc merged commit 186301e into master Mar 1, 2021
@release-clerk
Copy link

release-clerk bot commented Mar 1, 2021

Release Notes Persisted

Changed the default value of contextIsolation to true.

@jkleinsc jkleinsc deleted the ctx-isolation-by-default branch March 1, 2021 21:53
@trop
Copy link
Contributor

trop bot commented Mar 1, 2021

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants