content icon indicating copy to clipboard operation
content copied to clipboard

More clearly explain catch binding scope

Open Josh-Cena opened this issue 3 years ago • 5 comments

Description

Mention an Annex B feature, as well as highlighting that there should be zero duplicate names.

Motivation

Additional details

Related issues and pull requests

Josh-Cena avatar Sep 08 '22 13:09 Josh-Cena

If it is not implemented anywhere, we should not document it. We document implementations, not specs.

teoli2003 avatar Sep 08 '22 14:09 teoli2003

Annex B is implemented by all browsers (this is stated by the intro of this section), so for web developers it'll always work, just deprecated.

Josh-Cena avatar Sep 08 '22 14:09 Josh-Cena

Oops, didn't mean to dismiss, but this needs a re-review anyway...

Josh-Cena avatar Sep 22 '22 16:09 Josh-Cena

Pinging @teoli2003 again to decide what to do here, since I have another pending PR to improve the var and other scope-related pages.

Josh-Cena avatar Oct 29 '22 21:10 Josh-Cena

This pull request has merge conflicts that must be resolved before it can be merged.

github-actions[bot] avatar May 03 '23 04:05 github-actions[bot]

@teoli2003 This has been around forever, so maybe we can decide what is needed?

Josh-Cena avatar May 03 '23 07:05 Josh-Cena

I am (semi-)randomly bringing in another JS reviewer to hopefully get some more feedback...

Josh-Cena avatar May 22 '23 17:05 Josh-Cena

Thanks @wbamberg!

Josh-Cena avatar May 24 '23 04:05 Josh-Cena