doc, url: various improvements to WHATWG API#11330
Uh oh!
There was an error while loading. Please reload this page.
Conversation
TimothyGu added doc 
doc/api/url.md Outdated
There was a problem hiding this comment.
Probably "as percent-encoded strings" or "as a string of percent-encoded characters", or even "percent-encoded bytes"? "a percent-encoded string" sounds a bit weird given that they would be not just be encoded but also joined with & and =s.
Another way to explain it would be to mention that it is somewhat similar to querystring.stringify().
There was a problem hiding this comment.
How about:
-Returns the search parameters serialized as a URL-encoded string.+Returns the search parameters serialized as a string, with characters+percent-encoded where necessary.doc/api/url.md Outdated
There was a problem hiding this comment.
nit: "does not allow users to customize the.." seems easier to read?
doc/api/url.md Outdated
There was a problem hiding this comment.
nit: "Below it"?(as opposed to "Above it")
doc/api/url.md Outdated
There was a problem hiding this comment.
I know it's already there but technically, doesn't url.parse() percent-decode things..?
There was a problem hiding this comment.
url.parse() also encodes certain things:
>url.parse('http://\n.com').pathname'%0A.com'TimothyGuforce-pushed the url-components-doc branch from 7a14576 to 424ec4fCompare
jasnell commented Feb 15, 2017
TimothyGu commented Feb 16, 2017
PR-URL: #11330 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Joyee Cheung <[email protected]>
Also remove executable bit from doc/api/url.md's mode. PR-URL: #11330 Fixes: 4757ddc "doc: add basic documentation for WHATWG URL API" Reviewed-By: James M Snell <[email protected]> Reviewed-By: Joyee Cheung <[email protected]>
Also remove executable bit from doc/api/url.md's mode. Backport-of: nodejs#11330 Fixes: 84e2ff3 "doc: add basic documentation for WHATWG URL API"
Also remove executable bit from doc/api/url.md's mode. Backport-of: nodejs#11330 Fixes: 84e2ff3 "doc: add basic documentation for WHATWG URL API"
5 participants
Checklist
Affected core subsystem(s)
doc, url