social links page

Community feedback

• 

  Darshan•210

  @its-subhashPosted 10 months ago

  Hey @kanyijr, your solution looks pretty good, although I have few suggestions to make.

  • Your solution looks like too broad in 1440px which is usual screen size, so to prevent that form happening you can decrease max-width:700px to max-width:345pxof .container{} so it'll look better in both devices.

  • According to requirement links should glow green while hovering but you changed the opacity, so insted of changing opacity you should change background color.

  • Card borders are missing too, so consider adding those.

  -Explore :root so you can store repetating values at a same place, it does looks insignificant on these small project but you have to use it on bigger projects so using it is a good practice.

  Hope you find these Helpful

  Happy Coding...

  Marked as helpful

  0

  ---

  

  Mwenda Kanyi•180

  @kanyijrPosted 10 months ago

  @its-subhash oh yeah, good eye my friend.Thanks

  0