Skip to content
OverviewSolutions
4
Comments
4

Moirand

@Moirand60 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

Latest comments

  • Moirand•60
    @MoirandPosted about 2 months ago

    It would be great to make it responsive based on different device's screen sizes. Try harder and don't be afraid to fail. keep it up 🔥

    0
  • Moirand•60
    @MoirandPosted about 2 months ago

    Awesome

    0
  • Moirand•60
    @MoirandPosted about 2 months ago

    • I don't think put the avatar image in p::before is a best practice, but that's a pretty creative way.

    • Add these lines, so you don't have to define margin-bottom: 0px; in each element. Read about box-sizing: border-box; in this site. Also, these lines are common used in most of projects, so I recommend you to always add them.

    *,
*::after,
*::before {
    padding: 0px;
    margin: 0px;
    box-sizing: border-box;
}

    Awesome👍

    0
  • Moirand•60
    @MoirandPosted about 2 months ago
    • h1's size shall be 22px
    h1 {
  font-size: 22px;
  // The rest of code
}
    • You can put font-family property in body tag instead of h1 and p
    h1 {
  font-size: 20px;
  font-weight: 700;
  text-align: center;
  color: hsl(218, 44%, 22%);
  font-family: "Outfit", bold; // Remove this line
  padding: 0px 8px 0px;
}

p {
  font-size: 15px;
  font-family: "Outfit", regular; // Remove this line
  color: hsl(216, 15%, 48%);
  text-align: center;
  padding: 0px 8px 12px;
}

body,
html {
  height: 100%;
  margin: 0;
  background-color: hsl(212, 45%, 89%);
  display: grid;
  place-items: center;
  font-family: "Outfit", serif; // Add this line
}
    Marked as helpful
    0