3-column-preview-card-component-main

Community feedback

• 

  Anar•700

  @anar-solPosted about 3 years ago

  Hi =)

  As shown in the accessibility and structure reports, you are using the same id='column' three times. An id attribute is supposed to identify a unique element in the page. I don't think you need id for styling.

  You set the container height to 60vh, 60% of the viewport height and it becomes too high compared to the original design. Don't set a fixed height on the container, let the height be automatically calculated from the contained elements.

  If you want a certain amount of space between the elements in each column, try to use some margins / paddings or any other method to lay them out locally.

  You are also, setting each column width to 42vh, I don't think it's the best way to do.

  Why not just use a flex container, make it vertical on mobile (and perhaps tablet) and when there's enough space, make it horizontal with the same space for each column?

  1