fix(mobile): repair Clerk auth navigation headers - #5140
Conversation
- Hide Clerk's nested header and route host back actions to navigation - Upgrade Clerk Expo and related packages for the navigation fix
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
ApprovabilityVerdict: Approved 52fd4ad This PR fixes navigation header display on the mobile auth settings screen by hiding the duplicate native header and delegating back navigation to Clerk's built-in components. The changes are straightforward UI adjustments with limited scope, plus minor Clerk dependency patches. You can customize Macroscope's approvability policy. Learn more. |
What Changed
Why
Clerk’s auth screens were displaying conflicting or incorrect navigation headers inside the mobile settings flow. Delegating back navigation to the host stack and hiding the duplicate native header keeps the auth experience consistent with the surrounding app.
UI Changes
The mobile authentication and account screens now use Clerk’s navigation UI with the host screen’s back action. No screenshots are included.
Checklist
Note
Low Risk
UI and navigation wiring in settings auth only; no changes to auth logic or token handling beyond dependency bumps.
Overview
Fixes double or broken headers on mobile settings sign-in and account screens by hiding the React Navigation stack header and letting Clerk own the chrome.
SettingsAuthRouteScreennow setsheaderShown: falseinstead of a dynamic "Account" / "Sign in" title, and passesonHostBack(wired tonavigation.goBack()) intoAuthViewandUserProfileViewso Clerk’s back control pops the host stack instead of fighting the native header.Clerk is bumped (
@clerk/expo4.2.0,@clerk/react,@clerk/shared,@clerk/clerk-js) to versions that supportonHostBack.Reviewed by Cursor Bugbot for commit 52fd4ad. Bugbot is set up for automated code reviews on this repo. Configure here.
Note
Fix Clerk auth navigation headers in mobile settings screen
ConfiguredSettingsAuthRouteScreenand wiresonHostBackon bothUserProfileViewandAuthViewto callnavigation.goBack()via a memoized callback.@clerk/expo,@clerk/clerk-js,@clerk/react, and@clerk/sharedto their latest patch versions in the workspace catalog.Macroscope summarized 52fd4ad.