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: expose audio-state-changed
on webContents
#37366
Conversation
* `event` Event | ||
* `audible` boolean - Whether audio is currently playing. | ||
|
||
Emitted when media becomes audible or inaudible. |
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.
Again a little imprecise for the same reason, but ... revising this in a way that still scans well is not obvious to me. For example this would be more correct but feels awkward:
Emitted when media becomes audible or inaudible. | |
Emitted when the first frame or child content starts playing, or when the last stops playing. |
If you have a third idea then let's consider that; otherwise, maybe OK as-is
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.
I updated the property description but i think this is probably more understandable as-is 🤔
0475e00
to
b1125ed
Compare
b1125ed
to
506fb05
Compare
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.
API LGTM
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.
API LGTM
Release Notes Persisted
|
I was unable to backport this PR to "24-x-y" cleanly; |
Description of Change
Closes #37352.
Exposes and documents the
audio-state-changed
event onwebContents
. We previously emitted it, but it was a private and undocumented event. This documents and removes the-
prefix.Checklist
npm test
passesRelease Notes
Notes: Exposes the
audio-state-changed
event onwebContents
.