Ocena strony

pytanie zadane 25 kwietnia 2016 w Nasze projekty przez ShadoWs Bywalec (2,800 p.)

Witam serdecznie chciałbym was prosić o ocenę mojej strony i jakieś podpowiedzi odnośnie małych poprawek czy też zmian. Wiem że można na was liczyć. http://testdj.hekko24.pl/ Z góry dziękuję za pomoc :)

6 odpowiedzi

odpowiedź 25 kwietnia 2016 przez radek024 Szeryf (77,160 p.)

za dużo plików CSS, zminimalizuj to
stylizujesz body na position:relative? raczej rzadko spotykane rozwiązanie
dla mnie ksywki dj'ów to nagłówek główny - h1
taget:_blank niezalecany ze względu na to, że to użytkownik powinien wybierać, co ma się gdzie pojawić i z powodu trudności z dostępnością
stylizowanie inline - słabo...
stylizujesz id - niezalecane, jeżeli chcesz wiedzieć dlaczego, szukaj w moich poprzednich wypowiedziach
hr mógłby być border bottom dla h2, tym samym kod byłby prostszy i bardziej skalowalny
znacznik a powinien mieć artybut title - użytkownik powinien wiedzieć, w co klika
przerwy br? używaj margin oraz padding!
raz klasy po polsku, raz po angielsku
stylizuj klasy, a nie tagi
znaki specjalne robimy za pomocą encji, a nie copy&paste
ilość skryptów także przeraża

Z pewnością będą jeszcze jakieś błędy, ale tyle wymieniłem - w dużej części są one nieznaczne, ale warto je poprawić.

odpowiedź 25 kwietnia 2016 przez Maciek Gamrot Bywalec (2,830 p.)

Strona całkiem niezła, nie ma się do czego przyczepić, równie dobrze działa na telefonie.

komentarz 25 kwietnia 2016 przez Konrad Nabożny Stary wyjadacz (13,460 p.)

Ja bym się przyczepił o takie małe i lekko irytujące szczegóły jak np. po najechaniu na zdjęcie w galerii ikona podglądu pojawia się zbyt wolno, napis home zaraz pod stroną tytułową jest nieco wyżej niż nawigacja po stronie. Mógłby być on również nieco większy. Kolejna rzecz, nie wiem czy to przypadek ale pod "YouTube" jest facebook. Wyśrodkował bym w pionie również Copyright na samym dole. (To tylko moje sugestie ze strony zwykłego użytkownika)

PS. Przeniesienie na górę strony rozwiązane lecącą rakietą genialne jak dla mnie :)

odpowiedź 25 kwietnia 2016 przez Comandeer Guru (601,590 p.)

odpowiedź 25 kwietnia 2016 przez niezalogowany
edycja 25 kwietnia 2016

Pojedynczych tagów nie domyka się "/"

meta http-equiv="X-UA-Compatible" content="IE=edge" />

To można wysyłać jako nagłówek HTTP

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

type do cssów jest niepotrzebny.

Nie ma HTTP2.0 ? wszystkie CSSy zminimalizować i wrzucić do jednego pliku - im mniej requestów tym lepiej.
Ładujesz całe font-awesome.. dla 2 ikonek. Zdecydowanie optymalne rozwiązanie.
style.css wypadałoby wrzucić jako ostatnie.. a nóż coś sobie można nadpisać.
```
<div class="intro-heading">DJ SeBo & DJ DawiD</div>
```
To powinno być głównym nagłówkiem strony! - którego teraz brak.
target blank - dont do this. https://kornel.ski/pl/target, https://css-tricks.com/use-target_blank/
```
<a href="#Onas"> <span class="arrow"><i class="glyphicon glyphicon-menu-down"></i></span></a>
```
Pusty link == niepotrzebny link. Co do używania ikonek odsyłam do http://forum.pasja-informatyki.pl/128857/font-awesome-4-6-0-accessibility

<span class="sr-only">Toggle navigation</span>

To powinno być przetłumaczone.

Fajnie by było, gdyby zdjęcie DJów miało jakiś sensowny alt.
```
<h2>O nas</h2>
<hr class="os">
```
hr ma semantyczne znaczenie, tutaj powinno być użyte border-bottom dla h2.. albo na pseudo-elemencie strzelić kreskę, jeśli trzeba dopasować ładnie szerokość.
```
		</div>
		<br>
		<br>
		<br>
```
Nie używamy br do robienia odstępów. Margin-bottom!

<div class="col-lg-3 col-md-4 col-xs-6 thumb Sprzęt">
	<a href="img/testimonials-bg.jpg" class="thumbnail link fade-in">
		<img class="img-responsive" src="img/testimonials-bg.jpg" alt="" />
	</a>
</div>
<div class="col-lg-3 col-md-4 col-xs-6 thumb Sprzęt">
	<a href="img/testimonials-bg.jpg" class="thumbnail link fade-in">
		<img class="img-responsive" src="img/testimonials-bg.jpg" alt="" />
	</a>
</div>

To są de facto puste linki, mam nadzieje, że pojawi się tam alt, gdy galeria zostanie uzupełniona normalnymi zdjęciami.

<li><a href="#"><i class="fa fa-youtube"></i></a>

: )

No nie.. dlaczego label jest zakomentowany? przecież to jest podstawa jeśli chodzi o dostępność formularzy!
Brakuje tagu footer na końcu.
Adres strony powinien obsługiwać zmiany hashy podczas korzystania z menu.
Strona dostępna z www. i bez www. - .htaccess i ujednolicić adres proszę. Same problemy przy dwóch adresach są.
Google krzyczy, że należałoby zoptymalizować obrazy.
Na strzałce w dół, nie pojawia się outline przy focusowaniu.
"Witamy" za bardzo mi przypomina comic sans : d (ale to już osobista uwaga)
Najlepsze na koniec. Wyświetla mi się komunikat, że strona korzysta z cookies. Potwierdzam, strona korzysta z cookies - z jednego - z tego które obsługuje wyświetlanie komunikatu o cookies : )

CSS i Javascript na tej stronie.. to już temat na inną bajkę.

komentarz 25 kwietnia 2016 przez ShadoWs Bywalec (2,800 p.)

Bardzo dziękuje za tak obszerne rozpisanie problemów. Jestem początkujący w tych sprawach :)

komentarz 25 kwietnia 2016 przez radek024 Szeryf (77,160 p.)

Temat na inną bajkę? Rozumiem że w stylu Never Ending Story? :D

1	komentarz 25 kwietnia 2016 przez niezalogowany Magiczna brama do innego świata - wchodzisz w plik CSSa i świat zwalnia! : )

1	komentarz 25 kwietnia 2016 przez niezalogowany A już tak bardziej serio, to nie chciałem powtarzać tego co napisałeś. Jeśli chodzi o JSa - dałoby się to wszystko zamknąć w 20 linijek, i to z jakimś scroll-smoothem zamiast jquery.