Nft Card Component Solution

Community feedback

• 

  Renato Granados•160

  @Renato6GSPosted about 3 years ago

  Hi from Facebook!

  Very good job Rosman, I like the effect you put on the image.

  I get a small responsive error on screens larger than 20 inches, the card is not centered. And on screens larger than 1024px in height, the card looks too small.

  I can suggest you use "media querys" to adjust the card size and font size according to the user's screen.

  1

  ---

  

  Anthony Rosman•510

  @D3press3ddPosted about 3 years ago

  @Renato6GS thank you, let me check that erros because in my browser looks centered, i will try with other browser, really appreciate your comment

  0
• 

  Yu Dan•30

  @SelfReflectivePosted about 3 years ago

  Great Work Anthony Rosman!

  I especially like the hovering on the main Image its REALLY smooth. Gotta learn how to do that ;)

  One thing that i would like to mention ( its very suddle) is the ETH Symbol, its slightly larger than the text nexto it. If you look closely on the Sample the Icon is exactly in the middle of the text nexto it.

  I did it by putting both elements in a div and then adding flexbox. after that align the items in the center - that should do it. But im sure there are many different way to do it :D

  Cya Yu

  1
• 

  MikevPeeren•2,100

  @MikevPeerenPosted about 3 years ago

  Hey 👋

  Good job finishing this one 👏

  My only comment would be to add more descriptive text for the nft as a screenreader would like to know what they are buying.

  1

  ---

  

  Anthony Rosman•510

  @D3press3ddPosted about 3 years ago

  @MikevPeeren Thank you for your advice Mike, im currently learning so is really big that comment for me♥

  1
• 

  P

  Jairo Valderrama•420

  @jairovgPosted about 3 years ago

  Hi @D3press3dd, here are some comments:

  • You're adding a fixed width to the card element. Instead of doing that, I would suggest you use a max-width on this element, adding some left and right margins on mobile versions and an auto margin on greater devices.
  • Use more semantic elements, like the card-title element, this should be an h1. See more here.
  • I've seen you're using the base font size as 62.5%. I don't know if you did it intentional in order to get a base size of 10px so the font-size gets equivalent values like 1.2rem as 12px. You may have unexpected behaviors if a browser does not have a default font-size as 16px. I suggest you use the base font-size as the one in the style-guide and then calculate the rems for the rest.
  • As @Renato6GS mentioned, there is an issue with large devices, is not a browser related issue. The problem is that you placed a fixed size on the body as 144rem so the card is getting centered inside this fixed width.
  • Consider taking a look on your a11y report.

  PD: Nice transitions on page elements, btw.

  0

  ---

  

  Anthony Rosman•510

  @D3press3ddPosted about 3 years ago

  @jairovg thank you for your feedback, i will do the changes to improve, about the 62.5% i use that intentional because is tha way i learned in this course https://www.udemy.com/course/design-and-develop-a-killer-website-with-html5-and-css3/ , can you suggest where i can learn about rem in base to font size?

  I could not see the error maybe because my screen is just 1366px, maybe is for that reason

  0
• 

  Anthony Rosman•510

  @D3press3ddPosted about 3 years ago

  The tricky part in the hover is the duration of the transitions, thats all,without realizing i damaged the design😥😥, thanks for the observation💞💞 I will accommodate it, I still applied flexbox but maybe changing some things I forgot to fixed again.

  0