Responsive webpage using Css grid and flexbox

Community feedback

• 

  P

  David Turner•4,150

  @brodiewebdtPosted almost 3 years ago

  This looks very good. Alignment, spacing and hover effects all work well. There are a lot of ways to solve the same problem. I wouldn't worry about how you solved it. You may come up with other ways in the future.

  Wrap your background div in a Main tag and the attribution div in a Footer tag. Change the Equilibrium text to an H1. Every page should have an H1 for accessibility. You will have to re-style it to match the design. Do the same for the H4's. Change them to H2's. You don't want to skip heading levels. These things will clear the accessibility warnings.

  Download AXE DevTools and you can clear accessibility warnings while you code. https://www.deque.com/axe/devtools/

  Hope this helps.

  Marked as helpful

  1

  ---

  

  Emmanuel ifeanyichukwu•60

  @Mire-webPosted almost 3 years ago

  @brodiewebdt thank you, I really appreciate your feedback. I will implement all you have said.

  1