Skip to content
OverviewSolutions
10
Comments
10
P
Suraj Khatri
@SurajCasey

All comments

  • Suraj Khatri•200
    @SurajCaseyPosted 2 days ago

    Your solution looks so good. Only thing is you could have set the width and height properties to match exactly with the design. Codes are all readable and the layout looks good on different screen sized. You have done a great job with the codes and it looks amazing. Keep it up!

    0
  • Suraj Khatri•200
    @SurajCaseyPosted 13 days ago

    Your solution looks so good. Especially, in the mobile screen it matches perfectly. The codes are well-structrued and readable. The only thing that you could do is use flexbox to make the content be in the middle of the screen, you could do the following: body { background-color: #ecf2f8; height: 100vh; display: flex; justify-content: center; align-items: center; } The whole container will appear in the middle of the screeen and will look perfect.

    0
  • Loic Cape•290
    @loiccapeSubmitted 20 days ago
    What are you most proud of, and what would you do differently next time?

    I realy improve on the way i use grid and flex box !! I use less usless than i used before.

    Meet-landing-page

    1
    Suraj Khatri•200
    @SurajCaseyPosted 20 days ago

    Your solution looks good. It also has a lot of room for improvement. The codes looks redeable. It is responsive for the most screen sizes. There is seperate hero image and footer image for the mobile screen which you could have used. You could make vertical line in page number by using this html and css.

    <div class="vertical-line"></div> .vertical-line{ width: 1px; height: 80px; background-color: hsl(240, 18%, 85%); } Overall, looks good. Goodluck!
    Marked as helpful
    0
  • Suraj Khatri•200
    @SurajCaseyPosted 27 days ago

    Your solution looks amazing. Codes are well structured and readable. Also, the solution matches the design, But, there are few improvements that could be done. They are: You could include <main> </main> to make it more accessible after the body tag. You could add the backgroud color:hsl(210, 46%, 95%); to make it more similar to design.

    Marked as helpful
    1
  • Suraj Khatri•200
    @SurajCaseyPosted about 1 month ago

    The solution matches well with the design. It is accessible and the page is responsive to the mobile and desktop screen but for the the contents overflows for the screen sizes in between mobile and desktop. You have done well with the box shadows. The codes looks readable. As the font family is same for whole page you could use it in the body section so that you do not have to use it again.

    0
  • elijahraphael5•80
    @elijahraphael5Submitted about 1 month ago
    What are you most proud of, and what would you do differently next time?

    that i didnt skip i took the challenge and i was able to finish it up

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

    button and object fit

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

    nothing it challenging enough

    card component

    1
    Suraj Khatri•200
    @SurajCaseyPosted about 1 month ago

    Well done, But the web page is not responsive to small screen sizes. Try using grid method instead of flexbox. There is seperate images provided for the mobile and desktop screens which you can add to use it accordingly.The codes are readable. Keep it up!

    Marked as helpful
    0
  • Suraj Khatri•200
    @SurajCaseyPosted about 1 month ago

    good job, just needs margin for the container.

    0
  • Suraj Khatri•200
    @SurajCaseyPosted about 2 months ago

    Its good.

    0
  • Suraj Khatri•200
    @SurajCaseyPosted about 2 months ago

    It is a good design.

    0
  • Suraj Khatri•200
    @SurajCaseyPosted 5 months ago

    CSS is bit complicated.

    Marked as helpful
    0