Skip to content

Conversation

eps1lon
Copy link
Collaborator

@eps1lon eps1lon commented Sep 30, 2025

No description provided.

@eps1lon eps1lon requested a review from sebmarkbage September 30, 2025 20:59
@meta-cla meta-cla bot added the CLA Signed label Sep 30, 2025
@github-actions github-actions bot added the React Core Team Opened by a member of the React Core Team label Sep 30, 2025
if (nodes != null) {
for (let i = 0; i < nodes.length; i++) {
const node = nodes[0];
const node = nodes[i];
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This fixes e.g. the Initial Paint not getting any highlights.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Well I was planning on actually disabling it for the whole document since it's really annoying to flash the whole document every time you hover over a background or first step.

className={styles.SuspenseBreadcrumbsListItem}
aria-current={selectedSuspenseID === id}
onPointerEnter={highlightHostInstance.bind(null, id)}
onPointerEnter={highlightHostInstance.bind(null, id, false)}
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This fixes a crash. Flow.js doesn't have typings for Host Components so this slipped through.

@eps1lon eps1lon merged commit 1f460f3 into facebook:main Oct 1, 2025
247 checks passed
@eps1lon eps1lon deleted the sebbie/09-30-_devtools_fix_host_instance_highlighting branch October 1, 2025 15:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed React Core Team Opened by a member of the React Core Team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants