[www] http://www.dania.glt.pl Opcje

[3miel]

Witam,

strona wykonana na konkurs:

http://www.dania.glt.pl

Niestety zagapiłem się troszkę i nie zdążyłem dac Wam jej do oceny przed zakończeniem konkursu, ale i tak chciałbym usłyszeć wasze opinie na jej temat.

Pozdrawiam

---------------------
Edit

Zajec:
ad 1. Strona musiała być zrobiona jak najszybciej... z lenistwa:)
ad 2. jw:P Ale również gdzies się z takim czymś spotkałem i jakoś mi tak zostało
ad 3. teraz też to zauważyłem
ad 4. Gdzie tam masz zagnieżdżona tabelke? które menu (sa 3)
ad 5. na przyszłość będe tego stosował
ad 6. nie za bardoz mi to wychodzi, a jak juz wspomniałem było mało czasu, którego nie mogłem przeznaczć na eksperymentowanie.
dzieki za krytyke, przyda mi sie!

----
Zapomnialem opcji wyłączenia głosu... w przyszłości o tym nie zapomne.

No masz racje, zagnieżdzam tabelki, ale chyba nei wygląda to najgorzej, a na przyszłość na pewno bede używał stylów, właściwie to już zrobiłem stronke używając CSS bez tabel. Efekt imponujący, wkład pracy ogromny = zeby było cos pieknego musi być dużo enegri:)

Ten post edytował 3miel 14.03.2006, 18:06:58

[Zajec]

Przejście do prawdziwej strony bez wymagania Flash - dobrze.

1) Fragment kodu: <br /><br /> - do odstępów uzywa się raczej marginesów.

2) Fragment CSS: align:center; - a cóż to? ;-) Może chciałeś text-align?

3) Kolejny fragment:

[XML] pobierz, plaintext 
<div class="menu">
<a href="#" class="menu_link">link 1</a><div class="menu_seperator"></div>
<a href="#" class="menu_link">link 2</a><div class="menu_seperator"></div>
<a href="#" class="menu_link">link 3</a><div class="menu_seperator"></div>
</div>
[XML] pobierz, plaintext 

Wywal spokojnie te klasy i użyj po prostu konstrukcji

Kod

.menu a { }
.menu div { }

4) O ile tabelkowy szablon jeszcze jest do zniesienia, to zagnieżdżanie dodatkowej tabeli w celu zbudowania menu jest całkowicie zbędne. I średnio poprawne.

5) Menu to listy, powinieneś użyć <ul>

6) Generalnie miło by było, gdybyś wykorzystał same elementy blokowe. Pozbyłbyś się w ten sposób kwiatków typu:

Kod

<tr>
<td class="tresc_dol_l">&nbsp;</td>
<td class="tresc_dol">&nbsp;</td>
<td class="tresc_dol_p">&nbsp;</td>
</tr>