Skip to content

feat: add shared EstimatedChanges UI component - #369

Open
Julink-eth wants to merge 1 commit into
mainfrom
feat/shared-estimated-changes
Open

Julink-eth wants to merge 1 commit into
mainfrom
feat/shared-estimated-changes

Conversation

@Julink-eth

Copy link
Copy Markdown
Contributor

Explanation

Adds a shared EstimatedChanges Snaps JSX component to @metamask/snap-networks-utils.

The component consolidates the common transaction-confirmation UI for:

Estimated changes header and tooltip
Send and receive asset groups
Asset amount and logo display
Loading and fetching states
Scan error state
Empty estimated-changes state
Unknown asset amounts
The component accepts display-ready assets and translated labels, allowing each Snap to retain ownership of network-specific formatting and localization.

References

Checklist

  • I've updated the test suite for new or updated code as appropriate
  • I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
  • I've communicated my changes to consumers by updating changelogs for packages I've changed
  • I've introduced breaking changes in this PR and have prepared draft pull requests for clients and consumer packages to resolve them

@Julink-eth
Julink-eth requested review from a team as code owners September 24, 2026 15:20
@Julink-eth
Julink-eth deployed to default-branch September 24, 2026 15:20 — with GitHub Actions Active
@sonarqubecloud

Copy link
Copy Markdown

This branch was successfully deployed

1 active deployment
default-branch — 25bae1ee Deployed Sep 24, 2026 by Julink-eth via Determine whether this PR is a release PR #1280
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant