Sarah• 540
@AutumnsCode
Posted
Hello,
well done for completing this challenge.
I am happy to provide to give you some feedback:
- the
h1element should be the first heading in your page. I would suggest to use a paragraph element with a class instead to style the element. - the next paragrapgh should have a class since its the text below your h1-element is actually the main text of the challenge.
- I really enjoy that you include a hover effect on your h1-element. To made it even better, it would be great if you could wrap inside a anchor-element.
- The profile image is too large. Inside your css, you could add a width to reduce it.
- Almost every html-element has default styling. It can be really difficult to deal with them while styling like img. If I see an image in one of my project, I am always add reset to take away the default one. Here is the one for
image, picture
img, picture {
max-width: 100%;
display: block
}
I hope those tips and tricks will be helpful Best regards Sarah
Marked as helpful
0
