Skip to content
This repository has been archived by the owner on Nov 10, 2023. It is now read-only.

[Modals redesign]Hard to read Mixtypes parameter and arrays on action review screen for safe-apps txs #3511

Open
liliya-soroka opened this issue Feb 15, 2022 · 3 comments
Labels
Bug 🐛 Something isn't working Minor Needs to be fixed within the next 1-3 public releases.

Comments

@liliya-soroka
Copy link
Member

Description

Hard to read Mixtypes parameters on action review screen for safe-apps txs

Environment

  • Browser: Chrome
  • Wallet: MetaMask
  • Chain: Rinkeby

Steps to reproduce

  1. Go to the safe-apps
  2. Use Bulla banker app
  3. check "createBullaClaim" action details

The same problem if the parameter is an array

Expected result

separate each value in the mixtypes parameter

Obtained result

No separates between the valuems

Screenshots

image

@liliya-soroka liliya-soroka added Bug 🐛 Something isn't working Minor Needs to be fixed within the next 1-3 public releases. labels Feb 15, 2022
@katspaugh
Copy link
Member

@liliiaorlenko do we have a design for this?

@liliiaorlenko
Copy link

@liliiaorlenko do we have a design for this?

Nop, I can work on it. I will contact @liliya-soroka for details

@liliiaorlenko
Copy link

https://www.figma.com/file/47wkjxMZkyEeCRbCrLJBJa/safe%232995-Tx-details?node-id=2362%3A22517
design based on mobile solution that currently exists

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Bug 🐛 Something isn't working Minor Needs to be fixed within the next 1-3 public releases.
Projects
None yet
Development

No branches or pull requests

3 participants