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.

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

UI issues related to PR 2440/uni merge #339

Closed
11 of 12 tasks
anxolin opened this issue Apr 2, 2022 · 0 comments · Fixed by #380
Closed
11 of 12 tasks

UI issues related to PR 2440/uni merge #339

anxolin opened this issue Apr 2, 2022 · 0 comments · Fixed by #380
Labels
app:CowSwap CowSwap app Bug Something isn't working Medium Severity indicator. It causes some undesirable behavior, but the system is still functional

Comments

@anxolin
Copy link
Contributor

anxolin commented Apr 2, 2022

Related to #2536

  • 1. 'Connect to a wallet' button's boarder has a light boarder and seems like have a different boarder radius
    image.png
    image.png
    image

  • 2. Profile page: 'Connect to a wallet' button is too wide
    image.png

  • 3. Text in the network selector is almost invisible in the dark mode
    image.png

  • 4. Networks in the selector are not aligned
    image.png

  • 5. Token lists exceed the modal area
    image.png

  • 6. Arrow before' Add a send' field has too bold boarder + center-aligned
    image.png
    image.png

  • 7. Naming is inconsistent: 'Add a send' and 'Remove recipient'. I think, we need to use 'recipient' for both labels (or at least 'send' for both, as it is currently on Prod)
    image.png

  • 8. Too big tooltips
    image.png

  • 9. Account Icon on the 'Confirm' modal is too small
    image
    -The same is on the 'Claim' page
    image

  • 10. Network selector exceeds right screen boarder in mobile devices
    image

  • 11. Network selector is opened 'on hover' event instead of the 'on-click' as we currently use on Prod. So when I press on the network selector twice, it remains opened until I remove a mouse from it. Will we leave it as it is or change to the behavior we currently use?

  • 12. Sometimes I can't open the network selector after a network changing in a mobile view

Original issue gnosis/cowswap#2626 by @elena-zh

@anxolin anxolin added Medium Severity indicator. It causes some undesirable behavior, but the system is still functional app:CowSwap CowSwap app Bug Something isn't working labels Apr 2, 2022
@elena-zh elena-zh mentioned this issue Apr 4, 2022
@elena-zh elena-zh mentioned this issue Apr 6, 2022
24 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
app:CowSwap CowSwap app Bug Something isn't working Medium Severity indicator. It causes some undesirable behavior, but the system is still functional
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant