Re: [whatwg/url] Percent-encode additional characters in "fragment state". (#347) from Anne van Kesteren on 2017-10-10 (public-webapps-github@w3.org from October 2017)

From: Anne van Kesteren <notifications@github.com>
Date: Tue, 10 Oct 2017 10:20:15 +0000 (UTC)
To: whatwg/url <url@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <whatwg/url/pull/347/review/68231375@github.com>

annevk commented on this pull request.

Final nit I suspect, other than getting the tests in order.

> @@ -169,9 +169,11 @@ contains bytes that are not <a>ASCII bytes</a> might be insecure and is not reco
 <p>The <dfn oldids=simple-encode-set>C0 control percent-encode set</dfn> are the <a>C0 controls</a>
 and all <a>code points</a> greater than U+007E (~).
 
+<p>The <dfn>fragment percent-encode set</dfn> is the <a>C0 control percent-encode set</a> and
+U+0020 SPACE, U+0022 ("), U+003C(&lt;), U+003E (&gt;), and U+0060 (`).

Missing space after U+003C.

-- 
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/347#pullrequestreview-68231375

Received on Tuesday, 10 October 2017 10:20:39 UTC