Skip to content
OverviewSolutions
10
Comments
6

Sudharsan Jain

@Sudarsh1010Chengalpattu, TamilNadu, India200 points

Working my way to Front-End developer

I’m currently learning...

JavaScript & React

Latest solutions

View more solutions

Latest comments

  • Sudharsan Jain•200
    @Sudarsh1010Posted about 3 years ago

    I saw your code. In mobile version instead of using -grid-template-columns: auto; -grid-template-areas: "one" "two" "three" "four" "five";

    try this: -grid-template-rows: 1fr 1fr 1fr 1fr 1fr;

    Marked as helpful
    0
  • Sudharsan Jain•200
    @Sudarsh1010Posted about 3 years ago

    It's really good. reduce border-radius in annual plan box. also change color of the change link to match design file. that's all I got to say. You did great job <3

    0
  • Sudharsan Jain•200
    @Sudarsh1010Posted about 3 years ago

    Hey, Everything looks fine but there are some things that needs to be fixed

    1. Font Size
    2. Line Heights in Paragraph
    3. Adjust Images so You can lower your whole card size to match with the Design
    4. Last thing is add Border radius on whole card
    1
  • Sudharsan Jain•200
    @Sudarsh1010Posted about 3 years ago

    Everything looks fine. but checkout the font size and color. <3

    0
  • Sudharsan Jain•200
    @Sudarsh1010Posted about 3 years ago

    1. change the font-weight of SUBMIT button and 'How did we do' title to 700 font-weight: 700;

    2. On the screen above 1440px your component is not centered.

    3. Remove Border from SUBMIT button.

    4. There should be different on-hover styling and on-Click styling for the 1-5 rating button and remove the border.

    That's all and keep up the great work bud!

    Marked as helpful
    0
  • Sudharsan Jain•200
    @Sudarsh1010Posted about 3 years ago

    I think you forgot to add Hover Effect. Check active-states img in the design folder. and It's good

    Marked as helpful
    0