[portal] Nowa wersja www.monstrum.com.pl Opcje

[Kildyt]

Cześć!

Proszę o ocenę nowej wersji portalu. Gdy przeniesiemy większość starych rzeczy to wystartujemy oficjalnie.

Co do kodu PHP to nie jestem zbytnio zadowolony, no ale nie jest tragicznie. (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) Piszę aktualnie od nowa redbooka więc dużo popełnionych błędów nauczyło mnie jak pisać lepszy kod.

Poddaję się waszej krytyce. (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

Ten post edytował Kildyt 10.03.2009, 14:56:50

[.radex]

Ładnie (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

Kod

style="width: 54%;"

jak już musisz dawać width (nieładne) to się może nie fatyguj z inline i po prostu daj width="54%"

Kod

<div class="rateline"><div class="line" style="width: 0%; background-color: #E22D6F;"><hr /></div></div>

nie idzie tego skrócić? po za tym przy width: zaokrąglaj do jedności, bo i tak ten pasek jest zbyt mały, żeby zauważyć różnicę

Kod

<table class="listgame_top-margin">
                                <tr>
                                    <td style="width: 8%;"></td>
                                    <td style="width: 8%;"></td>
                                    <td style="width: 64%;" class="text-indent">Tytuł gry</td>
                                    <td style="width: 20%;" class="text-indent">Ocena</td>
                                </tr>
                            </table>
                            
                            <table class="listgame">

dwie tabele? <td> zamiast <th> w nagłówkach?

Ogółem divitis w kodzie.