Skip to content

[New] jsx-handler-names: support namespaced component names#3943

Merged
ljharb merged 1 commit intojsx-eslint:masterfrom
takuji:ignore-nested-component-names
Feb 17, 2026
Merged

[New] jsx-handler-names: support namespaced component names#3943
ljharb merged 1 commit intojsx-eslint:masterfrom
takuji:ignore-nested-component-names

Conversation

@takuji
Copy link
Copy Markdown

@takuji takuji commented Aug 6, 2025

This PR fixes the problem where the jsx-handler-names rule with ignoreComponentNames option throws an error when it checks an component that is namespaced like <Foo.Component onClick={doSomething} />.

Fixes #3954.

@takuji takuji changed the title [Fix] jsx-handler-names: support namespaced component names [Fix] jsx-handler-names: avoid an error on a nested comonent name with ignoreComponentNames option is enabled Aug 6, 2025
@takuji takuji force-pushed the ignore-nested-component-names branch from 61ebf18 to 06c9aa5 Compare December 30, 2025 17:20
@webmatrixxxl

This comment was marked as spam.

@ljharb ljharb force-pushed the ignore-nested-component-names branch from 06c9aa5 to ed049b0 Compare February 17, 2026 20:45
@ljharb ljharb changed the title [Fix] jsx-handler-names: avoid an error on a nested comonent name with ignoreComponentNames option is enabled [New] jsx-handler-names: support namespaced component names Feb 17, 2026
Copy link
Copy Markdown
Member

@ljharb ljharb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This does fix a crash, but it also adds support for something that wasn't previously supported.

@ljharb ljharb merged commit ed049b0 into jsx-eslint:master Feb 17, 2026
448 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

[Bug]: jsx-handler-names crashes on compound components

3 participants