Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

 
Reply to this topicStart new topic
> Strona gry
Outside
post
Post #1





Grupa: Zarejestrowani
Postów: 63
Pomógł: 1
Dołączył: 28.08.2013

Ostrzeżenie: (0%)
-----


Możecie ocenić stronę http://pvp.net.pl:81/
login: tester
hasło: lol123
Go to the top of the page
+Quote Post
Pawel_W
post
Post #2





Grupa: Zarejestrowani
Postów: 1 675
Pomógł: 286
Dołączył: 15.06.2009
Skąd: Wieliczka

Ostrzeżenie: (0%)
-----


mamy oceniać kod, grafikę, czy co? (IMG:style_emoticons/default/smile.gif)

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ł (IMG:style_emoticons/default/smile.gif)
-
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 (IMG:style_emoticons/default/smile.gif)
-
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 (IMG:style_emoticons/default/smile.gif)
- 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 (IMG:style_emoticons/default/smile.gif)
Go to the top of the page
+Quote Post
Turson
post
Post #3





Grupa: Zarejestrowani
Postów: 4 291
Pomógł: 829
Dołączył: 14.02.2009
Skąd: łódź

Ostrzeżenie: (0%)
-----


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ę
Go to the top of the page
+Quote Post

Reply to this topicStart new topic
2 Użytkowników czyta ten temat (2 Gości i 0 Anonimowych użytkowników)
0 Zarejestrowanych:

 



RSS Aktualny czas: 4.10.2025 - 03:35