Jak no moje oko i gust to jest bardzo dobrze. Jeśli coś umknęło to przepraszam.
Kilka drobnych uwag.. żeby było jasne nie czepiam się! moją intencją jest pomóc ...
Jeśli chodzi i skrypt JavaScript, są dwie szkoły : ..pod koniec body lub w sekcji head ale z określonym atrybutem..."defer" niektórzy dodają jeszcze atrybut "async" ale o tym proponuje poczytać...więcej...
https://stackoverflow.com/questions/10808109/script-tag-async-defer
https://developer.mozilla.org/en-US/docs/Web/HTML/Element/script
a więc jeśli jest w sekcji head:
<script src="script.js" defer></script>
w znacznikach <img> dodałbym atrybut "alt" który nie jest obowiązkowy ale ...
https://developer.mozilla.org/en-US/docs/Web/HTML/Element/img
W Css dodałbym w resetach jeszcze box-sizing: border-box;
Oczywiście żeby było jasne to tylko moje sugestie
Style i Skrypt w oddzielnych plikach co jest moim zdaniem bardzo dobrą praktyką!
W html podoba mi się Twoja Konwencja: <header> a następnie <main> użyłbym takiej samej! Wiemy że są różne preferencje
Jeśli chodzi o kod JS nie wypowiadam się (na razie). Pogłębiam wiedzę..
Btw też korzystałem ( i pewnie skorzystam jeszcze nie raz )z Frontend Mentor bardzo lubię tę stronę!
Pozdrawiam serdecznie i życzę miłego Weekendu !