Skip to content
OverviewSolutions
12
Comments
10

Mohammad Ebrahimi

@moheb2000Iran, Mashhad250 points

Med student at MUMS

Latest solutions

View more solutions

Latest comments

  • Mohammad Ebrahimi•250
    @moheb2000

    Good work 🌸🌸🌸

    If you want to center a component with flexbox you can use this:

    .your_div {
  display: flex;
  justify-content: center;
  align-items: center;
}

    and the parent element for your div must have height: 100%;

    Some other suggestions for achieving a better work:

    1. Don't use h3 if you don't have h1 and h2. Use h1 instead and change its styles with css
    2. Use margin and max-width instead of using width and height and also use rem unit instead of using pixels

    I hope my advice help you. Keep going! 🌸🌸🌸

  • Mohammad Ebrahimi•250
    @moheb2000

    Awesome work. I have some suggestions for a better result:

    1. When User click on reset button, custom tip field doesn't clear
    2. When user enter zero in people number field error will disappear
    3. Add a check for computing tips and total price so when user enter an invalid data, "Nan" and "Infinity" don't appear in price labels

    Keep going these awesome works 🌺🌺🌺

    Marked as helpful
  • Mohammad Ebrahimi•250
    @moheb2000

    Awesome! Really good work. Some suggestions for better result:

    1. I think It's better to use equal width and height like 40px for rating numbers to achieve a complete rounded circle.
    2. Using transitions for button and rating numbers is nice too.
    3. Also you can use bold font weight for button text and increase its letter spacing a little bit.

    Good work! Keep going🌸🌸🌸

    Marked as helpful
  • Mohammad Ebrahimi•250
    @moheb2000

    Fascinating! Keep going

    Marked as helpful
  • Mohammad Ebrahimi•250
    @moheb2000

    Hey! That's Awesome. Really good work. For better transition on social media icons you can use to transitions like this:

    a:hover i {
  /* Other codes */
  transition: background-color 0.5s, color 0.5s;
}

    and

    a i {
  transition: background-color 0.5s, color 0.5s;
}

    Good luck!

  • Mohammad Ebrahimi•250
    @moheb2000

    Great job ❤️❤️

    1- title color is a bit lighter than black. You can use 'Dark blue' color for title. its value is in style-guide.md file.

    Good lock 👌👌👌

    Marked as helpful
View more comments