Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Dashboard / Transaction queue: add the owners icon to the signatures element #1279

Closed
liliiaorlenko opened this issue Nov 29, 2022 · 0 comments · Fixed by #1436
Closed

Dashboard / Transaction queue: add the owners icon to the signatures element #1279

liliiaorlenko opened this issue Nov 29, 2022 · 0 comments · Fixed by #1436
Assignees
@usame-algan
Labels
minor Minor bug

Comments

@liliiaorlenko
Copy link

Bug description

For the design improvement and mobile team alignment I would like to add a owners icon to the signature element, and fix the colours at the element in both light and dark mode
Figma: https://www.figma.com/file/ptTs6lDBeUuLNySroJ5PiF/Web-Master-File?node-id=1%3A3&t=WbjFLE0kr1XPch41-1

Environment

  • Browser: Chrome
  • Wallet: MetaMask
  • Chain: Ethereum mainnet

Steps to reproduce

  1. Go to

Expected result

Obtained result

Screenshots
@katspaugh katspaugh moved this to New issues in Safe{Wallet} Nov 29, 2022
@usame-algan usame-algan moved this from New issues to In Progress in Safe{Wallet} Dec 22, 2022
@usame-algan usame-algan self-assigned this Dec 22, 2022
@usame-algan usame-algan mentioned this issue Dec 22, 2022
Merged
@usame-algan usame-algan moved this from In Progress to Peer Review in Safe{Wallet} Dec 22, 2022
@usame-algan usame-algan moved this from Peer Review to Ready for QA in Safe{Wallet} Dec 22, 2022
@francovenica francovenica moved this from Ready for QA to In Progress in Safe{Wallet} Dec 26, 2022
@usame-algan usame-algan moved this from In Progress to Ready for QA in Safe{Wallet} Jan 2, 2023
@francovenica francovenica moved this from Ready for QA to QA in progress in Safe{Wallet} Jan 2, 2023
@francovenica francovenica moved this from QA in progress to QA done in Safe{Wallet} Jan 2, 2023
@github-project-automation github-project-automation bot moved this from QA done to Done in Safe{Wallet} Jan 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@usame-algan usame-algan

Labels
minor Minor bug
Projects
Safe{Wallet}
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: Add owner icon to confirmations safe-global/safe-wallet-monorepo
2 participants
@usame-algan @liliiaorlenko