Skip to content
OverviewSolutions
8
Comments
3
Majd Allaham
@Vat123

All comments

  • Majd Allahamβ€’120
    @Vat123Posted over 1 year ago

    Hi, my congratulations you did a great job πŸŽ‰

    π’π¨π¦πž 𝐭𝐒𝐩𝐬

    You need to put a landmark in your code simply wrap your code with <main class='container'> landmark instead of <div class="container" The <main> HTML element is intended for the main content (content) of the <body> of the document (page). I hope it was helpful, you are great, keep up the good work

    and about the img tag most of the time it should have alt attribute so you can describe what the image is all about if it did not show on the screen like this <img class="img-imagen" src="image-qr-code.png" alt="QR-Code image">

    and you have to put the class name after the tag directly not after all the attributes

    hope it is helpful for you keep coding , good work

    Marked as helpful
    1
  • Majd Allahamβ€’120
    @Vat123Posted over 1 year ago

    great job you did

    some tips

    after finishing from coding you have to read your code again and delete the lines that are not that important because if anyone wants to read your code it will be easy as much as possible for him/her

    for the size of the of the div and for the photo you can check the properties of the image and see the width , height of the image so you can know what width to use and which height to select also

    hope it was helpful for you , happy coding

    0
  • Majd Allahamβ€’120
    @Vat123Posted over 1 year ago

    Hi, my congratulations you did a great job πŸŽ‰

    π’π¨π¦πž 𝐭𝐒𝐩𝐬

    You need to put a landmark in your code simply wrap your code with <main class='container'> landmark instead of <div class="container" The <main> HTML element is intended for the main content (content) of the <body> of the document (page). I hope it was helpful, you are great, keep up the good work

    Marked as helpful
    1