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

[UX]: NFT gallery view improvements #1590

Open
2 of 9 tasks
buberdds opened this issue Oct 23, 2024 · 4 comments
Open
2 of 9 tasks

[UX]: NFT gallery view improvements #1590

buberdds opened this issue Oct 23, 2024 · 4 comments
Assignees
Labels
ux/design UX/UI Design Change Report

Comments

@buberdds
Copy link
Contributor

IMPACT

  • High
  • Medium
  • Low

URGENCY

  • Critical
  • Major
  • Medium
  • Minor

PRODUCT ENVIRONMENT

No response

DESCRIPTION OF ISSUE

During demo day there were some ideas about improving NFT gallery/inventory.

One of an idea was to add additional "Next" button that looks more like gallery item.

DEVICE / VIEWPORT

  • Desktop
  • Mobile

STEPS TO REPRODUCE

No response

ALTERNATIVES AND SUGGESTIONS

No response

@buberdds buberdds added the ux/design UX/UI Design Change Report label Oct 23, 2024
@donouwens
Copy link
Collaborator

@buberdds yeah good point. I've added nav buttons in there now (see screenshot attached) - let me know if that works for you. Where there any other ideas you want to share?

Screenshot 2024-11-01 at 10 34 41

@buberdds
Copy link
Contributor Author

buberdds commented Nov 4, 2024

Gallery is rendered under "Inventory" tab not in nft details page

@donouwens
Copy link
Collaborator

Gallery is rendered under "Inventory" tab not in nft details page

Ah, the overview page. I understand the proposal, but don't see why we would move away from the navigation pattern that we use everywhere else. Was there any argumentation for that?

@buberdds
Copy link
Contributor Author

buberdds commented Nov 14, 2024

why we would move away from the navigation pattern that we use everywhere else.

Where we use such navigation pattern?

During the demo no one requested adding previous/next navigation on individual instance pages. On top of that, how would we even determine the IDs for "next" and "previous" in cases like this one:https://explorer.oasis.io/mainnet/emerald/token/0xF88C56280acec4Ca4b5b6ab93D53DA1A73c8D7CD/instance/65467268601666 ? I don’t think Nexus API would be extended to return such info.

I don't think NFT views are worth investing time. Personally, I would either ignore this request or keep any updates to a minimum.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ux/design UX/UI Design Change Report
Projects
None yet
Development

No branches or pull requests

2 participants