-
Notifications
You must be signed in to change notification settings - Fork 71
Edits for FPWD publication #403
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
Conversation
spec/Overview.html
Outdated
| <head> | ||
| <meta charset="utf-8"> | ||
| <title>Web Cryptography API</title> | ||
| <title>Web Cryptography</title> |
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.
Why did you make this change?
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.
In the new specs, the API name is usually implied considering the type of specification as a browser API. However, if you think it would be useful in the title, I will make the change.
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'm on the fence, but given that it doesn't define all things Web Cryptography I think it's better to keep it in.
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.
it makes sense to me
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.
Thanks for changing it back to Web Cryptography API!
I also think that it makes more sense as what we describe is the API
| ], | ||
| github: "https://github.com/w3c/webcrypto", | ||
| shortName: "WebCryptoAPI", | ||
| shortName: "webcrypto", |
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.
Will the former short name be redirected?
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.
yes, of course
twiss
left a comment
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.
👍 Thanks!
SHA: 37f41ce Reason: push, by twiss Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Closes w3c/webappsec#672
Preview | Diff