Strona o Józefie Piłsudskim - prośba oceny

pytanie zadane 21 października 2017 w Nasze projekty przez Gambr Dyskutant (7,530 p.)
zamknięte 21 października 2017 przez Gambr

adres strony

Powyższa strona, to projekt, który pisałem na konkurs historyczny o Piłsudskim. Proszę o feedback, jeżeli chodzi o front-end jestem bardzo słaby i zdecydowanie wolę aplikacje mobilne ,ale trochę konstruktywnej krytyki zawszę się przyda :p

komentarz zamknięcia: ...

komentarz 23 października 2017 przez Einstein21 Obywatel (1,420 p.)

W grze memo jest błąd. Jeśli dwa razy klikniesz na ten sam obrazek zalicza ci jako znalezione. Widzę że zrobiłeś wg. kursu i na tym koniec (oprócz zmiany treści). Nie jest to trudne do zrobienia więc polecam wykonać taką poprawkę

EDIT:

Ciekawostki są nie resoponzywne. Mam kwadratowy drugi monitor i nie mieści mi się calość

2 odpowiedzi

odpowiedź 21 października 2017 przez Benek Szeryf (91,010 p.)

Po co umieszczasz na stronie o Piłsudskim wpis dotyczący JS? WTF?

Witryna jest napisana od podstaw w czystym html i css. Jej budowa jest zgodna ze standardem html5.

Nie jest.

komentarz 21 października 2017 przez Gambr Dyskutant (7,530 p.)

Prosili aby wypisać wszystko co było użyte. Ja odpowiadałem za treść i javascript.

komentarz 21 października 2017 przez Gambr Dyskutant (7,530 p.)

A co do html, co tam poprawić, żeby było lepiej? Dodać więcej tagów semantycznych ?

komentarz 21 października 2017 przez Filip31411 Dyskutant (8,820 p.)

http://tutorials.comandeer.pl/html5-blog.html

komentarz 21 października 2017 przez Benek Szeryf (91,010 p.)

Walidator zwraca 1 ostrzeżenie i 21 błędów
JS powinien być wczytywany na końcu, po załadowaniu treści, tak by nie przedłużać ładowania całej strony. U Ciebie jest osadzony w sekcji head
Znacznik <br> nie służy do pozycjonowania elementów strony
Nie powinno się używać CSS-a inline:
```
<div class="face" style="height:332px">
```

odpowiedź 21 października 2017 przez ProgramistaStepek Nałogowiec (27,020 p.)

Twoja strona ma dwie twarze. Z jednej strony masa informacji historycznych, ciekawy sposób na przekazywanie wiedzy (memo), niezłe ciekawostki. To są zdecydowanie plusy twojej strony. Niestety kod pozostawia trochę do życzenia. Brakujące "alty", częste stosowanie <br>, style inline a to tylko niektóre z błędów.

komentarz 21 października 2017 przez Gambr Dyskutant (7,530 p.)

Da się w tym jakoś pogrzebać i to ulepszyć nie pisząc tego od nowa ?

komentarz 21 października 2017 przez ProgramistaStepek Nałogowiec (27,020 p.)

Nie mam teraz czasu na dokładne wypisywanie każdego twojego błędu. Na pewno do poprawy leci semantyka strony (a właściwie jej brak, lub fragmentaryczność), i polecam Ci się teraz na tym skupić.

komentarz 21 października 2017 przez Gambr Dyskutant (7,530 p.)

Okej, dodamy alty i znaczniki semantyczne. Dziękuje za feedback, spodziewałem się, że zostane zmasakrowany za swoją amatorszczyznę. Jakbyś miał czas, to mógłbyś zerknąc wieczorem na kod, kiedy już trochę nad nim popracuję?