John Mirage• 1,590
@john-mirage
Posted
Hello, nice work !!
Here some ways you can improve your project:
- You can remove the underlines of your links by adding the
text-decoration: none;rule. - Your card has a fixed width, you can make it responsive by adding a relative width of 100% and a max-width of 350px so the card will shrink when the screen width in under 350px
- There is no paddings around the card, you can add some horizontal paddings to make the card breathe on mobile device
- To help you start a project, you can add a css reset file. It defines some rules to ease your workflow.
- The card has lower paddings on mobile than on desktop, you can add a media querie to reflect this behavior.
- The social links can be placed in <li> elements surrounded by an <ul> element.
- You can add meta tags in your <head> element to display informations and an image for the social medias like facebook and twitter.
- You can define the inter font on the <body> element so the font spread accross all the children elements.
- The hover effect should only be applied to the devices who have hover feature, you can add a media querie to target those devices with
@media screen and (hover: hover) {}
0
