Skip to content
OverviewSolutions
36
Comments
25

Dinesh

@Dinesh10421,115 points

Prolific , Web Developer with a passion of creating websites. Love to write frontend code😍

Latest solutions

View more solutions

Latest comments

  • Dinesh•1,115
    @Dinesh1042

    Hi NGUYEN!

    Great work on this solution. Here are my few suggestions.

    • I think you haven't removed margin form the body looks like a white border.
    • Rating number are not accessible by the keyboard.
    • Adding transition on submit button would be better.

    Awesome work Keep going and happy coding.

  • Dinesh•1,115
    @Dinesh1042

    Yes, you can but avoid using pixels most often use rem or em on font size. You can take on this StackOverflow question Should I use px or rem value units in my CSS? .

    Fantastic Work Happy Coding!

    Marked as helpful
  • Dinesh•1,115
    @Dinesh1042

    Hi, ccreusat! Something unique makes people outstand. Fantastic on this challenge.

    Happy Coding!

    Marked as helpful
  • Dinesh•1,115
    @Dinesh1042

    Hi Thiago, Something went wrong with your deployment. It shows our friend 404 error. Try deploying again on the root folder of the project.

  • Dinesh•1,115
    @Dinesh1042

    Hi Ganesh! Great work on this challenge. I think you haven't seen the style readme file on the downloaded folder where they provide font-family and color for the project.

    Great work Happy Coding.

    Marked as helpful
  • Dinesh•1,115
    @Dinesh1042

    Hello Sakssingh, 👋 Amazing work on your first challenge. Here is my small suggestion

    • Just take a look at your report of your project for accessibility issues

    Great work Keep Coding, Happy Coding🤗

    Marked as helpful
View more comments