Skip to content
OverviewSolutions
12
Comments
152

Dharmik_487

@Dharmik481,740 points

I’m a mysterious individual who has yet to fill out my bio. One thing’s for certain: I love writing front-end code!

Latest solutions

View more solutions

Latest comments

  • Dharmik_487•1,740
    @Dharmik48

    Hey👋,

    Good job with the solution! Looks nice, I just have a couple of things I think you can improve:

    • At around 930px screen width, the form looks very thin, so I think you should increase the width a bit.
    • Also it would be good if you add transition on hover state.

    Keep Developing👍

  • Dharmik_487•1,740
    @Dharmik48

    Hey👋,

    Your solution looks pretty good! I just have a couple of suggestions:

    • Firstly, there is a vertical scroll bar in your solution and I don't think it should be there.
    • And I suggest you to remove the default outline on focus state from buttons and links.

    Apart from these you solution looks nice, keep it up👍

    Marked as helpful
  • Dharmik_487•1,740
    @Dharmik48

    Hey👋,

    Good job on completing the challenge! I just have one suggestion for you:

    • Try to use relative units like em and rem more instead of px as it is a good practice and helps in responsive design as well!

    Keep Developing👍

    Marked as helpful
  • Dharmik_487•1,740
    @Dharmik48

    Hey👋,

    Great job with the solution! Looks great, but.. I just have one suggestion for you:

    • Try to use relative units like em and rem more instead of px.

    Keep Developing👍

    Marked as helpful
  • Dharmik_487•1,740
    @Dharmik48

    Hey👋,

    Your solution is really good! But.. I found a few suggestions:

    • Try to use semantic html tags like main, section, header, etc. more as it is a good practice and good for SEO.
    • Also, I think you should remove the margin-top: 10rem; from 886px media-query as because of it there is a vertical scrollbar.

    Keep it Up👍

    Marked as helpful
  • Dharmik_487•1,740
    @Dharmik48

    Hey👋,

    Your solution looks good, but.. I have a few suggestions:

    • In the features section, when I try to click on the other feature, the details below won't change. So try to add it.
    • Also it would be great if you add some transition to hover state.
    Marked as helpful
View more comments