Skip to content
OverviewSolutions
5
Comments
2
Ayush Gandhi
@ayush577

All comments

  • Ashwin Gairola•30
    @ashwingairolaSubmitted about 4 years ago

    If there are any Angular devs out there, I would appreciate some feedback on the code quality.

    Other than that, I noticed that the heading in the hero section could have been better.

    Angular 10 project using SCSS.

    1
    Ayush Gandhi•220
    @ayush577Posted about 4 years ago

    Hey Ashwin, Great work on this! Nice to see people still using Angular 😄. The page looks great and functions well. The only thing I'd mention is to make sure you're using a container to containing website width for large screen and maybe font size not the same as a reference design.

    1
  • Ayush Gandhi•220
    @ayush577Posted over 4 years ago

    I believe you did amazing work, but there are a few things. I think you have to consider

    1. Theme switcher toggle should have a gradient background all the time
    2. Overview cards should have more height and equal padding from all the sides
    0