Skip to content

Conversation

@davidmurdoch
Copy link
Contributor

@davidmurdoch davidmurdoch commented Jul 16, 2025

Description

popup-init.html can't use CSS, so we manually keep its colors in sync with our design language. This screen only shows for a few milliseconds, so most won't ever notice it.

Open in GitHub Codespaces

@davidmurdoch davidmurdoch added the no-changelog no-changelog Indicates no external facing user changes, therefore no changelog documentation needed label Jul 16, 2025
@github-actions
Copy link
Contributor

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-extension-platform Extension Platform team label Jul 16, 2025
@metamaskbot
Copy link
Collaborator

Builds ready [258eca2]
UI Startup Metrics (1194 ± 62 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1194109514016212231325
load102294511845110581114
domContentLoaded101694011755010491108
domInteractive18134371642
firstPaint66769117141210221086
backgroundConnect2512362646255262
firstReactRender19153542028
getState10376101328
initialActions30255215
loadScripts77070392348800863
setupStore74203712
WebpackHomeuiStartup22641736281221823632724
load16671249209817917792007
domContentLoaded16591245207517717701998
domInteractive1913152181560
firstPaint1616343877183338
backgroundConnect271174123156
firstReactRender26790426117376402
getState164312321545
initialActions13219631841
loadScripts16551242206317517651985
setupStore197258262046
FirefoxBrowserifyHomeuiStartup14101189211715314551726
load1218104414719012851370
domContentLoaded1217104414719012841370
domInteractive1123653079107328
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect2414103122350
firstReactRender24206152529
getState10214919744
initialActions30325210
loadScripts1197102714518912611351
setupStore11316921656
WebpackHomeuiStartup16991427281722517512168
load14341213213416415041837
domContentLoaded14331213213416415041837
domInteractive105323676896303
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect25165682842
firstReactRender55426755964
getState11318120937
initialActions40648314
loadScripts14141197210916314861814
setupStore14427136825
Benchmark value 251 exceeds gate value 10 for chrome browserify home mean backgroundConnect
Benchmark value 3 exceeds gate value 1 for chrome browserify home mean initialActions
Benchmark value 42 exceeds gate value 41 for chrome browserify home p95 domInteractive
Benchmark value 262 exceeds gate value 18 for chrome browserify home p95 backgroundConnect
Benchmark value 15 exceeds gate value 1.2 for chrome browserify home p95 initialActions
Benchmark value 2264 exceeds gate value 2192 for chrome webpack home mean uiStartup
Benchmark value 267 exceeds gate value 235 for chrome webpack home mean firstReactRender
Benchmark value 13 exceeds gate value 7 for chrome webpack home mean initialActions
Benchmark value 2725 exceeds gate value 2454 for chrome webpack home p95 uiStartup
Benchmark value 61 exceeds gate value 57 for chrome webpack home p95 domInteractive
Benchmark value 339 exceeds gate value 334 for chrome webpack home p95 firstPaint
Benchmark value 402 exceeds gate value 370 for chrome webpack home p95 firstReactRender
Benchmark value 41 exceeds gate value 7 for chrome webpack home p95 initialActions
Benchmark value 1986 exceeds gate value 1970 for chrome webpack home p95 loadScripts
Benchmark value 1410 exceeds gate value 1405 for firefox browserify home mean uiStartup
Benchmark value 112 exceeds gate value 110 for firefox browserify home mean domInteractive
Benchmark value 4 exceeds gate value 1 for firefox browserify home mean initialActions
Benchmark value 12 exceeds gate value 9 for firefox browserify home mean setupStore
Benchmark value 1726 exceeds gate value 1660 for firefox browserify home p95 uiStartup
Benchmark value 328 exceeds gate value 195 for firefox browserify home p95 domInteractive
Benchmark value 44 exceeds gate value 24 for firefox browserify home p95 getState
Benchmark value 10 exceeds gate value 2 for firefox browserify home p95 initialActions
Benchmark value 56 exceeds gate value 27 for firefox browserify home p95 setupStore
Benchmark value 1700 exceeds gate value 1615 for firefox webpack home mean uiStartup
Benchmark value 1434 exceeds gate value 1380 for firefox webpack home mean load
Benchmark value 1434 exceeds gate value 1380 for firefox webpack home mean domContentLoaded
Benchmark value 106 exceeds gate value 100 for firefox webpack home mean domInteractive
Benchmark value 56 exceeds gate value 38 for firefox webpack home mean firstReactRender
Benchmark value 5 exceeds gate value 1 for firefox webpack home mean initialActions
Benchmark value 1414 exceeds gate value 1360 for firefox webpack home mean loadScripts
Benchmark value 15 exceeds gate value 13 for firefox webpack home mean setupStore
Benchmark value 2168 exceeds gate value 1935 for firefox webpack home p95 uiStartup
Benchmark value 1837 exceeds gate value 1660 for firefox webpack home p95 load
Benchmark value 1837 exceeds gate value 1660 for firefox webpack home p95 domContentLoaded
Benchmark value 303 exceeds gate value 156 for firefox webpack home p95 domInteractive
Benchmark value 64 exceeds gate value 50 for firefox webpack home p95 firstReactRender
Benchmark value 37 exceeds gate value 32 for firefox webpack home p95 getState
Benchmark value 14 exceeds gate value 2 for firefox webpack home p95 initialActions
Benchmark value 1814 exceeds gate value 1630 for firefox webpack home p95 loadScripts
Sum of mean exceeds: 643ms | Sum of p95 exceeds: 1825.8ms
Sum of all benchmark exceeds: 2468.8ms

Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 43 Bytes (0%)
  • ui: 0 Bytes (0%)
  • common: 15 Bytes (0%)

Copy link
Contributor

@georgewrmarshall georgewrmarshall left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@davidmurdoch davidmurdoch added this pull request to the merge queue Jul 18, 2025
@github-project-automation github-project-automation bot moved this from Needs dev review to Review finalised - Ready to be merged in PR review queue Jul 18, 2025
@DDDDDanica
Copy link
Contributor

LGTM!

Merged via the queue into main with commit 00ca721 Jul 18, 2025
279 of 281 checks passed
@davidmurdoch davidmurdoch deleted the fix-popup-init-bg-colors branch July 18, 2025 17:28
@github-actions github-actions bot locked and limited conversation to collaborators Jul 18, 2025
@metamaskbot metamaskbot added the release-13.1.0 Issue or pull request that will be included in release 13.1.0 label Jul 18, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

no-changelog no-changelog Indicates no external facing user changes, therefore no changelog documentation needed release-13.1.0 Issue or pull request that will be included in release 13.1.0 team-extension-platform Extension Platform team

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

5 participants