Skip to content
OverviewSolutions
8
Comments
8
P
SaeedAbakah
@SaeedAbakah

All comments

  • SaeedAbakah•110
    @SaeedAbakahPosted 5 days ago

    Very good build and kudos for using a framework. I guess it makes the job easier. Just a couple things to note;

    1. Do you mean to keep those border lines on the page or is it an oversight?
    2. At breakpoints of 1024px - 768px, the hero images didnt seem to be displayed on the website. Kindly look into that
    0
  • SaeedAbakah•110
    @SaeedAbakahPosted 27 days ago

    clean, simple and sustainable code

    0
  • Pickle•220
    @Augurk66Submitted about 1 month ago
    What are you most proud of, and what would you do differently next time?

    This is the first time that I used grid in a project. I am proud of the fact that I accomplished this assignment.

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

    Setting up grid for bigger screen. I checked some videos and some websites to center the left and right blocks.

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

    Working with sizing the page content. The content doesn't fit within a browser page.

    Four card section using grid

    1
    SaeedAbakah•110
    @SaeedAbakahPosted about 1 month ago

    very good build. Cant fault it!

    0
  • SaeedAbakah•110
    @SaeedAbakahPosted about 1 month ago

    very good build. clean code and very good use of clamp() for responsive typo

    0
  • SaeedAbakah•110
    @SaeedAbakahPosted about 1 month ago

    very good build. look at the color for the hero title

    Marked as helpful
    0
  • Naveen K•50
    @naveenkkannanSubmitted about 1 month ago
    What are you most proud of, and what would you do differently next time?

    I was able to nearly perfect the design without a Figma file.

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

    Without a Figma file, it was challenging to accurately measure the margins and padding.

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

    I would appreciate your help in verifying whether the font size and weight are appropriate.

    Social Links Profile

    1
    SaeedAbakah•110
    @SaeedAbakahPosted about 1 month ago

    Good efforts. Check the title of your index.html, i think you need to get rid of the "<h1>" there. Also instead of using individual divs, I think its more useful to use a <ul> to nest all the links and dont forget to add the <a> tag to make them a true link. hope it helps

    0
  • SaeedAbakah•110
    @SaeedAbakahPosted about 2 months ago

    Very good build, properly structured and reusable code.Well done!

    1
  • SaeedAbakah•110
    @SaeedAbakahPosted 2 months ago

    cool build but consider responsiveness

    0