-
Notifications
You must be signed in to change notification settings - Fork 82
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Display for L1 fees on a transaction #14208
Comments
Designs complete and added to the Send flows here: cc @John-44 |
@noeliaSD is this something that gonna be handled by the ui-team in 2.30, as a part of SendModal improvements? |
I'll assign them to the team but will look at the capacity and the priorty so we can discuss |
We need design for how to handle and show the L1 fees component we see as of now while doing a tx on optimism. Waiting for @benjthayer solution and opening this ticket so that this is not lost
current implementation:
![image](https://private-user-images.githubusercontent.com/60327365/344700432-a56d8a20-dcc4-4183-9835-95227963866f.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3Mzk2MjAzNTAsIm5iZiI6MTczOTYyMDA1MCwicGF0aCI6Ii82MDMyNzM2NS8zNDQ3MDA0MzItYTU2ZDhhMjAtZGNjNC00MTgzLTk4MzUtOTUyMjc5NjM4NjZmLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNTAyMTUlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjUwMjE1VDExNDczMFomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPWI5MjE4NzQ2NmE5ZWI3YTZlOGUzMjYxNzBmYjgxMjNmZmViNzQ4YWVjYzM0ZmI3ZTA1MjU3YzAxMzg1NGQ2ZmUmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0In0.A_cI1eHbMsk5dGCNcl4OkqGSoE1MWwta7P2jiSt4oq0)
expected:
![image](https://private-user-images.githubusercontent.com/60327365/344700565-6027c4c6-bb4d-490e-95c3-973dd759e2b2.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3Mzk2MjAzNTAsIm5iZiI6MTczOTYyMDA1MCwicGF0aCI6Ii82MDMyNzM2NS8zNDQ3MDA1NjUtNjAyN2M0YzYtYmI0ZC00OTBlLTk1YzMtOTczZGQ3NTllMmIyLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNTAyMTUlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjUwMjE1VDExNDczMFomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPWNlMjIxN2E0M2I4YjU3ODQ1ODkwMDhkMTdkNWRlOTVjNzkyMzNmZTk3OTRjOGNhMWFkMmVhNWI3NDU1M2Q1MzUmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0In0.2IkI_Su4OU1CSHBjGe76TqY7k8nuAnN53Knz1Qpwt8Y)
The text was updated successfully, but these errors were encountered: