Skip to content

Conversation

@GuillaumeRx
Copy link
Contributor

@GuillaumeRx GuillaumeRx commented Jan 6, 2026

Description

This PR fixes the info dot of the menu item by using a div instead of the Icon that doesn't have a background.

Open in GitHub Codespaces

Changelog

CHANGELOG entry: fix MenuItem info dot

Related issues

Fixes:

Manual testing steps

  1. look at the info dot in the extension menu

Screenshots/Recordings

Before

Screenshot 2026-01-06 at 15 47 14

After

Screenshot 2026-01-06 at 15 49 54

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

Fix menu info dot rendering

  • Replace badge Icon with a styled Box (10px circle, primaryDefault background) for the info dot in MenuItem
  • Adjust BadgeWrapper positionObj from { top: -6, right: 4 } to { top: 0, right: 4 }
  • Add imports for Box, BackgroundColor, and BorderRadius

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

@github-actions
Copy link
Contributor

github-actions bot commented Jan 6, 2026

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.

@metamaskbot metamaskbot added the team-core-platform Core Platform team label Jan 6, 2026
@github-actions github-actions bot added the size-S label Jan 6, 2026
@metamaskbotv2
Copy link

metamaskbotv2 bot commented Jan 6, 2026

Builds ready [4264832]
UI Startup Metrics (1311 ± 110 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyStandard HomeuiStartup13111014168811013771467
load1091863146510211671229
domContentLoaded1085857145910211631224
domInteractive2715106202385
firstPaint197681202207202325
backgroundConnect22220127211229240
firstReactRender1493151628
getState361774114060
initialActions108112
loadScripts87065812431019481005
setupStore1355161228
numNetworkReqs181181171169
BrowserifyPower User HomeuiStartup------
load------
domContentLoaded------
domInteractive------
firstPaint------
backgroundConnect------
firstReactRender------
getState------
initialActions------
loadScripts------
setupStore------
numNetworkReqs------
WebpackStandard HomeuiStartup806658119780858925
load64958591564669800
domContentLoaded64458090863662793
domInteractive2616148212280
firstPaint1066232255121222
backgroundConnect2551333030110
firstReactRender15113541824
getState3415133204170
initialActions104113
loadScripts64157890062659783
setupStore1374781439
numNetworkReqs171173171169
WebpackPower User HomeuiStartup------
load------
domContentLoaded------
domInteractive------
firstPaint------
backgroundConnect------
firstReactRender------
getState------
initialActions------
loadScripts------
setupStore------
numNetworkReqs------
FirefoxBrowserifyStandard HomeuiStartup15101203193415216231810
load12091032158711112641426
domContentLoaded12091026158711112641425
domInteractive893724247107209
firstPaint------
backgroundConnect722334757106185
firstReactRender13102221418
getState146108161249
initialActions103122
loadScripts116199014988812181328
setupStore155200211345
numNetworkReqs21986211583
BrowserifyPower User HomeuiStartup25621488355554130183206
load17651015274153422532479
domContentLoaded17651015274153422522479
domInteractive903391911085298
firstPaint------
backgroundConnect3562013353594511097
firstReactRender19126272328
getState1016529332118155
initialActions207123
loadScripts1515996256846720012302
setupStore625101714669208
numNetworkReqs69301763278159
WebpackStandard HomeuiStartup15121254196116115751873
load12821089153810013331477
domContentLoaded12811089153810013331476
domInteractive772718137102132
firstPaint------
backgroundConnect53172043459124
firstReactRender14104441521
getState157206221445
initialActions104122
loadScripts1248107614979313041432
setupStore164138221450
numNetworkReqs19985191377
WebpackPower User HomeuiStartup26831652339953931183337
load20261096276753324862690
domContentLoaded20261096276753424862690
domInteractive7329104111066161
firstPaint------
backgroundConnect3372611653514081083
firstReactRender2013154152231
getState966223531102168
initialActions203122
loadScripts17741081274446322192597
setupStore3352664349121
numNetworkReqs66211863771170
📊 Page Load Benchmark Results

Current Commit: 4264832 | Date: 1/6/2026

📄 Localhost MetaMask Test Dapp

Samples: 100

Summary

  • pageLoadTime-> current mean value: 1.04s (±44ms) 🟡 | historical mean value: 1.05s ⬇️ (historical data)
  • domContentLoaded-> current mean value: 723ms (±40ms) 🟢 | historical mean value: 731ms ⬇️ (historical data)
  • firstContentfulPaint-> current mean value: 76ms (±13ms) 🟢 | historical mean value: 77ms ⬇️ (historical data)

📈 Detailed Results

Metric Mean Std Dev Min Max P95 P99
pageLoadTime 1.04s 44ms 1.01s 1.34s 1.06s 1.34s
domContentLoaded 723ms 40ms 697ms 1.00s 736ms 1.00s
firstPaint 76ms 13ms 60ms 196ms 84ms 196ms
firstContentfulPaint 76ms 13ms 60ms 196ms 84ms 196ms
largestContentfulPaint 0ms 0ms 0ms 0ms 0ms 0ms
Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 58 Bytes (0%)
  • ui: 23 Bytes (0%)
  • common: 20 Bytes (0%)

@FrederikBolding FrederikBolding requested a review from Mrtenz January 7, 2026 08:47
@metamaskbotv2
Copy link

metamaskbotv2 bot commented Jan 7, 2026

Builds ready [4264832]
UI Startup Metrics (1311 ± 110 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyStandard HomeuiStartup13111014168811013771467
load1091863146510211671229
domContentLoaded1085857145910211631224
domInteractive2715106202385
firstPaint197681202207202325
backgroundConnect22220127211229240
firstReactRender1493151628
getState361774114060
initialActions108112
loadScripts87065812431019481005
setupStore1355161228
numNetworkReqs181181171169
BrowserifyPower User HomeuiStartup------
load------
domContentLoaded------
domInteractive------
firstPaint------
backgroundConnect------
firstReactRender------
getState------
initialActions------
loadScripts------
setupStore------
numNetworkReqs------
WebpackStandard HomeuiStartup806658119780858925
load64958591564669800
domContentLoaded64458090863662793
domInteractive2616148212280
firstPaint1066232255121222
backgroundConnect2551333030110
firstReactRender15113541824
getState3415133204170
initialActions104113
loadScripts64157890062659783
setupStore1374781439
numNetworkReqs171173171169
WebpackPower User HomeuiStartup------
load------
domContentLoaded------
domInteractive------
firstPaint------
backgroundConnect------
firstReactRender------
getState------
initialActions------
loadScripts------
setupStore------
numNetworkReqs------
FirefoxBrowserifyStandard HomeuiStartup15101203193415216231810
load12091032158711112641426
domContentLoaded12091026158711112641425
domInteractive893724247107209
firstPaint------
backgroundConnect722334757106185
firstReactRender13102221418
getState146108161249
initialActions103122
loadScripts116199014988812181328
setupStore155200211345
numNetworkReqs21986211583
BrowserifyPower User HomeuiStartup25621488355554130183206
load17651015274153422532479
domContentLoaded17651015274153422522479
domInteractive903391911085298
firstPaint------
backgroundConnect3562013353594511097
firstReactRender19126272328
getState1016529332118155
initialActions207123
loadScripts1515996256846720012302
setupStore625101714669208
numNetworkReqs69301763278159
WebpackStandard HomeuiStartup15121254196116115751873
load12821089153810013331477
domContentLoaded12811089153810013331476
domInteractive772718137102132
firstPaint------
backgroundConnect53172043459124
firstReactRender14104441521
getState157206221445
initialActions104122
loadScripts1248107614979313041432
setupStore164138221450
numNetworkReqs19985191377
WebpackPower User HomeuiStartup26831652339953931183337
load20261096276753324862690
domContentLoaded20261096276753424862690
domInteractive7329104111066161
firstPaint------
backgroundConnect3372611653514081083
firstReactRender2013154152231
getState966223531102168
initialActions203122
loadScripts17741081274446322192597
setupStore3352664349121
numNetworkReqs66211863771170
📊 Page Load Benchmark Results

Current Commit: 4264832 | Date: 1/6/2026

📄 Localhost MetaMask Test Dapp

Samples: 100

Summary

  • pageLoadTime-> current mean value: 1.04s (±44ms) 🟡 | historical mean value: 1.05s ⬇️ (historical data)
  • domContentLoaded-> current mean value: 723ms (±40ms) 🟢 | historical mean value: 731ms ⬇️ (historical data)
  • firstContentfulPaint-> current mean value: 76ms (±13ms) 🟢 | historical mean value: 78ms ⬇️ (historical data)

📈 Detailed Results

Metric Mean Std Dev Min Max P95 P99
pageLoadTime 1.04s 44ms 1.01s 1.34s 1.06s 1.34s
domContentLoaded 723ms 40ms 697ms 1.00s 736ms 1.00s
firstPaint 76ms 13ms 60ms 196ms 84ms 196ms
firstContentfulPaint 76ms 13ms 60ms 196ms 84ms 196ms
largestContentfulPaint 0ms 0ms 0ms 0ms 0ms 0ms
Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 58 Bytes (0%)
  • ui: 23 Bytes (0%)
  • common: 20 Bytes (0%)

@GuillaumeRx GuillaumeRx added this pull request to the merge queue Jan 7, 2026
Merged via the queue into main with commit 19da8a3 Jan 7, 2026
331 of 335 checks passed
@GuillaumeRx GuillaumeRx deleted the gr/fix-notification-dot branch January 7, 2026 10:26
@github-actions github-actions bot locked and limited conversation to collaborators Jan 7, 2026
@metamaskbot metamaskbot added the release-13.14.0 Issue or pull request that will be included in release 13.14.0 label Jan 7, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

release-13.14.0 Issue or pull request that will be included in release 13.14.0 size-S team-core-platform Core Platform team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants