Ocena portfolio

Question

Witam. Od jakiegoś czasu szkolę się w technologiach front-endowych i w końcu doszedłem do poziomu, który pozwolił mi na stworzenie portfolio, z którego jestem w miarę zadowolony. Jako, że przyda się opinia osób trzecich co do strony wizualnej i kodu strony, wstawiam ją tutaj. Zaznaczam, że gotowy jest tylko design strony, także nie zwracajcie uwagi na jej zawartość. Z góry dziękuję za wszystkie wskazówki.
LINK: https://graatz.github.io/portfolio-template/

Answer 1

1. Brak określonego języka strony - html lang="en"
2. Brak informacji o kodowaniu znakow - charset="UTF-8"
3. type="text/css" jest niepotrzebne
4. Jeśli nie będziesz korzystał z Server Push - masz o 4 pliki CSS za dużo
5. Skrypty na koniec body - nie blokuj renderowania strony
6. Poczytaj o stosowaniu nagłówków - u Ciebie są użyte źle
7. <a class="logo" alt="Logo"> - a nie ma atrybutu alt
8. Nie wiem czy strong w nagłówkach to najlepszy pomysł
9. Albo button, albo a - nie oba na raz
10. Bezsensowna lista w nav - listy w HTML
11. Dobrą praktyką jest nieużywanie id w CSS - class vs id
12. Linki z ikonkami w menu nie mają treści - O ikonkach słów kilka
13. Nie mogę wybrać linków z poziomu klawiatury
14. Szary tekst ma zbyt niski kontrast
15. Użyj tylko CSS-a do animacji projektów - Kurs CSS odc. 5
16. <br> nie </br>
17. Mam wrażenie, że mylisz span ze strong
18. beforeunload scrollTop(0) dlaczego?
19. Nie zostawiaj console.log na produkcji
20. Nie pisz CSS-a w JS, w JS zmieniaj tylko klasy
21. Potrzebny jest tutaj bootstrap.js?
22. Nie mogę zjechać niżej używając strzałek/page down
23. Na mobile nie mogę przescrollować projektów
24. Na mobile nie widzę opisów projektów
25. Na mobile elementy menu są zbyt blisko siebie
26. Na mobile, poziomo, na iPhonie 5, strzałka najeżdża na tekst

Z wyglądu (pomijając kontrast i zbyt małe klikalne elementy na mobile) jest ok.

Zrezygnowałbym z tego scrolla - więcej z tego problemów, niż pożytku.

Comments

Wpadłem popatrzeć na Portfolio Graatza i sam skorzystałem na Twoich uwagach. Ważne wskazówki. Thx.