DimensionDev / Maskbook

The portal to the new, open Internet. ([I:b])
https://mask.io
GNU Affero General Public License v3.0
1.51k stars 311 forks source link

[Demand] Remove redundant transaction dialog #4789

Closed zhouhanseng closed 2 years ago

zhouhanseng commented 3 years ago

Catalogue

Part 1: Introduction and Background

We shipped a new contract interaction popup recently which is global sensing, meanwhile we need to call useRemoteControlledDialog() at every place we want to use the transaction dialog that is less readable. And the multiple modal background shadow is visually undesirable when using transaction dialog with other dialog, even worse with the Fortmatic popups.

image

Part 2: Problem to Solve

Part 3: Expected Behavior

Notes

stale[bot] commented 2 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.