Ocena mojego portfolio - strona WWW

pytanie zadane 12 listopada 2016 w Nasze projekty przez xSvilen Mądrala (5,740 p.)
edycja 13 listopada 2016 przez xSvilen

Cześć! Proszę o ocenę mojej strony WWW (janmager.pl) w skali:

1) od 1 do 5 - pod względem wyglądu, działania, przyjemności z użytkowania
2) od 1 do 5 - pod względem technicznym, jakość kodu itp.

Z góry dzięki za wszystkie konstruktywne krytyki, rady, poprawki :)

2 odpowiedzi

odpowiedź 13 listopada 2016 przez radek024 Szeryf (77,160 p.)

UX bardzo pozytywnie, jak zresztą i w pozostałych projektach(już chyba kiedyś pisałem o którymś z nich). Aż żal wypominać błędy, bo strona wygląda super - ale no są i należy je poprawić:

<link rel="stylesheet" type="text/css" href="style.css">

Ten zapis powininen być zawsze na końcu - Ty dałeś go jako pierwszy arkusz styli.

<div class="main" style="margin-bottom: 100px;">

Style inline to zły pomysł

<h1>JAN MAGER</h1>

Tutaj powinieneś to zrobić za pomocą CSSa - text-tranform: uppercase;

<div class="clear-zone"></div>
<hr style="clear:both">

Tutaj div może zostać zastępiony pseudoelementem, a hr nie jest znacznikiem formatującym, a semantycznym - warto się zastanowić, czy nie zmienić go na border-top lub border-bottom.

<h1>kontakt@janmager.pl<div class="mystery-footer">Copyright &copy; 2013-2016 Jan Mager PC</div></h1>

X raz użycie znacznika tytułowania pierwszego stopnia mówi o tym, że outline będzie wyglądał źle... na dodatek div w h1? Trochę dziwne :v
I jakoś kod jest mało sematyczny

<a href="#.html"><img src="images/language-pl.png"></a>

Tutaj warto dodać opis do obrazków - odgrywają one znaczenie

<div class="social-icons">

To powinna być lista.
Po kliknięciu w angieską wersję strony artybut lang dla html jest taki sam.

Walidator wali o alt dla img i o tego diva w h1. Czyli jednak trochę błędów jest, a to tylko strona startowa.

odpowiedź 13 listopada 2016 przez Arkadiusz Waluk Ekspert (289,990 p.)
edycja 13 listopada 2016 przez Arkadiusz Waluk

No no, szczerze mówiąc to mi pod względem wizualnym strona bardzo się podoba, dawno tak fajnej tu nie widziałem. Czysta, prosta, bez wybuchowych efektów, tak lubię.

Co do kodu to z rzeczy które rzuciły mi się na szybko w oczy:

<link rel="stylesheet" type="text/css" href="style.css">

Atrybut type w HTML5 jest niepotrzebny.

```
<hr style="clear:both" class="mystery">
```
Style inline są słabe, do clearowania lepiej użyć pseudoelementów.
```
<img src="images/about-avatar.png">
```
Obrazki powinny mieć alt.
Brak nowych semantycznych znaczników z HTML5.
Nagłówek <h1> użyty kilka razy i to w dziwny sposób, np. w stopce.

Wygląd oceniłbym więc na 5, a kod na 3.

Edit: och, już mnie @radek024 uprzedził co do większości uwag dotyczących kodu (albo i wszystkich).

komentarz 13 listopada 2016 przez andriej0 Nowicjusz (100 p.)

Od strony wizualnej bardzo mi się podoba. Jeżeli chodzi o kod, to tak jak kolega wyżej wspomniał - jest kilka błędów. Powrót do strony głównej za pomocą strzałki w prawo jest moim zdaniem mało intuicyjny. Wydaje się, że po jego kliknięciu przejdę do następnej podstrony. Lepszym rozwiązaniem byłoby breadcrumbs lub wyświetlanie headera na wszystkich podstronach.

1	komentarz 13 listopada 2016 przez niezalogowany Brak nowych semantycznych znaczników z HTML5. HTML5 ma już z 4 lata na karku, z czego 2 jako oficjalny standard - nowe znaczniki to dużo powiedziane : p

komentarz 13 listopada 2016 przez Arkadiusz Waluk Ekspert (289,990 p.)

W zasadzie masz rację, tak czasem nazywam sobie to nowymi w porównaniu do pozostałych (w sensie że wcześniej ich nie było i dużo osób stosuje same divy). Ale ogólnie nie są nowe, więc spoko, poprawiłem w odpowiedzi :p