feat: edit custom network flow

[salimtb]

Description

Edit network flow from the modal added

Related issues

Fixes:

Manual testing steps

1. Run yarn && ENABLE_NETWORK_UI_REDESIGN=1 yarn start
2. Go to Settings -> Developer Options
3. Tun on the network new toggle
4. Go to the wallet page
5. Click on the network button ( see the video below )
6. click on the three points button
7. click on edit
8. you should be able to edit a network

Screenshots/Recordings

Before

before.mov

After

after.mov

Pre-merge author checklist

• I've followed MetaMask Contributor Docs and MetaMask Extension 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.

[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.

[darkwing]

We need comments for each of these props.

[bergeron]

An annoying thing is how Suggested name: flashes initially before disappearing. But I think this is an existing thing that occurred before our redesign, and can be fixed later.

Screen.Recording.2024-06-13.at.3.28.13.PM.mov

[bergeron]

approving with a few suggestions

The base branch was changed.

[bergeron]

Is this class name correct? Should it be new network instead of new tokens?

[salimtb]

It's because I'm using an existing CSS class; I don't want to create a new class.

[codecov]

Codecov Report

Attention: Patch coverage is 57.06806% with 82 lines in your changes missing coverage. Please review.

Project coverage is 64.93%. Comparing base (523af49) to head (667e58a).
Report is 14 commits behind head on develop.

Files	Patch %	Lines
.../multichain/network-list-menu/network-list-menu.js	45.16%	34 Missing ⚠️
...ttings/networks-tab/networks-form/networks-form.js	59.38%	26 Missing ⚠️
.../multichain/network-list-item/network-list-item.js	75.00%	5 Missing ⚠️
...i/pages/onboarding-flow/add-network-modal/index.js	37.50%	5 Missing ⚠️
...n/network-list-item-menu/network-list-item-menu.js	80.00%	3 Missing ⚠️
ui/ducks/app/app.ts	0.00%	3 Missing ⚠️
ui/pages/home/home.component.js	25.00%	3 Missing ⚠️
ui/pages/home/home.container.js	0.00%	2 Missing ⚠️
ui/selectors/selectors.js	66.67%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff            @@
##           develop   #25272    +/-   ##
=========================================
  Coverage    64.93%   64.93%            
=========================================
  Files         1385     1386     +1     
  Lines        54870    55010   +140     
  Branches     14408    14455    +47     
=========================================
+ Hits         35625    35717    +92     
- Misses       19245    19293    +48     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[metamaskbot]

Builds ready [667e58a]

• builds: chrome, firefox
• builds (beta): chrome
• builds (flask): chrome, firefox
• builds (MMI): chrome, firefox
• builds (test): chrome, firefox
• builds (test-flask): chrome, firefox
• build viz: Build System
• mv3: Background Module Init Stats
• mv3: UI Init Stats
• mv3: Module Load Stats
• mv3: Bundle Size Stats
• mv2: E2e Actions Stats
• code coverage: Report
• storybook: Storybook
• typescript migration: Dashboard
• all artifacts
• bundle viz:

  • background: 0, 1, 2, 3, 4, 5
  • common: 0, 1, 2, 3, 4, 5, 6, 7, 8, 9
  • content-script: 0
  • offscreen: 0
  • ui: 0, 1, 10, 11, 2, 3, 4, 5, 6, 7, 8, 9

Page Load Metrics (49 ± 5 ms)

Platform	Page	Metric	Min (ms)	Max (ms)	Average (ms)	StandardDeviation (ms)	MarginOfError (ms)
Chrome	Home	firstPaint	64	100	81	10	5
		domContentLoaded	9	21	11	3	1
		load	40	76	49	9	5
		domInteractive	9	21	11	3	1

Bundle size diffs [🚨 Warning! Bundle size has increased!]

• background: 0 Bytes (0.00%)
• ui: 7.8 KiB (0.11%)
• common: 587 Bytes (0.01%)

[EtherWizard33]

ran the branch and looked at the code, LGTM