Skip to content
OverviewSolutions
5
Comments
26
SHAHAB MALIK
@Shahab-Malikk

All comments

  • SHAHAB MALIK•345
    @Shahab-MalikkPosted about 3 years ago

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

    Marked as helpful
    1
  • Nayana Bhat M•90
    @nayanabhatmSubmitted about 3 years ago

    Hi there!,

    1. If I don’t specify the height in px, top: X% with relative position doesn’t work. Is there any way I can specify height in % instead of hardcoding the value?
    2. I have specified overflow: hidden but can still see the scrollbars when I resize the window.

    Profile Card using Flexbox

    3
    SHAHAB MALIK•345
    @Shahab-MalikkPosted about 3 years ago

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

    Keep Coding

    0
  • Juan Zapien•95
    @1ZapienSubmitted about 3 years ago

    Any advice on how to approach the email error handling would be appreciated. Any other suggestions on what to improve would be greatly appreciated.

    Responsive landing page using flex

    2
    SHAHAB MALIK•345
    @Shahab-MalikkPosted about 3 years ago

    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
    1
  • SHAHAB MALIK•345
    @Shahab-MalikkPosted about 3 years ago

    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.
    1
  • SHAHAB MALIK•345
    @Shahab-MalikkPosted about 3 years ago

    Hi nice work

    1. Use overflow-x: hidden property to remove horizental scrollbar.
    Marked as helpful
    0
  • SHAHAB MALIK•345
    @Shahab-MalikkPosted about 3 years ago

    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
    0
  • SHAHAB MALIK•345
    @Shahab-MalikkPosted about 3 years ago

    Hi nice work but there are some problems

    1. Firstly you have to check for responsiveness check after 580px(i am talking about mobile devices)
    2. Add transition effect on hover. Keep Coding
    0
  • SHAHAB MALIK•345
    @Shahab-MalikkPosted about 3 years ago

    Hi nice work implemented things well Keep Coding And yes add transition property to hover

    0
  • SHAHAB MALIK•345
    @Shahab-MalikkPosted about 3 years ago

    Hi Nice work implemented every thing well. Keep Coding

    0
  • Divyank Dubey•60
    @dubeydivyankSubmitted about 3 years ago

    Hello Everyone! This is the first time I wrote html and css. It took a while to finish it. Please do let me know how it can be improved. What are some obvious things that I missed which could have helped improve the final outcome?? Eagerly waiting for responses! :) Thank You!

    Order-smry-card using flexbox media querry

    2
    SHAHAB MALIK•345
    @Shahab-MalikkPosted about 3 years ago

    Hi nice work implemented things well Keep Coding

    0
  • SHAHAB MALIK•345
    @Shahab-MalikkPosted about 3 years ago

    Hi nice work there is only one problem and that is of main heading

    1. You can use div for main heading and can align it in center
    Marked as helpful
    1
  • SHAHAB MALIK•345
    @Shahab-MalikkPosted about 3 years ago

    Hi nice work

    1. First of all remove accessibility issues
    2. Make anchor tag effective you can use hover effect 3.There are also responsiveness issues this webpage is only responsive for mobile screens , so try to make it responsive
    3. Also work on social icons in footer Keep Coding
    1