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: allow Linux/Windows users to set notification timeout #20153
Conversation
9498433
to
04060e8
Compare
04060e8
to
346634c
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.
Small documentation nit
ebe5752
to
0806aee
Compare
d599c15
to
8555889
Compare
8555889
to
af01fa7
Compare
Release Notes Persisted
|
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.
Looks good to me, but I'd call out that using the reminder
scenario will force a notification sound to be played (the reminder
sound, to be precise).
I'm not sure if setting the notification to silent
actually overrides that behavior.
Either way, solid addition!
Description of Change
Closes #12458.
Allows users to set timeout type of Linux and Windows notifications - either
default
ornever
.cc @ckerr
Checklist
npm test
passesRelease Notes
Notes: Added
timeoutType
property to allows users to set timeout type for Linux and Windows notifications.