MetaMask / metamask-mobile

Mobile web browser providing access to websites that use the Ethereum blockchain
https://metamask.io
Other
2.18k stars 1.12k forks source link

fix(431-2): active network icon has too much margin and adding optional prop #12405

Open vinnyhoward opened 3 days ago

vinnyhoward commented 3 days ago

Description

The active network icon in the header has too much margin and is misaligned with the network title.

I have also added an optional prop of networkName that allows developers to pass in the network name of their choice. This is in preparation of multichain support, currently we are showing the selected network name, in multichain we want to be able to pass in a network name and not the selected chain.

Related issues

Fixes:

Manual testing steps

  1. Goto asset details, swaps, or buy. Basically any screen that has the header with network name

Screenshots/Recordings

(green icon at the very top) Before After
asset_details_ios_before asset_details_ios_after

Before

NA

After

NA

Pre-merge author checklist

Pre-merge reviewer checklist

github-actions[bot] commented 3 days ago

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[bot] commented 3 days ago

https://bitrise.io/ Bitrise

✅✅✅ pr_smoke_e2e_pipeline passed on Bitrise! ✅✅✅

Commit hash: 1fdf1bf7674b1147b624600b1c1174b2601f286f Build link: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/5fad9f4e-6f24-461b-b5a3-7dc5b3b53bee

[!NOTE]

  • You can kick off another pr_smoke_e2e_pipeline on Bitrise by removing and re-applying the Run Smoke E2E label on the pull request
sonarcloud[bot] commented 3 days ago

Quality Gate Passed Quality Gate passed

Issues
1 New issue
0 Accepted issues

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

See analysis details on SonarQube Cloud