[Bug]: When connecting to a website for the first time, the globe icon used as the website icon does not fill the container in the permissions updated toast
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 68/100
- Issue type
- Bug
- Clarity
- Mostly clear
- Activity status
- Quiet
- Tech stack
- react-native, typescript
Research direction
Reproduce the first-time connection flow on iOS and inspect the connection screen and permissions updated toast where the globe icon is rendered. The fix is done when the fallback globe fills its container in both locations, matching the expected screenshots.
Written by the indexing model from the issue text.
Description
Describe the bug
On first connect to a website, the favicon is not present in state, so the globe icon is used instead. But it is smaller than the container. It only takes up 25% of the available space.
Also, the globe icon is not used above the website title in the connect screen. See both screenshots below.
Expected behavior
Globe icon fills the container.
Screenshots/Recordings
Steps to reproduce
- Fresh install of MetaMask
- Visit opensea.io
- Connect with MetaMask
- Observe connection screen / toast
Error messages or log output
Where was this bug found?
Live version (from official store)
Version
7.77.1
Build number
n/a
Build type
None
Device
iPhone 16
Operating system
iOS
Additional context
No response
Severity
No response
- Dominant language
- TypeScript
- Stars
- 3k
- Forks
- 1.7k
- Avg merge
- 1d 14h
- Merged PRs (30d)
- 653
Contributor 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.
More from MetaMask/metamask-mobile
-
client-controller-update ta-bot-triage team-money-movement
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
MetaMask/metamask-mobile#36594 ·
-
client-controller-update ta-triaged team-core-platform
Difficulty 2/5 1-3 hours Newbie friendliness 65/100
MetaMask/metamask-mobile#36424 · 1 comment ·
-
client-controller-update ta-triaged team-assets
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
MetaMask/metamask-mobile#36357 · 1 comment ·
-
client-controller-update ta-triaged team-confirmations
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
MetaMask/metamask-mobile#36346 · 1 comment ·
-
client-controller-update ta-triaged team-money-movement
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
MetaMask/metamask-mobile#36345 · 1 comment ·
All issues in MetaMask/metamask-mobile
Similar issues
-
calcite-components needs triage refactor
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
Esri/calcite-design-system#15203 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 91/100
-
community first-timers-only good first issue hacktoberfest help wanted low hanging fruit up-for-grabs
Difficulty 1/5 Under an hour Newbie friendliness 95/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
Automattic/studio#4908 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 90/100