- From: Marcos Cáceres <notifications@github.com>
- Date: Tue, 03 Oct 2017 08:18:57 +0000 (UTC)
- To: w3c/manifest <manifest@noreply.github.com>
- Cc: Subscribed <subscribed@noreply.github.com>
Received on Tuesday, 3 October 2017 08:19:20 UTC
marcoscaceres commented on this pull request. > </li> - <li>If <var>unprocessed applications</var> is an <a>array</a>, then: + <li>For <var>V</var> in <var>A</var>: Nit: please use for each, IDL sequences are just arrays, so it's nice to keep the language that maps to things in JS. Also, please don't use the bracket syntax, just use: ```HTML For each `<var>relatedApp</var>` in `<var>manifest</var>. <a>related_applications</a>. ``` Then we get a link back to related_applications, and don't need to guess on the type. -- 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/613#pullrequestreview-66682503
Received on Tuesday, 3 October 2017 08:19:20 UTC