Skip to content
OverviewSolutions
7
Comments
6
P

Harshitkumar Patel

@harshitkumarpatel28Southall, United Kingdom120 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

  • P
    Harshitkumar Patel•120
    @harshitkumarpatel28Posted 1 day ago

    Congrats on your design layout!

    Looks good. I have some suggestion like use of Semantic HTML for the structure would be a good use. Mobile responsive layout can be improved. Spacing and line height can be improved.

    0
  • James Ojajuni•80
    @James-OjajuniSubmitted 8 days ago
    What are you most proud of, and what would you do differently next time?

    I made the website look good across different screen widths

    What challenges did you encounter, and how did you overcome them?

    I encountered the problem of not making the content in the second div overflow

    What specific areas of your project would you like help with?

    mehhh

    Product Preview Card with HTML5 and CSS

    1
    P
    Harshitkumar Patel•120
    @harshitkumarpatel28Posted 8 days ago
    • Use of Semantic HTML will be a good help
    • Spacing of contents on detail part is little bit needed
    Marked as helpful
    0
  • P
    Harshitkumar Patel•120
    @harshitkumarpatel28Posted 13 days ago
    • Use Semantic HTML can be done properly
    • Responsive layout needs to be checked in all devices
    Marked as helpful
    0
  • P
    aokoye•100
    @aokoyeSubmitted 14 days ago
    What are you most proud of, and what would you do differently next time?

    I'm happy with how comfortable I felt doing this. This is also the first time I used Figma Pro which was interesting.

    Social links profile

    1
    P
    Harshitkumar Patel•120
    @harshitkumarpatel28Posted 14 days ago
    • Layout is about similar with minor changes
    • Border-radius can be used to shape borders
    • Use Semantic elements will be a good structure
    • The layout is not mobile responsive
    0
  • P
    Harshitkumar Patel•120
    @harshitkumarpatel28Posted 14 days ago
    • Use Semantic HTML
    • Layout can be adjusted to the design. Have a look at flexbox which will help
    • Transition property will help for interactive effect
    • Font styling can be used from style-guide.md
    1
  • P
    Harshitkumar Patel•120
    @harshitkumarpatel28Posted 15 days ago
    • Semantic HTML was not used.
    • The fonts and size of the box could improve.
    • It is not responsive for mobile devices.
    • Code is structured well but can have improvements.
    • The design has 70% match with solution.
    0