fix: android scroll issue on reveal srp bottom sheet

[mathieuartu]

Description

In the reveal SRP flow, the user cannot scroll the accounts list inside the component with the SRP once this one is expanded

Changelog

CHANGELOG entry: bug fix where a user couldn't scroll through the SRP reveal bottom sheet on Android

Related issues

Fixes: #19999
Jira ticket: https://consensyssoftware.atlassian.net/browse/MUL-1366

Manual testing steps

Feature: reveal SRP screen

  Scenario: user wants to see all of one SRP's accounts on Android
    Given user has multiple SRPs with more than 10 EVM accounts in each

    When user goes to settings > security > reveal SRP > XX accounts
    Then user is able to scroll down the list of accounts without issues

Screenshots/Recordings

Before

Screen.Recording.2025-09-18.at.20.48.34.mov

After

Pre-merge author checklist

• I’ve followed MetaMask Contributor Docs and MetaMask Mobile Coding Standards.
• I've completed the PR template to the best of my ability
• I’ve included tests if applicable
• I’ve documented my code using JSDoc format if applicable
• I’ve applied the right labels on the PR (see labeling guidelines). Not required for external contributors.

Pre-merge reviewer checklist

• I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
• I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

---

Note

Resolves Android scrolling issues in the SRP reveal bottom sheet by constraining list height and enabling nested scrolling.

• Updates SRPList to use useWindowDimensions and pass vars.maxHeight (~70% of window height) to styleSheet
• Adds maxHeight to SRPList.styles.ts and a flatList style (flexGrow: 0) to prevent over-expansion
• Switches FlatList to react-native-gesture-handler and enables scrollEnabled/nestedScrollEnabled
• Cleans up redundant flex styles in base and srpListContentContainer

^{Written by Cursor Bugbot for commit a5650b9. This will update automatically on new commits. Configure here.}

[github-actions]

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

[github-actions]

🔍 Smart E2E Test Selection

• Selected E2E tags: SmokeAccounts, SmokeWalletPlatform
• Risk Level: medium
• AI Confidence: 85%

click to see 🤖 AI reasoning details

The changes modify the SRPList component which is used in multiple account management flows:

1. Style changes: Modified the base container styling (removed flex properties, added maxHeight) and added a new flatList style with flexGrow: 0
2. Component changes: Switched from React Native's FlatList to react-native-gesture-handler's FlatList, and added dynamic maxHeight calculation based on window dimensions (70% of screen height)

These changes affect:

• SelectSRP view (used when revealing recovery phrase)
• AddNewAccount view (used when adding accounts to specific SRPs)
• WalletRecovery view
• ProtectYourWallet settings section

The E2E tests in the multisrp folder (add-account.spec.ts, export-srp-from-settings.spec.ts) and accounts folder (reveal-secret-recovery-phrase.spec.ts, import-srp.spec.ts) directly test these flows and use the SRPList component.

Selected tags:

• SmokeAccounts: Covers account management, keyrings, and SRP reveal functionality
• SmokeWalletPlatform: Covers multi-SRP functionality including adding accounts to specific SRPs and exporting SRPs

Risk is medium because:

• The changes are primarily UI/styling related
• The switch to react-native-gesture-handler's FlatList could affect scrolling behavior in bottom sheets
• The maxHeight constraint could affect how the list displays with many SRPs

View GitHub Actions results

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[gantunesr]

Looks good, can you add a recording on the fix on the PR description?