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: Cite Image Resource #895
Conversation
146ffa9
to
4964c96
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.
Looks good to me, thank you.
Looks good to me as well. I'm wondering if |
Could we see |
The new monochrome purpose could be preferred by some Android launchers I think |
index.html
Outdated
@@ -1139,11 +1139,11 @@ <h3> | |||
<var>manifest</var>["<a>categories</a>"]. | |||
</li> | |||
<li>Set <var>manifest</var>["<a>icons</a>"] to the result of running | |||
<a>processing ImageResource members</a> given | |||
<a>processing ManifestImageResource members</a> given |
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.
<a>processing ManifestImageResource members</a> given | |
<a>processing `ManifestImageResource` members</a> given |
index.html
Outdated
<var>manifest</var>["<a>icons</a>"] and <var>manifest URL</var>. | ||
</li> | ||
<li>Set <var>manifest</var>["<a>screenshots</a>"] to the result of | ||
running <a>processing ImageResource members</a> given | ||
running <a>processing ManifestImageResource members</a> given |
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.
running <a>processing ManifestImageResource members</a> given | |
running <a>processing `ManifestImageResource` members</a> given |
index.html
Outdated
or to integrate the web application with an <abbr title= | ||
"operating system">OS</abbr>'s task switcher and/or system | ||
preferences. | ||
<a>ManifestImageResource</a>s that can serve as iconic |
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.
<a>ManifestImageResource</a>s that can serve as iconic | |
{{ManifestImageResource}}s that can serve as iconic |
index.html
Outdated
@@ -1528,7 +1528,7 @@ <h3> | |||
examination, to in fact be inappropriate (e.g. because its content | |||
type is unsupported), then the user agent MUST try the | |||
next-most-appropriate icon as determined by examining the | |||
<a>ImageResource</a>'s members. | |||
<a>ManifestImageResource</a>'s members. |
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.
<a>ManifestImageResource</a>'s members. | |
{{ManifestImageResource}}'s members. |
index.html
Outdated
@@ -1891,8 +1891,8 @@ <h3> | |||
</h3> | |||
<p> | |||
The <dfn>screenshots</dfn> member is an <a>array</a> of | |||
<a>ImageResource</a>s, representing the web application in common | |||
usage scenarios. | |||
<a>ManifestImageResource</a>s, representing the web application 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.
<a>ManifestImageResource</a>s, representing the web application in | |
{{ManifestImageResource}}s, representing the web application in |
index.html
Outdated
agent can use these values to select an image that is best suited to | ||
display on the end-user's device or most closely matches the end-user's | ||
preferences. | ||
Each <a>ManifestImageResource</a> represents an image that is used as |
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.
Each <a>ManifestImageResource</a> represents an image that is used as | |
Each {{ManifestImageResource}} represents an image that is used as |
index.html
Outdated
<li>Let |image| be the result of running [=process an image | ||
resource from JSON|processing an image resource from JSON=] given | ||
|entry| and |manifest URL|. |
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.
<li>Let |image| be the result of running [=process an image | |
resource from JSON|processing an image resource from JSON=] given | |
|entry| and |manifest URL|. | |
<li>Let |image| be the result of running [=process an image resource from JSON=] given | |
|entry| and |manifest URL|. |
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.
Ok, let's run with this...
However, I think we should remove platform
if we can. Let's discuss that elsewhere.
Closes #854
This change (choose one):
changes normative sections without changing behavior)
Commit message:
Cite the Image Resource spec.
Preview | Diff