Responsive solution HTML, CSS and Flexbox

Community feedback

• 

  バレンタイン 😈•64,190

  @VCaramesPosted almost 2 years ago

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

  • The “illustration” in this component is purely decorative. ⚠️ Its alt tag should be left blank to hide them from assistive technology.

  More Info:📚

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

  • The “Cancel Order” was created with the incorrect element ❌. When the user clicks on the button they should directed to a different part of you site. The anchor tag will achieve this.

  More Info:📚

  MDN The Anchor element

  • NEVER ❌ do this as it creates accessibility issues for users and it is outdated, instead download a px to rem/ em converter in your code editor.

  html {
    font-size: 62.5%;
  }
  

  • To properly center ✅ your content to your page, you will want to add the following to your body (this method uses CSS Grid):

  body {
      min-height: 100vh;
      display: grid;
      place-content: center;
  }
  

  More Info: 📚

  Centering in CSS

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

  Happy Coding! 🎆🎊🪅

  Marked as helpful

  0

  ---

  

  Valentina López•190

  @valentinalopezh98Posted almost 2 years ago

  @vcarames Thanks! Your feedback is very helpful. I will be checking the code

  1

  

  バレンタイン 😈•64,190

  @VCaramesPosted almost 2 years ago

  @valentinalopezh98

  Glad I could help! 😁

  0