search

0xProject / 0x-launch-kit-frontend

Apache License 2.0
112 stars 207 forks source link

Show your own NFT in “for sale” view #521

Closed mariano-aguero closed 5 years ago

chriskalani commented 5 years ago

@burnsben is there a design for this?

burnsben commented 5 years ago

Here's what I've come up with.

Screen Shot 2019-06-03 at 2 27 42 PM

Design can be found here: https://www.figma.com/file/RW4gjUf5pfu2ajO1BvVWfi/Launch-Kit-ERC-721?node-id=59%3A0

unjapones commented 5 years ago

@chriskalani @burnsben does this issue implies adding to the corresponding listings ... ?

  1. only my collectibles that are on sale, or
  2. all my collectibles
chriskalani commented 5 years ago

Let's put it on all views

unjapones commented 5 years ago

@chriskalani wrote:

Let's put it on all views

:+1:

But there is another not so evident question here. I'll add some context:

Currently, the listings "Recenlty listed" and "Most valued" only show collectibles from other users that are one sale.

If we got it right, the issue is also about adding "You own" badge _and also

show your collectibles in the lists "Recenlty listed" and "Most valued.

Did we got this last part ok? If the answer is yes, should we show only my collectibles that are on sale or all my collectibles?

chriskalani commented 5 years ago

If your own item happens to match that criteria then I think it should show up in those sections as well.