Responsive News Homepage using CSS Flex

Community feedback

• 

  Sanjeewa Marukona•200

  @eewa-SANJPosted almost 2 years ago

  Hello!. Your solution looks good but I have gone through your code. There are a few issues found in your codes. Here are my findings.

  1. Your site is not responsive, so it crashes and overlap here and there. You can fix those issues using media queries. If you want to help me more, contact me and I will help you

  2. Your favicon is not rendered due to the wrong path. Could you fix it?

  3. Your main image(image web 3 desktop) only rendered in small screens. But on small screens, your mobile version of that image should be rendered. <img srcset="assets/images/image-web-3-mobile.jpg 686w, assets/images/image-web-3-desktop.jpg 1460w" sizes="(max-width: 375px) 686px 1460px" src="assets/images/image-web-3-desktop.jpg" alt="Web 3.0" > . If you want how to implement this Read this

  4. There is an alignment issue in your sidebar and below three column divs. You can adjust it by resizing the sidebar according to the three-column width.

  5. You have missed the mobile hamburger menu in the tab and mobile screen sizes

  I think, my findings will help you to improve your coding.

  Happy coding😊

  Marked as helpful

  1

  ---

  

  Antwi Isaac•115

  @okraksPosted almost 2 years ago

  @eewa-SANJ thanks for pointing these out! I'll be fixing these issues

  0