@kunal-v2
Submitted
@jan28x
@kunal-v2
Submitted
@jan28x
Posted
A few improvements that could be made:
"./assets/images/avatar-jessica.jpeg"
.attribution { font-size: 11px; text-align: center; }
.attribution a { color: hsl(228, 45%, 44%); }
in the index.html file to the styles.css file.
Using an unordered list of links instead of buttons would also be a good idea as the actual site would be used to link the user to the creator's various social media profiles.
Setting the background colour as the one specified in the style guide would also make the solution closer to the design.
@mahbt
Submitted
@jan28x
Posted
Your solution is incredibly similar to the design!
Minor changes in the appearance of the site could be:
@dhiaa-zer
Submitted
@jan28x
Posted
Your solution looks really similar to the design and overall looks really well done!
The HTML is easy to read, and includes semantic HTML, like including Alt text for the image.
The layout looks generally good on both wide screens and phone screens.
It could be improved in small areas by:
Marked as helpful