Skip to content
OverviewSolutions
7
Comments
3
P
Omar Ismail
@OmarIsmail7980

All comments

  • Omar Ismail•150
    @OmarIsmail7980Posted over 1 year ago

    Hello Steven, Great solution you nailed it. You really brought the project to life. The only thing I would say is the card should be centered and it doesn't seem like your card is centered. if you're using flexbox you should try "align-items:center" to center it vertically. hope this helps

    0
  • Omar Ismail•150
    @OmarIsmail7980Posted over 1 year ago

    Hello Nicolas,

    great work on the project. there is few things we can improve on.

    • in the strength section: the strength is supposed to dynamically change from "Too weak", "Weak", "Medium", and "Strong". Away to do that is to keep track of how many checkbox are selected and calculate the strength based on that. for instance, if all boxes are checked it's "Strong" or if none are selected it's "Too Weak"

    -Small typo instead of 'To Weak" it's supposed to be 'Too Weak"

    hopefully this is helpful, Good luck

    Marked as helpful
    1
  • Omar Ismail•150
    @OmarIsmail7980Posted over 1 year ago

    Hello Hemang,

    great job on the project. On the New section, u need to add a bit of padding on the sides of the text. Also would you make the title a bit bigger

    0