@Arkadiusz-coder
Posted
Hi, congratulations with your first Frontend Mentor porject!
I have few suggestions: 1a. Your .attribution element should have valid links. "Frontend Mentor" is linked by https://www.frontendmentor.io?ref=challenge, and this is general address to the user's profile page. So when I clicked it's send me to my profile. It would be better if the link will send user to your frontendMentor profile. I suggest to replace this link with your https://www.frontendmentor.io/profile/molLbach 1b. there is no link behind your name, so I would suggest set link to your github profile, so people who want to know more about you would be send to your projects.
-
You used h3 element for the title of the page. I guess it's because the font-size of that element where more convenient to you but it is a better practice to use title elements according to their hierarchy, so always start with h1, never skip the number and adjust font size of the element with css. Because this elements are mostly for providing some order to your code.
-
It's also good practice to make some emantic html. So for example your .attribution element could be within <footer></footer> (and put at the bottom of the page); your .image could be within <picture></picture>; your text (h1 and p elements) within <main></main>;
I hope this will be helpful with your next projects :)
Marked as helpful
@molLbach
Posted
@Arkadiusz-coder Thank you for the feedback, I will do as you sugested