Re: [whatwg/webidl] Fix links, remove as many anchor definitions as possible (PR #1270)

@TimothyGu commented on this pull request.



> @@ -6936,8 +6869,8 @@ in ECMAScript, as defined by the <cite>ECMAScript Language Specification</cite>
 [[!ECMA-262]].
 
 Unless otherwise specified, objects defined in this section are ordinary objects as described in
-[=ECMA-262 Ordinary object internal methods and internal slots=], and if the
-object is a [=function object=], [=ECMA-262 Built-in function objects=].
+[[ecmascript#sec-ordinary-object-internal-methods-and-internal-slots]], and if the
+object is a [=function object=], [[ecmascript#sec-built-in-function-objects]].

This is wonderful. Thanks!

> @@ -7655,7 +7587,7 @@ values are represented by ECMAScript Object values.
     An ECMAScript value |V| is [=converted to an IDL value|converted=]
     to an IDL {{object}} value by running the following algorithm:
 
-    1.  If <a abstract-op>Type</a>(|V|) is not Object, then [=ECMAScript/throw=] a {{ECMAScript/TypeError}}.
+    1.  If <a abstract-op>Type</a>(|V|) is not Object, then [=ECMAScript/throw=] a {{TypeError}}.

This is another name clash between ES and Web IDL. Previously it linked to ECMAScript, but now it goes to Web IDL. It's mostly fine since Web IDL's `{{TypeError}}` is basically a redirect to ES. But since these algorithm steps are written in the style of ECMAScript (including the `throw` reference), I'd still prefer to use the ECMAScript link if possible.

>  Note: Don't confuse the "{{SyntaxError!!exception}}" {{DOMException}} defined here
-with ECMAScript's {{ECMAScript/SyntaxError}}.
+with ECMAScript's {{SyntaxError}}.
 "{{SyntaxError!!exception}}" {{DOMException}} is used to report parsing errors in web APIs,
 for example when parsing selectors,
-while the ECMAScript {{ECMAScript/SyntaxError}} is reserved for the ECMAScript parser.
+while the ECMAScript {{SyntaxError}} is reserved for the ECMAScript parser.
 To help disambiguate this further,
 always favor the "{{SyntaxError!!exception}}" {{DOMException}} notation
 over just using {{SyntaxError!!exception}} to refer to the {{DOMException}}. [[DOM]]

The new `{{SyntaxError}}` references are incorrect: it goes to the unrelated `"SyntaxError"` `DOMException` in Web IDL rather than ES. In fact, this paragraph explains exactly how they are different.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/whatwg/webidl/pull/1270#pullrequestreview-1309884309
You are receiving this because you are subscribed to this thread.

Message ID: <whatwg/webidl/pull/1270/review/1309884309@github.com>

Received on Wednesday, 22 February 2023 18:22:49 UTC