- From: Anne van Kesteren <notifications@github.com>
- Date: Wed, 24 Feb 2021 23:34:02 -0800
- To: whatwg/url <url@noreply.github.com>
- Cc: Subscribed <subscribed@noreply.github.com>
Received on Thursday, 25 February 2021 07:34:14 UTC
@annevk commented on this pull request. > @@ -205,7 +205,7 @@ U+0024 ($) to U+0026 (&), inclusive, U+002B (+), and U+002C (,). <p class=note>This is used by <cite>HTML</cite> for {{NavigatorContentUtils/registerProtocolHandler()}}, and could also be used by other standards to percent-encode data that can then be embedded in a <a for=/>URL</a>'s <a for=url>path</a>, -<a for=url>query</a>, or <a for=url>fragment</a>. Using it with +<a for=url>query</a>, or <a for=url>fragment</a>, and is also a <a>valid opaque-host string</a>. Using it with ```suggestion <a for=url>query</a>, or <a for=url>fragment</a>, or <a for=/>opaque host</a>. Using it with ``` would be more consistent I think. I did notice while looking at this that valid opaque-host string doesn't say that `%` is fine. Perhaps the forbidden host code points should be two sets after all. -- 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/584#pullrequestreview-598265818
Received on Thursday, 25 February 2021 07:34:14 UTC