Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Browser upgrades follow-up #6300

Merged
merged 4 commits into from
Dec 6, 2024

Conversation

christianbaroni
Copy link
Member

@christianbaroni christianbaroni commented Dec 4, 2024

What changed (plus any additional context for devs)

  • Now enabling/disabling WebView JS handlers when navigating to or away from the browser tab
  • Patches Reanimated to cache all worklets
    • This avoids repeated serialization of worklets, which based on thoroughly testing the app, can be safely cached — it noticeably smooths out gesture-driven interactions and should speed up anything that involves frequently/repeatedly called worklets (this change is being considered in Reanimated)
  • [Android] Removes a duplicate context menu that would show when tapping the edge of the tab button

Screen recordings / screenshots

What to test

…e browser tab, remove unnecessary 'worklet' directive
@christianbaroni christianbaroni added the release for release blockers and release candidate branches label Dec 4, 2024
Copy link
Member

@brunobar79 brunobar79 left a comment

Choose a reason for hiding this comment

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

Code LGTM! Will test in a bit a report back

@brunobar79 brunobar79 marked this pull request as ready for review December 5, 2024 00:35
@brunobar79
Copy link
Member

Launch in simulator or device for 45533b2

Copy link
Member

@brunobar79 brunobar79 left a comment

Choose a reason for hiding this comment

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

Tested on iOS and it works correctly

This avoids repeated serialization of worklets, which based on thorough testing of the app, can be safely cached — it noticeably smooths out gesture-driven interactions and should speed up anything that involves frequently/repeatedly called worklets
Copy link

socket-security bot commented Dec 5, 2024

Removed dependencies detected. Learn more about Socket for GitHub ↗︎

🚮 Removed packages: npm/react-native-reanimated@3.16.2

View full report↗︎

If you tapped around the edge of the tab button, you'd see a different context menu pop up than if you long pressed — this removes the duplicate menu
@brunobar79
Copy link
Member

Launch in simulator or device for c604402

@brunobar79 brunobar79 merged commit d4d70d8 into develop Dec 6, 2024
8 checks passed
@brunobar79 brunobar79 deleted the @christian/browser-upgrades-follow-ups branch December 6, 2024 00:13
ibrahimtaveras00 pushed a commit that referenced this pull request Dec 6, 2024
* Enable/disable WebView JS handlers when navigating to or away from the browser tab, remove unnecessary 'worklet' directive

* Patch Reanimated to cache all worklets

This avoids repeated serialization of worklets, which based on thorough testing of the app, can be safely cached — it noticeably smooths out gesture-driven interactions and should speed up anything that involves frequently/repeatedly called worklets

* [Android] Fix duplicate tab button context menu

If you tapped around the edge of the tab button, you'd see a different context menu pop up than if you long pressed — this removes the duplicate menu

* Fix Podfile.lock
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release for release blockers and release candidate branches
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants