Kod:
- bałagan w head - style własne zawszą idą na koniec
<img class="logo" src="img/logo3.svg" alt="logo artphotography">
<h2>Art photography</h2>
- gdzie tutaj to jeszcze można przyjąć to...
<div class="image">
<img src="img/1.jpg" alt="fotografia sportowa">
<div class="wrapper_text" data-aos="fade-up">
<h2>Fotografia sportowa</h2>
<p>
Lorem ipsum dolor sit amet, consectetur adipiscing elit. Vivamus maximus arcu sed sodales ultricies.
Duis scelerisque, ex sit amet tempus Maecenas purus eros, aliquam non pellentesque nec, hendrerit ut.
</p>
</div>
</div>
już niekoniecznie. Obrazek tutaj to wyłącznie dekoracja i nie potrzebuje atrybutu alt.
<section id="projectSection">
- nie wiesz do końca co będzie na tej stronie - ta część może się zmienić. Nazwami klas nie należy sugerować zawartości contentu. Jeżeli chcesz to robić, to specyficzne dla danego contentu rzeczy umieść za pomocą usadzenia drugiej klasy.
- czy galeria to nie lista zdjęć?
<h2 class="title_top">Nasi klienci</h2>
<h3 class="title_bottom">Oto najwięksi partnerzy naszej firmy</h3>
- znowu sugerowanie nazwami klas,
<li class="item">
<img src="img/wektor.svg" alt="firma x">
<h2>Firma X</h2>
<p>Lorem ipsum dolor sit amet, consectetur adipiscing elit. Nulla finibus nibh et nulla fermentum, ac porta erat scelerisque. Nullam quis libero efficitur eros facilisis facilisis vitae a nulla.</p>
</li>
- tutaj nie dokońca to wygląda ok. Przedewszystkim nie widzę tego jako listy, może bardziej artykuł wraz z sekcjami? Ponad img mogłoby mieć lepiej wypełniony alt.
- footer ewidentnie za wcześnie. Nie jestem pewny, ale footer chyba nie może zawierać jakichkolwiek nagłówków - a Twój posiada. To byłoby dobre section.
<div class="rectangle_team">
<li class="team_item"><img class="circle" src="img/person2.jpg"></li>
<p>Zofia Kowalska</p>
<p>zofia.k@art.com</p>
</div>
- mały zawał serca - czy ul może działać w taki sposób że ul > div > li? Ponadto fajnie by było gdyby kontakt mailowy był interaktywny - można wykorzystać mailto oraz znacznik a.
- extra_links_li - sugerujesz element w kodzie html,
<p class="copy">Wszelkie prawa zastrzeżone © 2017 Dziękuje za wizyte!</p>
- to byłby footer. No i ja dziękuję, a on dziękuje ;)\
<button onclick="topFunction()" class="click"><span class="textInside">top</span></button>
- onclick to zła praktyka - ale ok, z JSem różnie :P Nazwa klasy też jest średnia, czy span jest potrzebny? Poza tym mało mówiący komunikat "top" - pierwsza myśl jaka mnie napotkała to "chyba nie załadowała mi się grafika.."
W CSS:
.foto1{
grid-row: 1/3;
}
.foto6{
grid-column: 1/3;
height: 400px;
}
- skoro używasz kolumn, to dlaczego nie zastosujesz również wierszy?
.circle{
border-radius: 50%;
}
- Klasa znowu lekko niesemantyczna - nie lepiej coś na styl rounded-img?
- używasz float: left oraz float: right, ale nie widzę abyś je czyścił. Warto o tym pamiętać, ponieważ później mogą wystąpić ewentualne problemy z layoutem.
Podsumowywując: do poprawy :P Zostawiasz również dużo zbędnego miejsca w pliku, warto to czyścić. Ponadto musisz nieco logiczniej używać nazw klas, wtedy Twój plik CSS może stać się chudszy.
