SamO• 110
@SamOwens
Posted
Looks pretty good. Just a few very minor things
- If possible you should use rem values for sizes as opposed to px.
- You don't really need the surround div, so it would be considered DOM bloat.
- You'd be better off giving the images their own class rather than using their wrapper to target them but again, very very minor.
- The padding over the SVG looks a bit off
1
Shinzenko• 60
@shinzenko
Posted
@SamOwens Hey thanx man I agree with all the points the last one though in my Pc it showed that the padding on top was little more than intended so i had to reduce it at top but once I deployed it i realised i shouldnt have reduced it, anyway noted about the other issues.
0
