3-column-preview-card-component-main

Community feedback

• 

  

  Grace•32,130

  @grace-snow

  Posted about 4 years ago

  Hi

  Looks nice, just a few issues with the html

  • remove empty ids
  • you can only have one h1 per page
  • those buttons should be anchor tags

  If you place transparent border on the buttons nothing will move when you click them and the white border appears. A small thing but nice to stop that little jump

  Hope these help. Nice work!

  13
• 

  Rayane•1,935

  @RayaneBengaoui

  Posted about 4 years ago

  Hello Chevalier Antoine,

  Congrats for completing the challenge ! 😁

  I'd like to suggest :

  • When hovering your buttons, the content is pushed away and the card extends. It's because you add a border to the button when it originally doesn't have one. The border is part of the 'Box-model', so to avoid this behavior you could add border: solid 2px hsla(0, 0%, 100%, 0.75) to your buttons.

  • Here in the design the text is spaced, you could do this by using the property line-height.

  • Also, you could add the property transition to your buttons to make the transition smoother and cursor: pointer to make them look clickable.

  Overall, well done for the challenge and happy coding ! 😃

  11