Skip to content
OverviewSolutions
5
Comments
5
Victor Tyslenko
@VicTyslenko

All comments

  • Victor Tyslenko•90
    @VicTyslenkoPosted about 1 year ago

    Hi! Looks great, but maybe no need this text in the bottom "Challenge by Frontend Mentor. Coded by Vahid."

    0
  • Victor Tyslenko•90
    @VicTyslenkoPosted about 1 year ago

    Hi!) You can add cursor-pointer on Submit button and the numbers as well. Also, as on design, it is supposed to be grey color when its active state and orange just on :hover. And you can delete this text "Challenge by Frontend Mentor. Coded by Navine Rai." on the bottom.

    Marked as helpful
    1
  • Victor Tyslenko•90
    @VicTyslenkoPosted about 1 year ago

    Hi!) The assessment component doesn't work when I try to submit. Also active state doesn't change the color of the numbers. But the size of the component is very good!

    Marked as helpful
    1
  • Victor Tyslenko•90
    @VicTyslenkoPosted about 1 year ago

    Hi, I would suggest to fix paddings and margins, also your component is too big to compare with the design. On the "thank you" page the description text doesn't have a proper width as on design. Good luck!)

    0
  • Victor Tyslenko•90
    @VicTyslenkoPosted about 1 year ago

    It would be better to add proper paddings, margins, background colors( button, list-items) and also the correct linear gradient in the left-side part. Plus use some hsla() for better transparency)

    Marked as helpful
    0