[Projekt] Pierwsza moja strona na Div'ach Opcje

[rafolik]

Witam

Prosił bym o ocene mojego PIERWSZEGO projektu wykonanego na "DIV'ach"
Mile widziana także ocena laya..
Oto link do strony: http://www.rafolik.kom.pl/pc/

Pozdrawiam

Ten post edytował rafolik 30.03.2007, 11:39:33

[ano]

Kilka rad:
1. Zamiast:

[HTML] pobierz, plaintext 
<div id="data">Dzisiaj jest piątek, 30 marca 2007 r. 89 dzień roku, imieniny: Amelii Dobromira Leonarda&nbsp;</div>
<div id="logo"> </div>
[HTML] pobierz, plaintext 

Może lepiej byłoby:

[HTML] pobierz, plaintext 
<span>Dzisiaj jest[...]</span>
<h1>Nazwa strony</h1>
[HTML] pobierz, plaintext 

(odpowiednio to ostyluj)

2. Takie coś:

[HTML] pobierz, plaintext 
<div id="lewo">
<div class="menu">
[HTML] pobierz, plaintext 

możesz zapisać jako:

[HTML] pobierz, plaintext 
<div id="lewo">
<div></div>
[HTML] pobierz, plaintext 

a w stylach odnosić się:
#lewo div { ... }

3. W menu pousuwaj <br />, zamiast tego przerwy kontroluj marginesami

4. W menu, zamiast np.
<li>Hardware</li>
używaj <h2> hardware</h2> << jest to tytuł 2 stopnia.

5. W <div id="srodek"> do tekstu używaj paragrafów, np:

[HTML] pobierz, plaintext 
<p>
  Lorem ipsum dolor sit amet, consectetuer adipiscing elit. 
  Nullam aliquam enim id felis. Phasellus lobortis, pede quis 
  nonummy scelerisque, enim ligula imperdiet magna, luctus 
  laoreet felis eros nec purus. Proin porttitor felis quis 
  nisl. Sed pharetra tempus dolor. Nunc sollicitudin. 
  Praesent vestibulum consequat justo. Cum sociis natoque 
  penatibus et magnis dis parturient montes, nascetur 
  ridiculus mus. 
</p>
[HTML] pobierz, plaintext 

i nie musisz nawet stosować <hr/>, wystarczy ostylować <p> (zastosuj border-bottom dla <p/>)

6. <p>R E K L A M A</p> << zamień na <h2>Reklama</h2> (możesz w stylach ustawić odstępy między literami i żeby tekst był pisany dużymi literami)

7. Jeszcze jedna rada, stosuj formatowanie kodu, żebyś mógł w przyszłości łatwiej odczytywać kod w dużych projektach.
Przykład:
zamiast

[HTML] pobierz, plaintext 
<ul>
<li>aaa</li>
<li>aaa</li>
</ul>
[HTML] pobierz, plaintext 

pisz:

[HTML] pobierz, plaintext 
<ul>
  <li>aaa</li>
  <li>aaa</li>
</ul>
[HTML] pobierz, plaintext 

To tyle rad, pozdrawiam!

PS. jak na pierwszy projekt to wyszło b. dobrze! Moje rady odnoszą się do raczej mało znaczących błędów lub semantyki (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)