Re: [w3c/manifest] Add id member to manifest (#988)

@marcoscaceres commented on this pull request.



> +              <td>"https://example.com/my-app/start"</td>
+              <td>"https://example.com/my-app/start"</td>
+            </tr>
+            <tr>
+              <td>"%6D"</td>
+              <td>"https://example.com/my-app/start"</td>
+              <td>"https://example.com/m"</td>
+            </tr>
+            <tr>
+              <td>"😀"</td>
+              <td>"https://example.com/my-app/start"</td>
+              <td>"https://example.com/%F0%9F%98%80"</td>
+            </tr>
+          </table>
+          <p>Since id is resolved against start_url's origin, providing id as
+          "../foo", "foo", "/foo", "./foo" all resolves to the same thing. It is

```suggestion
          "../foo", "foo", "/foo", "./foo" all resolves to the same [=identifier=]. It is
```

-- 
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/988#pullrequestreview-760409410