To jest dopiero mój drugi że tak powiem "projekt", więc nie oczekujcie jakichś cudów. Chodziło mi głównie o przejrzystość i wykorzystanie css-a do formatowania wyglądu(najlepiej się uczę css-a w praktyce ). Proszę o konstruktywną ocenę w skali od 1 do 10. Sprawdzałam w Firefoxie, Operze i IE-nic sie nie rozjeżdżało .

Oto adres: Strona

No to już jakiś plus

Graficznie top mi się nie podoba, kolory troche mało współgrające...

5/10 (+1 za Valid XHTML)

Całkiem dobrze. Widzę jedna kilka dziwnie wyglądających znaczników.

Pierwszy to pusty tagktóry z tego co widzę, to robi tylko odstęp... Jeśli chcesz mieć newsy w odpowiedniej odległości to nadaj po prostu odpowiedni margines dla klasy .tresc

Druga sprawa: lekki tag soup w postacinie dało się tego jako jednego znacznika zrobić?

Oczywiście to już jest czepianie się, wymaganie kompletnej dokładności :-) Kod ogólnie mi pasuje.


P.S.
Prawie wszystkim <li> nadajesz klasę .li - czy nie łatwiej byłoby właściwości tej klasy przypisać domyślnie tagowi li, a jedynie te, które mają wyglądac inaczej - oznaczać dodatową klasą?

masz tam cos wstawione na chama nie powinno tka byc (jak daje ignoruj css to ma grafike)

kod wyglada prawie ok

Wiem, te puste znaczniki miałam poprawić i zapomniałam, bo wcześniej wpisywałam sobie przykładowe treści (zanim trochę uporzadkowałam kod .
@Zajec wezmę sobie twoje uwagi do serca odnośnie tych tagów li, rzeczywiscie trochę nie przemyślałam tego.

mi sie jedna rzecz nie podoba:

div#all
{
width: 700px;
margin:auto;
background:url(yellow.jpg);
}


nie lepiej okreslic kolor dla div? (yellow.jpg to "obrazek" o wymiarach 1px na 1px)

Zastosowałam tak, ponieważ w palecie kolorów dostępnych dla web-a niestety nie było odpowiedniego koloru, gdyby takowy był, to oczywiście zrobiłabym tak, jak mówisz .

Ale poza "slownym" masz jeszcze pare innych

Podpowiem

Szesnastkowy: #D9D095
RGB: rgb(217, 208, 149) LUB rgb(85%, 82%, 58%)

Dzięki hwao! Tak to bym żyła w niewiedzy . Poprawiam te obrazki o których mówiłeś, jutro poprawie resztę kodu zdognie z waszymi sugestiami. Jeszcze raz dzięki za wszystkie sugestie.

Mi sie za to bardzo podoba. Kolory wg mnie pasuja do siebie, wszystko czytelne. W kod sie nie wglebialem ale na 1 rzut oka tez prezentuje sie bardzo przejzyscie.
Nie podobaja mi sie 2 rzeczy:
- w topie napis 'Miss Moneypenny's workshop' jest za maly.
- biale tlo.
Biorac pod uwage ze strona ta ma byc homepage lub blog daje 7/10

Jak dla mnie bardzo ok, kod niemal idealny (tego moglbym sie od Ciebie uczyc , kolorki moim zdaniem dopasowane elegancko, tylko tak jak juz to bylo wspomniane tytul w top'ie dalbym wiekszy, zeby sie rzucal w oczy.
Fajny projekt.

Nie w moim stylu, ale ok. Kod jak już poprzednicy pisali na prawdę ok, kolory hmmm tu mam więcej zastrzeżeń, ale to chyba dlatego że nie lubię tego typu stron...

Ogólnie bardzo ładnie wykonane

Pozdrawiam

nadal masz fragmenty "nie zalezne od css" a idea xhtml'a polega na tym ze zmieniajac css mozesz zmienic calkowicie wyglad strony.

[HTML] pobierz, plaintext 
<img id="note" src="1.jpg" alt="kartka">
[HTML] pobierz, plaintext 

Powinnas zastowic

[HTML] pobierz, plaintext 
<div class="imgBlock1"></div>
[HTML] pobierz, plaintext 

Teraz podpinasz do tego blocku css'a odpowiedniego (width heigh i backgrounda: url( './1.jpg' ); ) i masz ok (zrob tak ze wszytkimi poza reklama)

Troche duzo Div'ów masz a moglas ladnie uzyc innych znacznikow.

Poprawiłam kod, usunęłam zbędne znaczniki, obrazki. Teraz powinno być wszystko jak trzeba. Hwao, jeszcze raz dzięki za wskazówki .

Cały czas mnie męczy sposób, w jaki zrobiłaś to menu. Połowa znaczników jest tam nadal zbędna. Zrobiłem Ci menu o takiej samej funkcjonalności, a dwa razy krótszym kodzie:





Natomiast zamiast obecnego

[HTML] pobierz, plaintext 
<div id="menu1">
 
    <div id="menu2">
 
        <div class="linki">
 
            <p class="category">Kategoria</p>
 
                <ul class="ul">
 
                    <li><a href="index.html#">link 1</a></li>
 
                    <li><a href="index.html#">link 1</a></li>
 
                </ul>
(....)
(....)
(....)
           </div>
    </div>
[HTML] pobierz, plaintext 

dajesz tylko takie coś:

[XML] pobierz, plaintext 
<ul id="menu1">
	<li>Kategoria</li>
	<li><a href="index.html#">link 1</a></li>
	<li><a href="index.html#">link 1</a></li>
	<li>Kategoria</li>
	<li><a href="index.html#">link 1</a></li>
	<li><a href="index.html#">link 1</a></li>
	<li>Kategoria</li>
	<li><a href="index.html#">link 1</a></li>
	<li><a href="index.html#">link 1</a></li>
</ul>
[XML] pobierz, plaintext 

Layout fajny, chociaż kolory dałbym ciut bardziej nasycone.
Jak go zobaczyłem od razu z czymś mi się skojarzył --> mtv5 i taki program pimp my ride Ale to chyba przez tą czcionkę

Witam,

+ Kobieta dała do oceny projekt
+ Fajny klimatowy top, kojarzy mi się z latami '70

- Nazwa autora wpisu nie dochodzi do końca, czyli do góry.

Walidacji nie sprawdzałem, ponieważ zrobili to moi poprzednicy. Ogólnie bardzo ładnie.

Pozdrwiam!