- From: Alwin Blok <notifications@github.com>
- Date: Thu, 07 Oct 2021 10:06:31 -0700
- To: whatwg/url <url@noreply.github.com>
- Cc: Subscribed <subscribed@noreply.github.com>
- Message-ID: <whatwg/url/pull/655/review/774149658@github.com>
@alwinb commented on this pull request. I like this a lot, much better! > @@ -2603,6 +2591,21 @@ these steps. They return an <a>ASCII string</a>. <li><p>Return <var>output</var>. </ol> +<p>The <dfn export lt="URL path serializer|URL path serializing">URL path serializer</dfn> takes a +<a for=/>URL</a> <var>url</var> and then runs these steps. They return an <a>ASCII string</a>. + +<ul> + <li><p>If <var>url</var> does not have a <a for=url>hierarchical path</a>, then return If going with the suggestion by @TimothyGu, this can be rephrased as "If url has an opaque path, then return url's path". > @@ -1811,18 +1808,16 @@ these steps: <var>base</var>'s <a for=url>scheme</a> is equal to <var>url</var>'s <a for=url>scheme</a>, set <var>state</var> to <a>special relative or authority state</a>. - <p class=note>This means that <var>base</var>'s <a for=url>cannot-be-a-base-URL</a> is - false. + <p class=note>This means that <var>base</var> has a <a for=url>hierarchical path</a>. Maybe we can state "This means that base is special and has a hierarchical path" Or possibly "This means that base is special and therefore has a hierarchical path", to connect to the previous note about that. > -<p class="note no-backref">A <a lt="is special">special</a> <a for=/>URL</a> always has a -<a for=list lt="is empty">non-empty</a> <a for=url>path</a>. +<p class=note>A <a lt="is special">special</a> <a for=/>URL</a> always has a The statement holds as a consequence of the parser, right? We could state it as an invariant on URL records here instead, one that the parser maintains? I personally find it easier to think about it that way, but I'm curious what others think. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/whatwg/url/pull/655#pullrequestreview-774149658
Received on Thursday, 7 October 2021 17:06:44 UTC