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: add focus and blur events for WebContents #25873
feat: add focus and blur events for WebContents #25873
Conversation
Depends on changes in electron/electron#25873 and electron/electron#25874
@electron/wg-releases requesting to add 11-x-y backport label 🙇 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's have a test :)
The API WG approved this at our Oct 19 2020 meeting. |
8859d94
to
40e832f
Compare
@nornagon Tests added |
Just to clarify, does this event also trigger when the browserwindow is focused (e.g. through alt+tab)? or only when focus stays in the browser window and switches between webcontentses? |
It does trigger when focusing the |
@samuelmaddock it looks like the new test is failing on macOS. |
The releases WG approved this PR for backport to 11-x-y at the Oct 28, 2020 meeting |
@samuelmaddock any updates on this? Looks like we are close to being able to merge this if the test failures can be resolved. |
No updates, I'll see if I can get around to this by the end of the week. |
please get this merged asap |
Any updates? |
89ad21d
to
8ceab22
Compare
8ceab22
to
644c749
Compare
test: add focus and blur WebContents event tests test: confirm that webcontents focus event is fired on browserwindow focus fix: mac focus event test timeout
644c749
to
d3a80e6
Compare
Release Notes Persisted
|
Description of Change
Adds 'focus' and 'blur' events to
WebContents
.resolves #22201
cc @electron/wg-api
Checklist
npm test
passesRelease Notes
Notes: Added 'focus' and 'blur' events to
WebContents