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

Cart system feedbacks #489

Closed
NicolasMahe opened this issue Nov 29, 2023 · 3 comments · Fixed by #526
Closed

Cart system feedbacks #489

NicolasMahe opened this issue Nov 29, 2023 · 3 comments · Fixed by #526
Assignees

Comments

@NicolasMahe
Copy link
Member

Let's centralize here the cart system feedbacks to not block PR #471

@NicolasMahe
Copy link
Member Author

NicolasMahe commented Nov 29, 2023

  • Add a go back button when displaying an error. It's really annoying to close the cart and reopen again
Screenshot 2023-11-29 at 15 48 32
  • Replace the network icon on the button "Continue with [icon]" by the network name
Screenshot 2023-11-29 at 15 50 21
  • The hover and click area on the nft in the cart doesn't match. I need to click on the picture or texts to be redirected to the nft.
Screenshot 2023-11-29 at 15 53 10
  • Add html hint / tooltip on the add to cart and remove from cart button. As a user I'm not sure what action the icons are going to trigger, specially the - button: Use a better icon.
Screenshot 2023-11-29 at 15 54 27
  • Open the cart when I add a NFT. It's not super obvious how to open the cart.

  • No more modal with the transaction status? Specially when doing the approval, it takes time and I don't have much info.
    Moved to Cart system improvements #529

  • There is no selection of quantity for ERC1155. It purchases one edition only.
    Moved to Cart system improvements #529

@NicolasMahe NicolasMahe mentioned this issue Nov 29, 2023
11 tasks
@EmmanuelDrouin
Copy link
Member

EmmanuelDrouin commented Dec 8, 2023

Feedback based on Nicolas proposal:

  • I agree on all feedback if I do not comment those

The hover and click area on the nft in the cart doesn't match. I need to click on the picture or texts to be redirected to the nft.

I think we should only give the ability here to get redirected to the item and not the collection. I feel like this will be more efficient for the users. This is the behavrio on Opensea and Rarible, you can only get redirected to the item, no link to the collection

Open the cart when I add a NFT. It's not super obvious how to open the cart.

This is not something we should do, it will become very annoying for users to have to close the cart every time they add items. You can compare with OpenSea, Rarible, Looksrare and so on, nobody is opening the cart when adding an item.

My feedback

UI

  • The Width of the cart is pretty small, compared to Opensea and Rarible I think we could extend it to give more spaces for the content (currently very very truncated)
  • Summary screen > Keep the same consistency in displaying assets in terms of style as the previous screen (rounded corner, not fully rounded, same info with collection/name). As well as the same order, between those 2 screens the order changes which downgrades the accessibility for a user
    • First screen
      • Screen Shot 2566-12-08 at 17 59 48
    • Second screen with the wrong style
      • Screen Shot 2566-12-08 at 18 00 30

UX

Purchase with all items on the same chain (only 1 chain in the cart)

  • After you successfully purchase your asset, the actionable route is only the explorer.
    • Change style for the secondary button
    • Add the primary button below to redirect to the user "owned" page to see the NFTs he just purchased

Purchase with items on different chains (multiple chains in the cart)

  • add a button to continue checkout on another chain without "closing/re-opening" the cart

@ismailToyran ismailToyran linked a pull request Jan 16, 2024 that will close this issue
1 task
@EmmanuelDrouin
Copy link
Member

@ismailToyran did you implement my feedback UI/UX as well from this issue? Can we close this issue if all the work left is tracked on dedicated issues?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants