Skip to content
Submitted over 2 years ago

Hard coded the css but made it

SuperSet1607ā€¢310
@mohit1607
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Base Apparel coming soon page
  • HTML
  • CSS
  • JS
1newbie

Design comparison

SolutionDesign

Solution retrospective

Hello, Awesome community I tried my best to make it look alike but I have some issues with sizing. How can I improve giving sizes instead of hardcoding? and also the Javascript part I am unable to change the Css directly by element.style.display = 'none'. Any other options ? thanks and please drop a code review.

Community feedback

  • Shashree Samuelā€¢9,260
    @shashreesamuelPosted over 2 years ago

    Hey good job completing this challenge

    Keep up the good work

    Your solution looks great however I think that the font weight of the title is supposed to be lighter, secondly the description is supposed to have word wrap and lastly the input field is a bit too long

    In terms of accessibility issues simply wrap all the content between main tags

    I hope this helps

    Cheers Happy coding šŸ‘

    Marked as helpful
    1

Please log in to post a comment

Log in with GitHub

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord