Skip to content

Conversation

@FrederikBolding
Copy link
Member

@FrederikBolding FrederikBolding commented Mar 7, 2024

Description

This PR changes the Snap UI renderer to use gap for spacing between elements instead of margin. This is more effective than our previous attempts at normalizing spacing with nested panels etc. The new standard is 8px of gap between everything except panels at the root level which has 16px of gap (at the request of @eriknson). This PR removes all the margins for the existing components.

Open in GitHub Codespaces

@FrederikBolding FrederikBolding added the team-snaps-platform-deprecated DEPRECATED: please use "team-core-platform" instead (or "team-new-networks" for Solana snap issues) label Mar 7, 2024
@FrederikBolding FrederikBolding requested a review from a team as a code owner March 7, 2024 13:15
@metamaskbot metamaskbot added the INVALID-PR-TEMPLATE PR's body doesn't match template label Mar 7, 2024
@codecov
Copy link

codecov bot commented Mar 7, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 68.72%. Comparing base (be87264) to head (abdc876).
Report is 3 commits behind head on develop.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop   #23369   +/-   ##
========================================
  Coverage    68.72%   68.72%           
========================================
  Files         1105     1105           
  Lines        43250    43250           
  Branches     11562    11562           
========================================
  Hits         29720    29720           
  Misses       13530    13530           

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

@FrederikBolding FrederikBolding merged commit b3674e3 into develop Mar 7, 2024
@FrederikBolding FrederikBolding deleted the fb/use-gap-for-snap-ui branch March 7, 2024 15:13
@github-actions github-actions bot locked and limited conversation to collaborators Mar 7, 2024
@metamaskbot metamaskbot added the release-11.14.0 Issue or pull request that will be included in release 11.14.0 label Mar 7, 2024
@metamaskbot
Copy link
Collaborator

Builds ready [abdc876]
Page Load Metrics (1126 ± 414 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint761861253718
domContentLoaded1190372210
load6321981126862414
domInteractive1190372210
Bundle size diffs
  • background: 0 Bytes (0.00%)
  • ui: -164 Bytes (-0.00%)
  • common: 0 Bytes (0.00%)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

INVALID-PR-TEMPLATE PR's body doesn't match template release-11.14.0 Issue or pull request that will be included in release 11.14.0 team-snaps-platform-deprecated DEPRECATED: please use "team-core-platform" instead (or "team-new-networks" for Solana snap issues)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants