Skip to content
OverviewSolutions
17
Comments
94
Nikola D
@NikolaD93

All comments

  • Nikola D•1,530
    @NikolaD93

    Almost forgot 😁 , I also made a video preview of the project on my YouTube channel, check it out 👇

    Click here to watch the video

  • Nikola D•1,530
    @NikolaD93

    👋 Well done my friend. Looks amazing! 🌟

    Marked as helpful
  • Nikola D•1,530
    @NikolaD93

    Hello there! 👋

    Congrats for completing the challenge! 🙌

    I have a suggestion for you:

    Put all your footer social icons in an unordered list - <ul></ul> tag, then in list - <li></li> tags, which then have <a></a> tags. 👇

    https://www.w3schools.com/html/html_lists_unordered.asp

    Hope this was helpful for you!

    Happy holidays! 🎉✌

    Marked as helpful
  • Nikola D•1,530
    @NikolaD93

    Well done my friend, just keep on practicing and improving your skills. 👌👌

    Try to do some harder challenges, you've mastered newbie 😁😁✌

    Marked as helpful
  • Nikola D•1,530
    @NikolaD93

    Hello my friend! 👋

    Congrats for completing the challenge! Everything looks great and the code is clean, just keep up with the good work! 🙌

    Hope this was helpful for you!

    Happy holidays! 🎉✌

    Marked as helpful
  • Nikola D•1,530
    @NikolaD93

    Hello there! 👋

    Congrats for completing the challenge! Everything looks great and the code is clean, just keep up with the good work! 🙌

    Hope this was helpful for you!

    Happy holidays! 🎉✌

    Marked as helpful
  • Nikola D•1,530
    @NikolaD93

    Hello there! 👋

    Congrats for completing the challenge! 🙌

    I have a suggestion for you:

    1.Make sure to write some description on all of your images in the alt attribute. 👇

    https://www.w3schools.com/tags/att_img_alt.asp

    2.You can put all of your element in one parent container with a <main></main> tag.That represents the use of semantic HTML, which is more descriptive and SEO friendly.

    Hope this was helpful for you!

    Happy holidays! 🎉✌

    Marked as helpful
  • Nikola D•1,530
    @NikolaD93

    Hello there! 👋

    Congrats for completing the challenge! Everything looks great and the code is clean, just keep up with the good work! 🙌

    Hope this was helpful for you!

    Happy holidays! 🎉✌

    Marked as helpful
  • Nikola D•1,530
    @NikolaD93

    Hello there! 👋

    Congrats for completing the challenge! 🙌

    I have a suggestion for you:

    1.You can do some reset CSS styles. By resetting your styles, you avoid defaulting to the browser’s built-in styles, which differs from browser to browser. 👇

    You can reset styles like this, just write the following code in the CSS file:

    • { margin: 0; padding: 0; box-sizing: border-box; }

    Hope this was helpful for you!

    Happy holidays! 🎉✌

    Marked as helpful
  • Nikola D•1,530
    @NikolaD93

    Hello there! 👋

    Congrats for completing the challenge! 🙌

    I have a suggestion for you:

    1.Make sure to write some description on all of your images in the alt attribute. 👇

    https://www.w3schools.com/tags/att_img_alt.asp

    Hope this was helpful for you!

    Happy holidays! 🎉✌

    Marked as helpful
  • Nikola D•1,530
    @NikolaD93

    Hello there! 👋

    Congrats for completing the challenge! Everything looks great and the code is clean. 🙌

    One tiny suggestion, you can put all of your element in one parent container with a <main></main> tag, instead of putting it in a div .design-container. That represents the use of semantic HTML, which is more descriptive and SEO friendly.

    Hope this was helpful for you!

    Happy holidays! 🎉✌

    Marked as helpful
  • Nikola D•1,530
    @NikolaD93

    Hello there! 👋

    Congrats for completing the challenge! 🙌

    Yes your solution is good well done, I have a suggestion 👇

    To center the container use display:flex; justify-content:center; align-items:center.

    Here are some other ways to center a div or text 👇

    Center a div or text

    Happy holidays! 🎉✌

  • Nikola D•1,530
    @NikolaD93

    Hello there! 👋

    Congrats for completing the challenge! 🙌

    I have a suggestion for you:

    1.Make sure to write some description on all of your images in the alt attribute. 👇

    https://www.w3schools.com/tags/att_img_alt.asp

    Hope this was helpful for you!

    Happy holidays! 🎉✌

  • Nikola D•1,530
    @NikolaD93

    Hello there! 👋

    Congrats for completing the challenge! 🙌

    The code looks clean and the design is great, looks like you still got it!

    I would just not leave the product__image div empty however, but I can see you added an image in CSS later, which is not bad also.

    Hope this was helpful for you!

    Happy holidays! 🎉✌

    Marked as helpful
  • Nikola D•1,530
    @NikolaD93

    Hello there! 👋

    Congrats for completing the challenge! 🙌

    The code looks clean and the design is great!

    Hope this was helpful for you!

    Happy holidays! 🎉✌

    Marked as helpful
  • Nikola D•1,530
    @NikolaD93

    Hello there! 👋

    Congrats for completing the challenge! 🙌

    I have a suggestion for you:

    1.Make sure to write some description on all of your images in the alt attribute. 👇

    https://www.w3schools.com/tags/att_img_alt.asp

    Hope this was helpful for you!

    Happy holidays! 🎉✌

    Marked as helpful
  • Nikola D•1,530
    @NikolaD93

    Hello there! 👋

    Congrats for completing the challenge! 🙌

    I have a suggestion for you:

    1.Make sure to write some description on all of your images in the alt attribute. 👇

    https://www.w3schools.com/tags/att_img_alt.asp

    Hope this was helpful for you!

    Happy holidays! 🎉✌

    Marked as helpful
  • Nikola D•1,530
    @NikolaD93

    Hello there! 👋

    Congrats for completing the challenge! Everything looks great and the code is clean. 🙌

    One tiny suggestion, you can put your CSS file in one folder named public, that's the best practice.

    Hope this was helpful for you!

    Happy holidays! 🎉✌