Skip to content

Conversation

pandablue0809
Copy link
Member

@pandablue0809 pandablue0809 requested a review from ihomp September 12, 2025 15:16
@pandablue0809
Copy link
Member Author

@ihomp
i fixed it. plz review again. 🙏

@Anna15170221
Copy link
Contributor

Anna15170221 commented Oct 2, 2025

@pandablue0809
I think we shouldn't show any info on nft if it is not readable. If you keep some info, let's make sure it can be read.

Screenshot 2025-10-02 at 14 08 32

@Anna15170221
Copy link
Contributor

@pandablue0809
Frankly speaking I don't like the way it actually looks, probably we need to optimise and show everything in 1 style.
For example last sold nfts are out of the table, header in small characters, Nft offers are in the table, header is of different font size. It all looks disordered everywhere.
Screenshot 2025-10-02 at 14 28 09
Screenshot 2025-10-02 at 14 28 55

@ihomp
Copy link
Member

ihomp commented Oct 2, 2025

Agree with Anna's points.

also on the desktop we can show more than 3 - as there is space left..

@pandablue0809
Copy link
Member Author

@ihomp @Anna15170221
i fixed it. plz review again. 🙏

@Anna15170221
Copy link
Contributor

@pandablue0809
for me looks better now, but the info on nft pictures is still not readable = cut
Screenshot 2025-10-06 at 15 39 24

@pandablue0809
Copy link
Member Author

@ihomp
i fixed it. plz review again. 🙏

@Anna15170221
Copy link
Contributor

@pandablue0809

  • since you are now showing lots of nfts, the phrase "recently acquired" is not suitable (some of them were minted /purchased several years ago). Let's replace it with Owned NFTs
  • If nft is with 18+ content - we don't show the picture, we show this
Screenshot 2025-10-08 at 16 23 05 Screenshot 2025-10-08 at 16 38 15

@pandablue0809
Copy link
Member Author

pandablue0809 commented Oct 8, 2025

@ihomp
i fixed except 18+ content.
i can't work that.
can you fix it?

@ihomp ihomp merged commit bc492c3 into Bithomp:main Oct 9, 2025
@pandablue0809 pandablue0809 deleted the feat/588/update-showing-nft branch October 9, 2025 07:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants