Skip to content

Conversation

@Yajur-Grover
Copy link
Contributor

@Yajur-Grover Yajur-Grover commented Oct 18, 2024

Description

Integration
Commit Range: facebook/react-native@223e98c...0d6908f

Notable Commits

Issues Resolved

New follow-up issues

Changelog

Should this change be included in the release notes: no

Add a brief summary of the change to use in the release notes for the next release.

Microsoft Reviewers: Open in CodeFlow

@Yajur-Grover
Copy link
Contributor Author

Yajur-Grover commented Nov 5, 2024

The 'E2E Test App Fabric' test seems to be failing on two of the tests on the Pressable component - both involving a Text component (link to CI):

× Pressables can change text on press/rest, state rest (20359 ms)
× Text can have pressable behavior (20349 ms)

I looked at the components that are failing in RNTester on both main and this integration branch and there was no visual difference in the behaviour - not sure why they are unable to be found when running the test command.

Edit: this issue has been resolved - the default accessible value for text got changed in the merge process.

@Yajur-Grover Yajur-Grover marked this pull request as ready for review November 6, 2024 18:10
@Yajur-Grover Yajur-Grover requested review from a team as code owners November 6, 2024 18:10
context,
rawProps,
"endDraggingSensitivityMultiplier",
sourceProps.endDraggingSensitivityMultiplier,
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we still need this file or can we delete it?

Copy link
Contributor

Choose a reason for hiding this comment

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

I also see that NativeAnimatedHelper.win32.js is still in the repo, I'm not sure #11041 is actually resolved

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Removed #11041 from resolved issues. Deleted the unused fork.

accessible == null
? onPress != null || onLongPress != null
: accessible,
default: accessible !== false,
Copy link
Contributor

Choose a reason for hiding this comment

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

just curious, is this specific for windows? If so, we should add a comment so it doesn't get removed in future integrations

Copy link
Contributor Author

@Yajur-Grover Yajur-Grover Nov 6, 2024

Choose a reason for hiding this comment

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

Yes - added a comment for future reference.

accessible == null
? onPress != null || onLongPress != null
: accessible,
default: accessible !== false,
Copy link
Contributor

Choose a reason for hiding this comment

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

same question here!

Copy link
Contributor

@TatianaKapos TatianaKapos left a comment

Choose a reason for hiding this comment

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

LGTM!

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.

2 participants