Skip to content

fix: call SetCanActivate in setFocusable#21846

Merged
zcbenz merged 1 commit intomasterfrom
fix-win-focus
Jan 22, 2020
Merged

fix: call SetCanActivate in setFocusable#21846
zcbenz merged 1 commit intomasterfrom
fix-win-focus

Conversation

@zcbenz
Copy link
Contributor

@zcbenz zcbenz commented Jan 21, 2020

Description of Change

Fix #21459.

This PR fixes the issue that calling setFocusable(true) could not turn an unfocusable window back to focusable.

Checklist

Release Notes

Notes: Fix BrowserWindow.setFocusable(true) not working on Windows.

@electron-cation electron-cation bot added the new-pr 🌱 PR opened recently label Jan 21, 2020
@zcbenz zcbenz requested review from brenca and deepak1556 January 21, 2020 09:01
@zcbenz zcbenz merged commit 36fac7b into master Jan 22, 2020
@zcbenz zcbenz deleted the fix-win-focus branch January 22, 2020 01:13
@release-clerk
Copy link

release-clerk bot commented Jan 22, 2020

Release Notes Persisted

Fix BrowserWindow.setFocusable(true) not working on Windows.

@trop
Copy link
Contributor

trop bot commented Jan 22, 2020

I was unable to backport this PR to "6-1-x" cleanly;
you will need to perform this backport manually.

@trop trop bot removed the target/6-1-x label Jan 22, 2020
@trop
Copy link
Contributor

trop bot commented Jan 22, 2020

I was unable to backport this PR to "7-1-x" cleanly;
you will need to perform this backport manually.

@trop
Copy link
Contributor

trop bot commented Jan 22, 2020

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

@trop
Copy link
Contributor

trop bot commented Jan 22, 2020

@zcbenz has manually backported this PR to "7-1-x", please check out #21855

@trop
Copy link
Contributor

trop bot commented Jan 22, 2020

@zcbenz has manually backported this PR to "6-1-x", please check out #21856

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

Labels

new-pr 🌱 PR opened recently

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Setting focusable:false breaks the focus event

2 participants