Edyta

Latest comments

• 

  Ramon•10

  @ramonvdkloosterSubmitted over 3 years ago

  

  3 Column Preview Card - Responsive + Hover

  1

  10
  

  Edyta•100

  @Infinit-dotPosted over 3 years ago

  Hi, overall good job, but I see some things you can improve

  • to solve the problem with the buttons you shouldn't use the explicit height of the container, you can use height: auto, min-height: 600px or height: max-content.
  • you have container-flex but you are not using it for anything, fewer containers possible is always better unless you need them for additional styling.
  • try to use HTML semantics like main, footer etc. this will solve accessibility issues
  • you can improve styling to match better the design

  Marked as helpful

  1