Responsive pages using grids and flexbox

Community feedback

• 

  John Omoke•240

  @jomoke814Posted about 2 years ago

  Hello, @vishal-singh5128 great work on your project. A suggestion to help you improve your code:

  • instead of using br you can just use flexbox on your cards like so: display: flex; flex-direction: column; justify-content: space-between; and this will space everything inside the cards.
  • The car images/icons in this component are purely decorative; They add no value. So their Alt Tag should be left blank and have an aria-hidden=“true” to hide them from assistive technology.

  Marked as helpful

  1