3-column preview card component using HTML & CSS Flexbox

Community feedback

• 

  P

  Grzegorz Sterczewski•680

  @gsterczewskiPosted over 3 years ago

  Hello Moises 👋, my name is Grzegorz.

  You did really good job, and implemented the design almost perfectly.

  One minor thing to improve is to add focus state to the anchors, so user can navigate it with keyboard.

  Of course those links are dead, but I think it is a good practice to remember about a11y (I'm working on it myself).

  Overall, excellent work, I'm hoping to see more of your work here.

  Keep up the good work! 💪

  See you on the coding trail 😉

  Cheers!

  Marked as helpful

  1

  ---

  

  Moises•60

  @devmoisesPosted over 3 years ago

  @GSterczewski thank you very much for the recommendation on the focus, I did not know it but now I will apply it.

  0
• 

  ToNisamBioJa•330

  @ToNisamBioJaPosted over 3 years ago

  Hello!

  Really good job on this one. You got it really close to the original design and added a nice hover animation.

  The only thing you could change is the border radius but thats about it.

  Keep it up! Happy codding!

  Marked as helpful

  0

  ---

  

  Moises•60

  @devmoisesPosted over 3 years ago

  @ToNisamBioJa thank you very much for your comment, I will take into account your recommendation about border radius. ✌️

  0

  

  Moises•60

  @devmoisesPosted over 3 years ago

  Hi @ToNisamBioJa I forgot to ask if the change was for the border-radius of the button or the card? Greetings.

  0
• 

  David•175

  @Dups11Posted over 3 years ago

  Hi, just a little improvement :

  For better results with border radius you should :

  1. Border-radius: 10px on section "cards" not on the "card_items"

  2. overflow-hidden on section "cards"

  Apply this rules after remove all other border-radius.

  Marked as helpful

  0

  ---

  

  Moises•60

  @devmoisesPosted over 3 years ago

  @Dups11 thank you for your comment, I will take your recommendations into account.

  0

  

  Moises•60

  @devmoisesPosted over 3 years ago

  Hello @Dups11, apply your recommendations and they look good, although I have not uploaded it to the repository yet for some doubts. In the prototype the middle card has the complete radius-edges and in the last one they only have the right top and the right bottom (this in the pc version). Will it be part of the challenge to consider those options or should we apply it as part of an upgrade?

  0
• 

  P

  Patrick•14,265

  @palgrammingPosted over 3 years ago

  Really slick hover animation. Great Job 🌟🌟🌟🌟🌟

  Marked as helpful

  0

  ---

  

  Moises•60

  @devmoisesPosted over 3 years ago

  @palgramming Thank you very much, I am learning about animations with CSS from scratch, it is great I will be applying them in the following challenges. 👍

  0