Codereview

Mateuszto
  • Rejestracja:prawie 7 lat
  • Ostatnio:5 miesięcy
  • Lokalizacja:Poznań
  • Postów:163
0

Cześć, jeżeli miał ktoś trochę czasu na zajrzenie do kodu byłbym wdzięczny ;)
https://github.com/Mateuszto/Pomarancza---Strona-do-cwiczenia

Pomijając nazewnictwo jak ktoś pod ostatnim moim postem napisał(cr z innej stronki) To mam nadziję, że będzie okej ;) Czekam na uwagi i inne porady;) Z góry dziękuje!

czysteskarpety
czysteskarpety
  • Rejestracja:około 10 lat
  • Ostatnio:ponad 4 lata
  • Lokalizacja:Piwnica
  • Postów:7697
1

nie ma za wiele do oceny, na pewno jak pisałeś rzucają się w oczy polskie nazwy klas, nie wiem czy musisz dodawać aż tyle typów fontów do jednej strony, warto też przy takich linkach dodawać latin-ext bo bez tego często nie wyświetlają się polskie znaki


JE
  • Rejestracja:prawie 7 lat
  • Ostatnio:ponad 6 lat
  • Postów:1
1

Kilka rzeczy, które rzucają się w oczy:

  • h1 wstawił bym do header, a nie do nav, gdyż odnosi się do całej strony
  • Obrazki nie mają alt="", akurat przy zdjęciach na tej stronie powinny być puste
  • To nie powinnno być w nagłówku, a w tagu p
Kopiuj
<h2 class="podtekst">Pomarańcza – nazwa, jaką określa się niektóre gatunki roślin należące do rodzaju cytrus, zwykle o pomarańczowej skórce i włóknistym miąższu. Jest to nazwa tylko zwyczajowa, nie stanowi ona odrębnej jednostki taksonomicznej. Gatunki pomarańczy pochodzą z Chin i wschodniej Azji.</h2>

To powinno być linkiem

Kopiuj
<h3 class="nadus">Kliknij i poczytaj</h3>
  • Od tego istnieje semantyczny tag footer
Kopiuj
 <section id="foot">
      <h2>Mateusz Tomczyk - Wszelkie prawa zastrzeżone</h2>
 </section>
  • id zostaw do JS, styluj tylko po klasach

Zarejestruj się i dołącz do największej społeczności programistów w Polsce.

Otrzymaj wsparcie, dziel się wiedzą i rozwijaj swoje umiejętności z najlepszymi.