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

New gallery item detail 3.0 #958

Closed
Tracked by #1619
yangwao opened this issue Oct 29, 2021 · 3 comments
Closed
Tracked by #1619

New gallery item detail 3.0 #958

yangwao opened this issue Oct 29, 2021 · 3 comments
Labels
$$ ~51-150usd p2 core functionality, or is affecting 60% of app

Comments

@yangwao
Copy link
Member

yangwao commented Oct 29, 2021

We've had a quick designer session to knock some bugs out

  • Align description with JPEG
  • put padding inside, probably same as actions?

image

  • put on same level
    image

#955 plus link facts there as well under price chart as we assume no one is checking it but keep it for someone
#957 show only when you can do something with nft
#956
#990 will be done separately

Store folded preferences in Vuex so whenever you open another detail it would have a state of current one

Contextual NFTs should be foldable as well.
Default, price chart, history chart and contextual nfts should be folded
Mind that Collection, Creator was moved under price

139410941-97cfa445-4dd4-42c6-9c7a-dc547416aae7

@yangwao yangwao added the p2 core functionality, or is affecting 60% of app label Oct 29, 2021
@yangwao yangwao added the $$ ~51-150usd label Nov 11, 2021
@yangwao
Copy link
Member Author

yangwao commented Nov 11, 2021

Currently @MiracleCorunus is working on this

I guess we should have a separate interface for

  • creator
  • collector

As it's different personas.

Under collector, you want to collect
Under creator, you manage your portfolio.

Let's start with collector one, an important one for us.
Merge Price box with Actions if it's for Collecting and not the owner of NFT

When creator, showing rest buttons would work in the same place.

Put it in a place where is an existing action component is, price on the top.
Sharing under the same place. Check #1063 #1082 done by @roiLeo

The history chart should be full width.

image

@yangwao
Copy link
Member Author

yangwao commented Nov 18, 2021

Continue to #1209

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
$$ ~51-150usd p2 core functionality, or is affecting 60% of app
Projects
None yet
Development

No branches or pull requests

1 participant