All comments

• 

  Andrew Festa• 50

  @afesta12

  Submitted about 2 years ago

  My solution feels quite a bit 'hacky', which is the best word I could come up with.

  I could not get things to line up correctly using flex, so I switched the layout between flex and grid during a media query, which I'm assuming is not a best practice solution.

  I did learn a bit about displaying different images, but is there a preferred method to doing this? Display none seemed to work, but I also read about people using a background image.

  Thank you, and any feedback is highly appreciated! =)

  

  Responsive Card Using Grid and Flex

  6

  80

  

  allys• 90

  @allyskorsah

  Posted about 2 years ago

  Hello @afesta12 well done on completing the challenge. I also used the same methods you used. I don't think switching between flex and grid is that bad. I would have to check out the display none thing though.

  1

  This comment has received 1 replyShow
• 

  Toncea Dorinel Ionut• 30

  @DorinelIonut

  Submitted about 2 years ago

  I want an opinion about this one, is the best i could do just by myself withouth any help. If you think i can improve something pls leave an ideea about that. Thanks

  

  NFT-preview-card-component

  2

  20

  

  allys• 90

  @allyskorsah

  Posted about 2 years ago

  @Dorinellonut good work. However you should also work on the active state parts of the project. And for the overlay part you can check out my solution for the code. It may not work 100% for yours but you can use it sort of like a guide. And look at some materials on how css overlay works. I had to play around with the codes before I got what I presented. It maybe frustrating if you don't get it at first but take a deep breath 😮‍💨,calm down 😌 and go over your materials again.

  @Dorinellonut well done. 👍

  Marked as helpful

  0
• 

  Toncea Dorinel Ionut• 30

  @DorinelIonut

  Submitted about 2 years ago

  I want an opinion about this one, is the best i could do just by myself withouth any help. If you think i can improve something pls leave an ideea about that. Thanks

  

  NFT-preview-card-component

  2

  20

  

  allys• 90

  @allyskorsah

  Posted about 2 years ago

  Hello. You should check the link you put out for the solution on github. The link directs to the code for the QR code thing and not the NFT challenge.

  That aside good job 👏 👍, but I want to know, did you add the code for the active design?

  Marked as helpful

  0

  This comment has received 2 repliesShow