All comments

• 

  Brandon• 80

  @oallelsefailo

  Submitted almost 2 years ago

  I'm not sure if I did the text 'correctly' in terms of positioning. I used padding to try to line it up with the image but i'm not sure if there's a better way to ensure they are aligned.

  I also feel like I used an excessive amount of classes..

  The on hover for the main image was a bit of a challenge but I think it came out well.

  

  NFT Preview Card with HTML/CSS

  1

  7

  0

  

  Maria• 130

  @maaghia

  Posted almost 2 years ago

  I can see that you did a really good job cloning that card. however I have a small tip concerning the img tag <img src="images/image-equilibrium.jpg">, make sure you add some alternate text. the alternate text shows in case the image fails to load or the user uses a screen reader (blind people do) you can put in there the name of the image or some expression that describes it

  0
• 

  Maria• 130

  @maaghia

  Submitted almost 2 years ago

  Challenge completed in less than 2h. Any feedback for a better or more optimized approach is highly appreciated!

  

  qr-code-component

  2

  4

  0

  

  Maria• 130

  @maaghia

  Posted almost 2 years ago

  Your comment helped me understand the way accessibility issues should be handled I will take this into consideration thank you.

  0
• 

  Maria• 130

  @maaghia

  Submitted almost 2 years ago

  I got a little bit confused with th edimentions of components, I might try to adjest them in a second trial. All your comments are wellcome.

  

  order-summary-component

  2

  2

  0

  

  Maria• 130

  @maaghia

  Posted almost 2 years ago

  This was very helpful, thank you

  0