MetaMask / MetaMask/metamask-extension

Parse MainNavigator into Domain Folders

Open

#28,938 opened on Dec 4, 2024 0 comments 0 reactions 1 assignee View on GitHub

@frankvonhoven is already working on this.

Since Dec 4, 2024.

INVALID-ISSUE-TEMPLATE team-mobile-platform

Dominant language: TypeScript
Stars: 13.2k
Forks: 5.6k
Avg merge: 2d 5h
Merged PRs (30d): 451

Description

This is to track the work around separating Navigators out of the single file MainNavigator. This would establish a pattern and reduce the file size and separate concerns into manageable Folders for navigation.

Potential Folder Structure Solution:

Nav
- App
- Main
  - MainNavigator.js
- Wallet
- Asset
  - AssetStackFlow.js
  - AssetModalFlow.js
- Transactions
  - TransactionsHome.js
- Browser
  - BrowserFlow.js
- Snaps
  - SnapsSettingsStack.js
- Notifications
  - NotificationsOptInStack.js
  - NotificationsModeView.js
- Settings
  - SettingsFlow.js
- Home
  - HomeTabs.js
- Webview
  - Webview.js
- Send
  - SendView.js
  - SendFlowView.js
- NFT
  - NftDetailsModeView.js
  - NftDetailsFullImageModeView.js
- Bookmark
  - AddBookmarkView.js
- Offline
  - OfflineModeView.js
- Payment
  - PaymentRequestView.js
- Swaps
  - Swaps.js
- Password
  - SetPasswordFlow.js

Contributor guide

Open the contributing guide

First steps

Read the whole issue, then the project's contributing guide.
Comment on the issue to say you are picking it up — it saves two people doing the same work.
Fork the repository and make your change on a branch.
Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

A short digest of beginner-friendly GitHub issues.