-
Notifications
You must be signed in to change notification settings - Fork 49
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
Editorial: modernize the language around actions #212
Conversation
This will make it easier to give them more associated concepts.
I went ahead with this as it's editorial. Happy to address any nits of course, as always. |
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.
The only nit, otherwise looks fine.
the main body of the notification to open their inbox. | ||
<p>Here a service worker shows a notification with a single "Archive" <a for=/>action</a>, allowing | ||
users to perform this common task from the notification without having to open the website (for | ||
example the, notification platform might show a button on the notification). The user can also |
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.
example the, notification platform might show a button on the notification). The user can also | |
example, the notification platform might show a button on the notification). The user can also |
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.
Oh wow, how did that happen. Thanks!
This will make it easier to give them more associated concepts.
Preview | Diff