Responsive page using HTML5, SASS and BEM.

Community feedback

• 

  バレンタイン 😈•64,170

  @VCaramesPosted almost 2 years ago

  Hey there! 👋 Here are some suggestions to help improve your code:

  • The NFT alt tag description needs to be improved upon. This is what your users are purchasing. You want to describe what the image is; Assume you’re describing the image to someone over the phone.

  More Info:📚

  https://www.w3.org/WAI/tutorials/images/

  • Wrap the "NFT image" and "Equilibrium #3429" in an anchor tags. The anchor tag will allow users to click on content and have them directed to another part of your site.

  More Info:📚

  MDN The Anchor element

  • Your CSS Reset is extremely bare and being underutilized. To fully maximize your CSS reset, you want to add more to it.

  Here are few CSS Resets that you can look at and use to create your own or just copy and paste one that is already prebuilt.

  https://www.joshwcomeau.com/css/custom-css-reset/

  https://meyerweb.com/eric/tools/css/reset/

  http://html5doctor.com/html-5-reset-stylesheet/

  If you have any questions or need further clarification, feel free to reach out to me.

  Happy Coding!🎄🎁

  1

  ---

  

  Elio Flores•400

  @eliofloPosted almost 2 years ago

  @vcarames Thanks for your feedback, helps me a lot!

  I updated my repo adding an anchor tag, alt property, and normalize file (scss file). From now on I will use normalize in the frontendmentor challenges.

  Merry Christmas!🎄

  Elio Flores

  0