Card using Flexbox to center, and variables for colors.

Community feedback

• 

  P

  Fluffy Kas•7,715

  @FluffyKasPosted over 2 years ago

  Heyo,

  I think you did great here. Apart from the things mentioned by @Samadeen, you could look into the following:

  1. Would be nice if you wrapped the layered image in an <a> tag, as I'm fairly sure it's supposed to take you somewhere when it's clicked on.

  2. Here's a useful article about alt texts. I suggest reading it but I'm going to sum up the parts relevant here: when writing alts, don't include words like image, icon, illustration etc. Using an <img> tag makes it self-explanatory what it is. For profile pictures, it's best to use the name of the person (here, it would be "Jules Wyvern."). For the icon, "Ethereum." would be a bit better.

  Marked as helpful

  2

  ---

  

  Fizzify•20

  @FizzifyPosted over 2 years ago

  @FluffyKas Wow! Yeah this seems to be helpful. I've never known about this and I do agree with you with the img tag. Thank you, cheers.

  0
• 

  Abdul•8,540

  @SamadeenPosted over 2 years ago

  Hey!! Cheers 🥂 on completing this challenge.. .

  Here are my suggestions..

  • You should use <main class="container"> instead of <div class="container">.
  • Go down orderly when you are using the headings h1 down to h2 down to h3 and so on.
  • You can wrap your attribution section in a footer tag to avoid accessibility issues.

  This should fix most of your accessibility issues

  . Regardless you did amazing... hope you find this useful... Happy coding!!!

  Marked as helpful

  1

  ---

  

  Fizzify•20

  @FizzifyPosted over 2 years ago

  @Samadeen Alright! I'll begin working on my solution and apply these to my other projects. Cheers!

  0