Skip to content
OverviewSolutions
13
Comments
7

newtothis90

@newtothis90240 points

I’m a mysterious individual who has yet to fill out my bio. One thing’s for certain: I love writing front-end code!

Latest solutions

View more solutions

Latest comments

  • newtothis90•240
    @newtothis90

    I see the stars aren't inline with the review rating.

    Since that is your current markup, you could try making your ".stars" class flex-direction: row;

    instead of

    flex-direction: column;

    .stars{ flex-direction: row; }

    and then you may or may not need to adjust the width of the ".stars" class

    Otherwise, looks awesome man!

    Keep going!

    Marked as helpful
  • newtothis90•240
    @newtothis90

    Your font isn't showing up as it is supposed to.

    I don't believe you can just link to the google fonts URL using font-face.

    I'd look into just going to the google webfonts page and using their <link>

    such as:

    <link rel="preconnect" href="https://fonts.googleapis.com"> <link rel="preconnect" href="https://fonts.gstatic.com" crossorigin> <link href="https://fonts.googleapis.com/css2?family=Outfit&display=swap" rel="stylesheet">

    You would place those tags in the <head> of your html document and then you are free to use the font wherever you would like!

    Good Job and Happy Coding!

    Marked as helpful
  • newtothis90•240
    @newtothis90

    Just a few things I noticed:

    1. Try increasing the padding on your .parent-div div{} Perhaps try starting at 30px and go from there.

    2. The button width is way too large and the height isn't tall enough.

    3. The button margin from the paragraph needs to be longer.

    Sidenote: If you are familiar with Grid, you could easily space all of your elements out vertically.

    It looks really good other than those few items! Great Job and Keep Going!

  • newtothis90•240
    @newtothis90

    Everything looks great! The one thing I immediately noticed is that the "line-height" isn't correct on the <h1> and on the product description.

    Otherwise it looks awesome!

    Happy Coding!

    Marked as helpful
  • newtothis90•240
    @newtothis90

    IN THIS INSTANCE, there's no need to give your <img> a class name at all since you are already using img{}. Giving it a class isn't wrong at all, but a little less code with not giving them a class.

    Hope this helps somewhat. If you have a reason for giving them a class of .image, please explain. I would love to hear your opinion.

    Regardless, GREAT JOB!

  • newtothis90•240
    @newtothis90

    Great work! Looking at your CSS actually helped me learned a few things!

View more comments