Add note for required web app manifest members #24700
Open
+16
−9
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
This PR adds a note to the manifest members that are required for a web app to be installable.
Bonus edit: In the
icons
file, I've rephrased the safari note so that we don't use "key" because we refer them as (manifest) members in content.Supporting details
The following browser-specific information is good to be included in the BCD table (instead of being added as a note on respective manifest member pages):
Context: mdn/content#35847 (comment)
Assumptions and questions
I am assuming that by virtue of "mirror" value, the added note will be replicated to all chromium-based browsers. Is that correct?
I've added the note only to Chrome, but it also applies to Chrome Android, Edge, Opera (and Android), Samsung Internet, WebView Android.
I am unclear about some of the values showing in the table. For example:
display
on MDNdisplay
in BCDI am not sure what's the source of the values visible on MDN.