feat: show private key list as full screen#28703
Conversation
Replace BottomSheet with stack header + full-screen layout aligned with AddressList. Remove sheet gesture scroll wiring and transparent modal presentation. Made-with: Cursor
|
Overall looking good. The full scree private key screen looks good but the password input screen looks a bit odd with all the info floating at the top. In my opinion the password input screen makes sense to be a bottom sheet since its contextual but I don't feel strongly. Curious what @DostoevskysPiano thinks. screen-20260414-192712-1776209212209.mp4
|
🔍 Smart E2E Test Selection
click to see 🤖 AI reasoning detailsE2E Test Selection: Performance Test Selection: |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #28703 +/- ##
==========================================
+ Coverage 82.19% 82.28% +0.09%
==========================================
Files 4987 5033 +46
Lines 131211 132087 +876
Branches 29316 29481 +165
==========================================
+ Hits 107843 108688 +845
+ Misses 16017 16015 -2
- Partials 7351 7384 +33 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
|
✅ E2E Fixture Validation — Schema is up to date |





Description
The multichain private key list previously used a
BottomSheetand transparent modal presentation. It now uses the same full-screen pattern as Address List:HeaderCompactStandardviauseLayoutEffect/navigation.setOptions, a flex root, and slide-in card options on the parent stack screen.Cancel uses
navigation.goBack()instead of closing a sheet. BottomSheet-specific code (gesture scroll ref context, RNGHScrollViewwrapper for FlashList,panGestureHandlerProps) was removed.Changelog
CHANGELOG entry: Updated multichain private key list to open as a full screen instead of a bottom sheet
Related issues
Fixes: #28809
Fixes: https://consensyssoftware.atlassian.net/browse/MUL-1611
Manual testing steps
Screenshots/Recordings
Before
N/A
After
Screen.Recording.2026-04-15.at.8.34.08.PM.mov
Pre-merge author checklist
Pre-merge reviewer checklist
Note
Medium Risk
Changes navigation/presentation for the private key export list (a sensitive flow), which could impact back navigation, header behavior, or modal stacking. No changes to key export/verification logic, so risk is mainly UI/navigation regressions.
Overview
Shows the multichain private key list as a full-screen screen instead of a bottom sheet, including adding a
HeaderCompactStandard-style header vianavigation.setOptionsand simplifying the layout to a flex-root view.Updates the parent navigator configuration for
Routes.MULTICHAIN_ACCOUNTS.PRIVATE_KEY_LISTto use an opaque card background and a horizontal slide-in interpolator (replacingtransparentModal). Removes BottomSheet-specific gesture/scroll wiring and updates tests to assert the header title viasetOptions, adding a back buttontestID(PrivateKeyListIds.GO_BACK).Reviewed by Cursor Bugbot for commit b0d6ed1. Bugbot is set up for automated code reviews on this repo. Configure here.