Column Card using flexbox

Community feedback

• 

  Santiago Morales•370

  @Aleroms

  Posted almost 3 years ago

  Hi Dario,

  Congratulation on completing this challenge. Your solution looks great. I have some suggestions regarding your solution if you don’t mind:

  • it looks like you have multiple <h1>. it is recommended not to have more than one h1 on the page. Multiple <h1>tags make using screen readers more difficult, decreasing your site’s accessibility.

  • it seems that you did not include a hover effect on the buttons. You can get the effect by adding

  a:hover { background-color: transparent; color: $transparent-white; border: $transparent-white 2px solid; }

  This will make the <a> as the design has it. I would also add a 2px solid on the <a> This way when the hover on <a> , it doesn't add an additional 2 pixels to the height and width making the elements shift.

  Cheers!

  22

  Marked as helpful