@its-subhash
Posted
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
tomax-width:345px
of.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
@kanyijr
Posted
@its-subhash oh yeah, good eye my friend.Thanks