Skip to content
OverviewSolutions
5
Comments
25

SHAHAB MALIK

@Shahab-MalikkKhushab345 points

Software Engineering Student

Latest solutions

Latest comments

  • SHAHAB MALIK•345
    @Shahab-Malikk

    Hi , Great work implemented things well I just want to add something "Use cursor:pointer property " for links and buttons.

    Marked as helpful
  • SHAHAB MALIK•345
    @Shahab-Malikk

    Hi @Nayana great work implemented things well there is no issue in website

    Keep Coding

  • SHAHAB MALIK•345
    @Shahab-Malikk

    Hi nice work but there are a few problems

    1. There should be some margin between links in header as they are designed for mobile devices so they are not looking good at laptop screens.

    2. In testimonial section remove the overflow:scroll .

    Marked as helpful
  • SHAHAB MALIK•345
    @Shahab-Malikk

    Hi Brother Nice Work implemented things well But there are few problems

    1. Remove accessibility issues as well Html issues
    2. Your web is responsive great work
    3. Add transition effect on mobile nav as it appears just after clicking button
    4. Also add hover effect on mobile nav Hrefs.
  • SHAHAB MALIK•345
    @Shahab-Malikk

    Hi nice work

    1. Use overflow-x: hidden property to remove horizental scrollbar.
    Marked as helpful
  • SHAHAB MALIK•345
    @Shahab-Malikk

    Hi nice work There are few problems

    1. First of all use property overflow-x:hidden to avoid from horizentally scrolling
    2. Padding for your header links is also too much decrease it so that it may look pretier
    3. Add transition effect on hover
View more comments