Skip to content

fix: cp-7.57.0 accounts list not scrollable on android#21180

Merged
juanmigdr merged 4 commits into
mainfrom
fix/accounts-list-not-scrollable
Oct 15, 2025
Merged

fix: cp-7.57.0 accounts list not scrollable on android#21180
juanmigdr merged 4 commits into
mainfrom
fix/accounts-list-not-scrollable

Conversation

@juanmigdr

@juanmigdr juanmigdr commented Oct 15, 2025

Copy link
Copy Markdown
Member

Description

Accounts list is not scrollable in android, I have reverted some changes

Changelog

CHANGELOG entry:

Related issues

Fixes: #21181

Manual testing steps

Feature: my feature name

  Scenario: user [verb for user action]
    Given [describe expected initial app state]

    When user [verb for user action]
    Then [describe expected outcome]

Screenshots/Recordings

Before

After

Pre-merge author checklist

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

Route FlashList scrolling through react-native-gesture-handler ScrollView to fix Android scrolling in the Multichain account selector list.

  • Multichain Account Selector List (app/component-library/components-temp/MultichainAccounts/MultichainAccountSelectorList/MultichainAccountSelectorList.tsx):
    • Use react-native-gesture-handler's ScrollView via FlashList renderScrollComponent to restore vertical scrolling on Android.
    • Update imports to include ScrollView and ScrollViewProps.

Written by Cursor Bugbot for commit 11759c1. This will update automatically on new commits. Configure here.

@juanmigdr juanmigdr requested a review from a team as a code owner October 15, 2025 07:40
@juanmigdr juanmigdr enabled auto-merge October 15, 2025 07:41
cursor[bot]

This comment was marked as outdated.

bergarces
bergarces previously approved these changes Oct 15, 2025

@bergarces bergarces left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@github-actions github-actions Bot added size-XS and removed size-L labels Oct 15, 2025
@sonarqubecloud

Copy link
Copy Markdown

@ccharly ccharly left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM (not tested though, but I recall testing the account-list before this ScrollView got removed, so we're good).

@juanmigdr juanmigdr added this pull request to the merge queue Oct 15, 2025
Merged via the queue into main with commit 27cf85f Oct 15, 2025
152 of 156 checks passed
@juanmigdr juanmigdr deleted the fix/accounts-list-not-scrollable branch October 15, 2025 09:01
@github-actions github-actions Bot locked and limited conversation to collaborators Oct 15, 2025
@metamaskbot metamaskbot added release-7.58.0 Issue or pull request that will be included in release 7.58.0 release-7.57.0 Issue or pull request that will be included in release 7.57.0 and removed release-7.58.0 Issue or pull request that will be included in release 7.58.0 labels Oct 15, 2025
@metamaskbot

Copy link
Copy Markdown
Collaborator

Missing release label release-7.57.0 on PR. Adding release label release-7.57.0 on PR and removing other release labels(release-7.58.0), as PR was cherry-picked in branch 7.57.0.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

release-7.57.0 Issue or pull request that will be included in release 7.57.0 size-XS team-assets

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Accounts List Android cannot be scrolled

4 participants