- From: Matt Giuca <notifications@github.com>
- Date: Sun, 06 Nov 2022 18:41:43 -0800
- To: w3c/manifest <manifest@noreply.github.com>
- Cc: Subscribed <subscribed@noreply.github.com>
- Message-ID: <w3c/manifest/pull/1056/review/1169731809@github.com>
@mgiuca commented on this pull request. > + </p> + <ol class="algorithm"> + <li>If |target URL| was not given, set |target URL| to [=start URL=]. + </li> + <li>Let |traversable| be the result of + <a data-cite="html#create-a-fresh-top-level-traversable"> + creating a fresh top-level traversable</a> with |target URL| + and |POST resource| (or null if |POST resource| was not given). + </li> + <li>Let |browsing context| be the |traversable|'s + <a data-cite="html#nav-bc">active browsing context</a>. + </li> + <li>[=Apply=] |manifest| to |browsing context| prior to the + |traversable| [=navigating=] to |target URL|. + </li> + </ol> Could you sneak in a "return _browsing context_" at the end here? (And say that it returns a browsing context.) The return value will be ignored, for now, but you'll be able to make use of it in `web-app-launch` and it's not unreasonable for this algorithm to return the browsing context it creates. -- Reply to this email directly or view it on GitHub: https://github.com/w3c/manifest/pull/1056#pullrequestreview-1169731809 You are receiving this because you are subscribed to this thread. Message ID: <w3c/manifest/pull/1056/review/1169731809@github.com>
Received on Monday, 7 November 2022 02:41:55 UTC