Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

All comments

  • Faisal Alharbi• 330

    @LovelyFaisal

    Posted

    Hi lordvitor11! You've done great 💪

    Here are some suggestions to improve your code:

    • Use letter-spacing property to add spacing between letters

    The letter-spacing property increases or decreases the space between characters in a text.

    Click here for more information

    Good job, and happy coding!

    Marked as helpful

    0
  • Faisal Alharbi• 330

    @LovelyFaisal

    Posted

    Hi @pedroddjkkk ! You've done great 💪

    • Use <main> instead of a simple <div> this way you improve the semantics and accessibility showing which is the main block of content on this page. Remember that every page should have a <main> block and that <div> doesn't have any semantic meaning.

    Click here for more information

    Good job, and happy coding!

    1
  • Faisal Alharbi• 330

    @LovelyFaisal

    Posted

    Hi @jamilpl! You've done great 💪

    Here are some suggestions to improve your code:

    • Use <main> instead of a simple <div> this way you improve the semantics and accessibility showing which is the main block of content on this page. Remember that every page should have a <main> block and that <div> doesn't have any semantic meaning.

    Click here for more information

    Good job, and happy coding!

    Marked as helpful

    0
  • Faisal Alharbi• 330

    @LovelyFaisal

    Posted

    Hi @machadogustavo You've done great 💪

    Here are some suggestions to improve your code:

    • To center the component using flexbox add this property min-height: 100vh; to body element

    Click here for more information

    Good job, and happy coding!

    Marked as helpful

    1
  • Faisal Alharbi• 330

    @LovelyFaisal

    Posted

    Hi @guidoghg ! You've done great 💪

    Here are some suggestions to improve your code:

    • You need to add the alt attribute to your image like this <img src="images/image-product-desktop.jpg" class="desktopProduct" alt="qr code to frontendmentor">

    The required alt attribute provides an alternate text for an image, if the user for some reason cannot view it (because of slow connection, an error in the src attribute, or if the user uses a screen reader).

    You can learn more here

    • Use <main> instead of a simple <div> this way you improve the semantics and accessibility showing which is the main block of content on this page. Remember that every page should have a <main> block and that <div> doesn't have any semantic meaning.

    Click here for more information

    Good job, and happy coding!

    1
  • Faisal Alharbi• 330

    @LovelyFaisal

    Posted

    Hi @keiffon You've done great 💪

    Here are some suggestions to improve your code:

    • You can center the component using flexbox Add this property to body element
    main {
    min-height: 100vh;
    display: flex;
    align-items: center;
    justify-content: center;
}

    You can learn more about flexbox here

    • Use <main> instead of a simple <div> this way you improve the semantics and accessibility showing which is the main block of content on this page. Remember that every page should have a <main> block and that <div> doesn't have any semantic meaning.

    Click here for more information

    Good job, and happy coding!

    Marked as helpful

    0
  • Faisal Alharbi• 330

    @LovelyFaisal

    Posted

    Hi @Jackgomez You've done great 💪

    • You can center the component using flexbox Add this property to body element
    main {
    min-height: 100vh;
    display: flex;
    align-items: center;
    justify-content: center;
}

    And delete margin: 15rem auto; from .box

    You can learn more about flexbox here

    • Use <main> instead of a simple <div> this way you improve the semantics and accessibility showing which is the main block of content on this page. Remember that every page should have a <main> block and that <div> doesn't have any semantic meaning.

    Click here for more information

    Good job, and happy coding!

    Marked as helpful

    1
  • Faisal Alharbi• 330

    @LovelyFaisal

    Posted

    Hi! You've done great 💪

    • Use <main> instead of a simple <div> this way you improve the semantics and accessibility showing which is the main block of content on this page. Remember that every page should have a <main> block and that <div> doesn't have any semantic meaning.

    Click here for more information

    Good job, and happy coding!

    Marked as helpful

    0
  • Faisal Alharbi• 330

    @LovelyFaisal

    Posted

    Hi! You've done great 💪

    Here are some suggestions to improve your code:

    • You need to add the alt attribute to your image like this <img src="image-qr-code.png" alt="qr code to frontendmentor">

    The required alt attribute provides an alternate text for an image, if the user for some reason cannot view it (because of slow connection, an error in the src attribute, or if the user uses a screen reader).

    You can learn more here

    • Use <main> instead of a simple <div> this way you improve the semantics and accessibility showing which is the main block of content on this page. Remember that every page should have a <main> block and that <div> doesn't have any semantic meaning.

    Click here for information

    Good job, and happy coding!

    Marked as helpful

    0
  • Faisal Alharbi• 330

    @LovelyFaisal

    Posted

    Hi! You've done great 💪

    Here are some suggestions to improve your code:

    • Use <main> instead of a simple <div> this way you improve the semantics and accessibility showing which is the main block of content on this page. Remember that every page should have a <main> block and that <div> doesn't have any semantic meaning.
    • Replace the <h2> containing the main title with <h1> note that this title is the main heading for this page and every page needs one h1 to show which is the most important heading. Use the sequence h1 h2 h3 h4 h5 to show the hierarchy of your titles in the level of importance, never jump a level.

    Click here for more information

    Marked as helpful

    1
  • Faisal Alharbi• 330

    @LovelyFaisal

    Posted

    Hi! You've done great 💪

    Here are some suggestions to improve your code:

    • The component isn't centered correctly on larger screens to solve it you need to change max-width: 1440px; from body to max-width: 100%;

    • Use <main> instead of a simple <div> this way you improve the semantics and accessibility showing which is the main block of content on this page. Remember that every page should have a <main> block and that <div> doesn't have any semantic meaning.

    Click here for more information

    Good job, and happy coding!

    2
  • Faisal Alharbi• 330

    @LovelyFaisal

    Posted

    Hi! You've done great 💪

    Here are some suggestions to improve your code:

    Use <main> instead of a simple <div> this way you improve the semantics and accessibility showing which is the main block of content on this page. Remember that every page should have a <main> block and that <div> doesn't have any semantic meaning.

    Click here for more information

    0