@AgataLiberska
Posted
Hi @chinmayagarwal3007! Your solution looks pretty good, although I think you should have a look at how it looks on tablet - on smaller devices the social media icons overlap the cta button.
I also think those icons should be links with aria-labels and some extra spacing, because when you scale them larger on hover, the whole container moves around.
I would also make sure that the background image doesn't repeat using background-repeat
and background-size
Hope this helps :)