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

  • Rohit Boseโ€ข 30

    @ROHIT-130SE

    Submitted

    It's nice to practice and learn more, I hope to work on more challenges at the earliest possible and please feel free to review and share your feedback on this work.

    Karthik SPโ€ข 110

    @karthisp

    Posted

    Great work Rohit.

    Just a few suggestions

    • Give <html>, <body>, #container a height of 100% and align #container to the 'center' using align-items CSS property for flex.

    Marked as helpful

    0
  • Kyleโ€ข 50

    @kylecloete

    Submitted

    This has been one of my favourite challenges to complete so far, making use of JavaScript and actually see you code do something was so interesting for me

    When researching solutions for this challenge as well as looking at a couple of YouTube videos. I was left confused no offense to those people but what they were doing just seemed so convoluted for a beginner project.

    I recommend looking at freeCodeCamps video "JavaScript Programming - Full Course"

    Any Advice would be greatly appreciated especially regarding the structure of my html and appropriate tags to use for which section and how to clean up some of my JavaScript

    Karthik SPโ€ข 110

    @karthisp

    Posted

    Great Job.

    Looks very close to the original design.

    Just a few suggestions

    1. increase the 'letter-spacing' for button text
    2. Increase line height for <p>
    3. 1 rem margin for '.rating_component' and <h1> should be good enough
    0
  • Karthik SPโ€ข 110

    @karthisp

    Posted

    Hi @MelvinAguilar you've done a great job with implementing the given design. Keep up the good work. I've got a lot of things to learn from you.

    Just a small suggestion:

    I see that you've used clip for sr-only (H1), instead try using clip-path. Apparently, that's what is recommended. https://developer.mozilla.org/en-US/docs/Web/CSS/clip-path

    1