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

Proste ale działa

Lay zbyt prosty by coś oceniać.

Jak napisał frickle, proste ale działa Nagłówki (Software,hardware) jakoś mógłbyś zmienić. Tło jakieś może. Napis R E K L A M A nie pasuje no i tło lewego i prawego menu mógłby być trochę jaśniejszy.

jak na pierwsze cięcie to jest elegancko co do laya - to jak poprzednicy - prosty (nie specjalnie mi się podoba) ale na tle tego co widywałem to jest nieźle

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

W IE prawy blok spada na dół.

Ano: Wielkie dzięki za te wszystkie rady, napewno da mi to do myślenia ;]

Babejsza: Pod jakim IE ? mi pod 7 i 6 wyświetla prawidłowo
Ale to że IE jest zuuue to chyba każdy wie ? i nie powinien go używać.

Pozdro!

sprawdź jeszcze raz pod IE 6.0 a to ze jest złe to nie zmienia faktu ze trzeba dbać także o userów tej przeglądarki

Na IE6 się rozjeżdża, pod IE7 jest ok.

Najbardziej z tej strony podoba mi się top
Reszta też raczej w porządku, ale czegoś mi tu brakuje :/

1. Pomiędzy divami nie rób br'ów rób to za pomocą marginesów.
2. IP jest zbędne ;P
3. Piszesz w xHTML więc zamiast:

[HTML] pobierz, plaintext 
<iframe src='http://blok.radiosfera.pl/blok.php?n=5' frameborder='0' scrolling='no' 
   width='120' height='200'></iframe>
[HTML] pobierz, plaintext 

Napisz:

[HTML] pobierz, plaintext 
<iframe src="http://blok.radiosfera.pl/blok.php?n=5" frameborder="0" scrolling="no"
   width="120" height="200"></iframe>
[HTML] pobierz, plaintext 

Ocena: jak na pierwszą strone na divach to 5/10

Jeszcze jedna rada: przejdź na xhtml 1.0 strict, a stronę wysyłaj jako application/xhtml+xml, zobacz tutaj: http://doktorno.boo.pl/content_negotiation.php <chodzi o ten skrypt w php