Projekt zakodowany z PSD do oceny.

@Tworest Ale jak mamy ocenić kod bez live demo? 🙂 

Składnia kodu jest w porządku, używasz angielskich nazw klas, kod jest schludny i przejrzysty, oddzielasz poszczególne sekcje komentarzami.

Niestety brakuje responsywności, tekst w niektórych miejscach ma za duży rozmiar, dolna sekcja do poprawy.

Tekst Placeholder jest zbyt duży, przycisk mógłbby być zaokrąglony a napis równo, w stopce napis mógł być jednolitego koloru oraz znacznie mniejszy. 

A i duże grafiki dodawaj w nowym formie WebP, dla szybszego wczytywania.

1 godzinę temu, CzareK napisał:

Zazwyczaj nagłówka poziomu H1 używa się jedynie raz na stronie.

Jest już obalone to stwierdzenie.
Nie mam co prawda pod ręką artykułu na ten temat, ale `h1` możesz użyć także wewnątrz `article` i SEO na tym nie traci.

Ale cenna uwaga, bo sam heading jest źle zbudowany (zamykany).

1. Nie stylujemy po ID!!!

2. <button><a href="">Poznaj nas</a></button>

   Albo button, albo link. Nie ma nic pomiędzy.

3. <h1 class="col-1">Stwórz juz teraz<span> własną markę</span></h2>
   <h2 class="col-2">razem z nami !</h3>

   Brak zamkniętego znacznika h1, nie mówiąc już o tym że h2 i h3 to tutaj nie jest potrzebne.

4. <h1>Damian</h1>
   <h2>Grafik</h2>

   To też jest źle zrobione. Znaczniki H muszą mieć jakieś znaczenie, a nie pojedyncze wyrazy.

5. <footer>
   	<span>2021©</span><h1>Tworest i Mateo</h1>
   	<i class="fab fa-facebook"></i>
   </footer>

   W stopce h1? A po co?

6. Inputy nie są podpisane (Nie mają label),

7. <br>

   Nie używaj już stylowania z poprzednich wersji HTML. Użyj do tego CSS.

8. W CSS staraj się używać flexa lub grida. Unikaj używania float jeżeli nie jest to konieczne.

9. Zachęcam do korzystania ze zmiennych w CSS 🙂

10. Brak RWD,

11. href nie może zostawać pusty. Zamiast tego w zawartość wpisz #,

12. Swoje prace wstawiaj na Github,

13. Do deployu strony możesz się posłużyć Vercel lub Github Pages