Możecie ocenić stronę http://pvp.net.pl:81/
login: tester
hasło: lol123
Pełna wersja: Strona gry
mamy oceniać kod, grafikę, czy co? 
jeżeli kod, to:
- skrypty js powinny być załączane na końcu strony (w wp_footer())
- doctype html5 a żadnego html5 nie widzę - nie używasz nawet tak oczywistych tagów jak header, footer...
- duży styli inline
-
-
-
-
- za dużo inline js, mógłbyś to wrzucić do osobnego pliku
- stosujesz dużo niepotrzebnego kodu w stylach, np. divy mają domyślnie width: 100%, niepotrzebnie im go nadajesz
-
- patrzę na kod i zastanawia mnie po co jest div#wrapper - przecież on nic nie robi...
- dodatkowo mógłbyś skorzystać z jakiejś wtyczki, która zajmie się cache
ogólnie nie jest źle
jeżeli kod, to:
- skrypty js powinny być załączane na końcu strony (w wp_footer())
- doctype html5 a żadnego html5 nie widzę - nie używasz nawet tak oczywistych tagów jak header, footer...
- duży styli inline
-
Kod
<br style="clear:both">
ktoś tu chyba o clear fix nie słyszał -
Kod
<a href="#pu_register" class="login-window register" style="display:none;"></a>
do rzeczy obsługiwanych przez js nie używa się <a>-
Kod
<li id="menu-item-27" class="menu-item menu-item-type-post_type menu-item-object-page menu-item-27">
za pomocą rozszerzenia klasy Walker mógłbyś się pozbyć tych niepotrzebnych klas, minimalnie zaoszczędzisz na transferze i kod będzie czytelniejszy -
Kod
@import url('http://pvp.net.pl:81/wp-content/plugins/ml-slider/assets/metaslider/public.css?ver=2.3');
nigdy nie używaj importu (to samo dotyczy fonta w stylu)- za dużo inline js, mógłbyś to wrzucić do osobnego pliku
- stosujesz dużo niepotrzebnego kodu w stylach, np. divy mają domyślnie width: 100%, niepotrzebnie im go nadajesz
-
Kod
body,
input,
textarea,
.page-title span,
.pingback a.url {
font-family: 'Lato', serif;
}
h3#comments-title,
h3#reply-title,
#access .menu,
#access div.menu ul,
#cancel-comment-reply-link,
.form-allowed-tags,
#site-info,
#site-title,
#wp-calendar,
.comment-meta,
.comment-body tr th,
.comment-body thead th,
.entry-content label,
.entry-content tr th,
.entry-content thead th,
.entry-meta,
.entry-title,
.entry-utility,
#respond label,
.navigation,
.page-title,
.pingback p,
.reply,
.widget-title,
.wp-caption-text {
font-family: 'Lato', serif;
}
input[type="submit"] {
font-family: 'Lato', serif;
}
to jest chyba jakiś żart input,
textarea,
.page-title span,
.pingback a.url {
font-family: 'Lato', serif;
}
h3#comments-title,
h3#reply-title,
#access .menu,
#access div.menu ul,
#cancel-comment-reply-link,
.form-allowed-tags,
#site-info,
#site-title,
#wp-calendar,
.comment-meta,
.comment-body tr th,
.comment-body thead th,
.entry-content label,
.entry-content tr th,
.entry-content thead th,
.entry-meta,
.entry-title,
.entry-utility,
#respond label,
.navigation,
.page-title,
.pingback p,
.reply,
.widget-title,
.wp-caption-text {
font-family: 'Lato', serif;
}
input[type="submit"] {
font-family: 'Lato', serif;
}
- patrzę na kod i zastanawia mnie po co jest div#wrapper - przecież on nic nie robi...
- dodatkowo mógłbyś skorzystać z jakiejś wtyczki, która zajmie się cache
ogólnie nie jest źle
Rejestracja
Notice: Undefined index: register in /var/www/dev/wp-content/themes/pvpnet/header.php on line 98
http://pvp.net.pl:81/testuje-nius/#more-249
Napis 'PVP.NET.PL NADCHODZI' nie mieści się
Notice: Undefined index: register in /var/www/dev/wp-content/themes/pvpnet/header.php on line 98
http://pvp.net.pl:81/testuje-nius/#more-249
Napis 'PVP.NET.PL NADCHODZI' nie mieści się
To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.