- From: Matt Giuca <notifications@github.com>
- Date: Thu, 03 Aug 2017 18:38:39 -0700
- To: w3c/manifest <manifest@noreply.github.com>
- Cc: Subscribed <subscribed@noreply.github.com>
Received on Friday, 4 August 2017 01:39:01 UTC
mgiuca approved this pull request. > @@ -359,16 +359,16 @@ </li> <li>If <a>obtaining the manifest</a> results in an error, a user agent can, at this point, fall back to using the <a>top-level - browsing context</a>' <code>Document</code>'s metadata to - populate an <a>installation process</a>' UI. + browsing context</a> <code>Document</code>'s metadata to populate + an <a>installation process</a> UI. I think this should read "populate the installation process's UI" (i.e., s/an/the) because the installation process was already instantiated by Step 2.1. I think the apostrophe here was the author following Rule 1c [here](http://www.grammarbook.com/punctuation/apostro.asp): "Some writers and editors add only an apostrophe to all nouns ending in s." So it is correct by some standards, but I think it's pretty unusual and better with an 's. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/w3c/manifest/pull/592#pullrequestreview-54262909
Received on Friday, 4 August 2017 01:39:01 UTC