Skip to content

Add 'instance class fields / resolving identifier in parent scope'#1892

Merged
ljharb merged 1 commit intocompat-table:gh-pagesfrom
davidtaylorhq:instance-class-fields-resolving-identifier-in-parent-scope
Jun 19, 2024
Merged

Add 'instance class fields / resolving identifier in parent scope'#1892
ljharb merged 1 commit intocompat-table:gh-pagesfrom
davidtaylorhq:instance-class-fields-resolving-identifier-in-parent-scope

Conversation

@davidtaylorhq
Copy link
Contributor

This is to detect https://bugs.webkit.org/show_bug.cgi?id=236843, which affects Safari 15 and below.

Discussion in babel/babel#14289

@ljharb ljharb requested a review from chicoxyzzy June 10, 2024 15:42
@davidtaylorhq davidtaylorhq force-pushed the instance-class-fields-resolving-identifier-in-parent-scope branch from e8732fe to a40a7a2 Compare June 10, 2024 16:30
davidtaylorhq added a commit to davidtaylorhq/babel that referenced this pull request Jun 10, 2024
This is a workaround for https://bugs.webkit.org/show_bug.cgi?id=236843, and aims to resolve babel#14289.

Pending:
- acceptance of compat-table/compat-table#1892
- reviewing test failures
@nicolo-ribaudo
Copy link
Contributor

I can confirm that this passes in Firefox 125 and Chrome 74.

@nicolo-ribaudo
Copy link
Contributor

Also Firefox 69

@ljharb ljharb merged commit a40a7a2 into compat-table:gh-pages Jun 19, 2024
@ljharb ljharb temporarily deployed to github-pages June 19, 2024 15:51 — with GitHub Pages Inactive
@nicolo-ribaudo
Copy link
Contributor

Note: Firefox 69 and Chrome 74 are not included in this PR yet! @davidtaylorhq do you mind opening a new one? :)

@davidtaylorhq
Copy link
Contributor Author

Thanks for the merge! Added those two extra data points in #1899

davidtaylorhq added a commit to davidtaylorhq/babel that referenced this pull request Jul 10, 2024
This is a workaround for https://bugs.webkit.org/show_bug.cgi?id=236843, and aims to resolve babel#14289.

Pending:
- acceptance of compat-table/compat-table#1892
- reviewing test failures
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants