3 Column preview card component - Vanilla CSS and HTML

Community feedback

• 

  Muhammad Abul Kalam Asif•120

  @Abulkalam-AsifPosted about 3 years ago

  Your site is good but you should make it responsive too.

  Marked as helpful

  1

  ---

  

  Orlando Guevara•220

  @coding-tomatoPosted about 3 years ago

  @Abulkalam-Asif You're right! I just made it responsive for the established viewports widths. I'll make the cards width adjustable, thank you.

  Edit: fixed! thanks again

  1
• 

  P

  Ken•4,915

  @kens-visualsPosted about 3 years ago

  Hey @VeryEvilTomato 👋🏻

  I have a small suggestion for the project. It's really good that you've adapted responsiveness for the website, but you're missing one small, but pretty important thing.

  • You're missing :hover on the button. If you go to the design folder of the project, you'll see an image with a title of active-state, there you'll see how the buttons should look like on :hover.

  I hope this was helpful 👨🏻‍💻 overall, you did a great job for the first project. Cheers 👾

  1

  ---

  

  Orlando Guevara•220

  @coding-tomatoPosted about 3 years ago

  @kens-visuals Thank you! I thought these were for the active state though? You should be able to click on them and see this behaviour. Let me know if you think hover was the correct approach.

  0

  

  P

  Ken•4,915

  @kens-visualsPosted about 3 years ago

  @VeryEvilTomato well, in a real life website yes, if there's a button or a link it should do something other than changing the :hover state, but in this project it's just for practice, and for functionality. But don't worry, you'll have a chance in the future to work with buttons and make them do something 🙃

  Marked as helpful

  1

  

  Orlando Guevara•220

  @coding-tomatoPosted about 3 years ago

  @kens-visuals Hahaha it's fine; I know for certain that hover states are super important, just thought that that was what was required for the project 😂

  0