[strona] www.recenzje-filmow.net

[Pilsener]

To gotowy skrypt, więc ocenię tylko wygląd:

- grafika jest praktycznie niepocięta i za dużo waży:

https://www.recenzje-filmow.net/wp-content/...arrowcolumn.jpg - lepiej już zapisać całe tło jako jeden jpg.

- top mógłby być mniejszy, a napis ostrzejszy

- w dobrym tonie jest upchać szukajkę w topie

- czerwony nie nadaje się na tekst

- mile widziany hover w linkach i menu

- tło strony nie kontrastuje z treścią - sugeruję jaśniejsze

- tło pod treść wymaga korekt - te gradienty są zbyt chaotyczne, kreskowane linie niepotrzebne - chaos i niespójność

- bordery i separatory - widać brak koncepcji - chaos i niespójność, jak wyżej - ujednolicić: wygląd boksów, nagłówków, borderów, marginesy i paddingi

[bunitas]

czy teraz lepiej?

[Pilsener]

- top mógłby być mniejszy, a napis ostrzejszy

- dałeś top równie duży, który w dodatku waży 230 kb i w porównaniu z tamtym jest kiczowaty - zwłaszcza te napisy. Tamten top były dużo lepszy - wystarczyło tylko popracować nad napisem (z braku dobrych pomysłów najlepiej dać prosty, zwykły napis - wygładzić go ładnie i już), zrobić go np. w kolorach niebiesko-białych, całość zwęzić do 150 px

- https://www.recenzje-filmow.net/wp-content/...-sidebar-h2.jpg - te nagłówki są nieczytelne z winy tych pasków. I warto ujednolicić wszystkie nagłówki - np. zrobić dwa rodzaje. Jeden do treści, jeden do menu.

- kanał rss nie musi być tak wyeksponowany

- czerwony już nie razi i jest hover w linkach i menu - poza tym niewiele zmian. Wciąż kolumny nie są równe, po prawej pusta przestrzeń, sugeruję dać prawej kolumnie tło identyczne z lewą,

https://www.recenzje-filmow.net/wp-content/...arrowcolumn.jpg - wywalić to koniecznie (i inne graficzne tła), zainwestować w background:color w arkuszu CSS.

[bunitas]

ehh, czy tak lepiej?

co do tej pustej przestrzeni po prawej to przyznam się szczerze, że nie mam pojęcia jak to zmienić, szukałam we wszystkich plikach... :|

[bunitas]

Okay. Serwis przeszedł dużą zmianę. Co teraz sądzicie?

[Pilsener]

- strasznie wolno się wczytuje, header nie załadował mi się całkowicie, musiałem odświeżyć, w IE 6.0 za pierwszym razem header w ogóle się nie załadował

- strona wymaga bardzo dużo dżawa-skryptów

- grafika zajmuje chyba ponad 1MB, w czym ją zapisujesz? Header ponad 300 kb, a miniaturka filmu:

filmweb.pl/po/43/13/4313/7110620.3.jpg - prawie 300kb - to niedopuszczalne

- i dlaczego na innym serwerze? Spowalniamy wczytywanie się strony i uzależniamy jej działanie od kilku serwerów

- dlaczego header jest wczytywany dwa razy? Raz jako background, raz jako img?

Okodowanie też nie jest najlepsze:

- https://validator.w3.org/ - ponad 40 błędów pomimo wstecznie kompatybilnego doctype transitional

- stałe elementy laya wrzucamy jako background w css, a nie img w html

- style i skrypty umieszczamy w zewnętrznym pliku

- strona "na divach" dosłownie:

<div id="title3">Najpopularniejsze posty</div>

- używamy nagłówków h1,h2,h3, paragrafów

- <br /> a nie <br> i używamy marginesów, a nie <br>utalnych metod, aby oddzielić od siebie elementy

- kod strasznie niesemantyczny, kupa divów, a tak naprawdę wystarczyłoby kilkanaście, masa zbędnego kodu!

- w dodatku ten kod nie prowadzi do dobrego ani spójnego wyglądu, pierwszy lepszy przykład:

<div class="postbg"><p><img ALIGN="left" WIDTH="240" SRC="https://gfx.filmweb.pl/po/01/87/340187/7185496.3.jpg" HEIGHT="340" /><br />
Jedna z lepszych polskich komedii jakie ostatnio oglądałam.<br />
Gdy pierwszy raz zobaczyłam zapowiedź, stwierdziłam, że...<br />
Rewelacyjna gra aktorka Grochowskiej i Leonarda. Dobrani są idealnie, ona mądra,…</p>
</div>

- nie używamy dużych liter w kodzie HTML

- nie używamy <br /> do oddzielania akapitów czy robienia odstępów

- nie używamy zdeprecjonowanych atrybutów typu height, align - od tego jest CSS

- nie określamy rozmiarów obrazka, do zmniejszania obrazków używamy PHP, a nie HTML

- nie opuszczamy wymaganego atrybutu alt dla img'ów

- używajmy float do wkomponowania obrazka w treść

Jak to powinno wyglądać?

<div class="postbg"><p><img class="lewy" src="https://gfx.filmweb.pl/po/01/87/340187/7185496.3.jpg" alt="opis obrazka" />
Jedna z lepszych polskich komedii jakie ostatnio oglądałam.<br />Gdy pierwszy raz zobaczyłam zapowiedź, stwierdziłam, że...</p>
<p>Rewelacyjna gra aktorka Grochowskiej i Leonarda. Dobrani są idealnie, ona mądra,…</p>
</div>

Klasa w arkuszu CSS dla obrazka:

.postbg .lewy {
float: left;
margin-right: 5px;
}

- i ładnie obrazek

Niestety nie jest to najlepiej wykonana strona.

[bunitas]

ok, dzięki bardzo za konstruktywną krytykę

top schudł do 60kb.

są 2 ponieważ jedno to tło, a drugie obrazek, żeby był linkiem do strony głównej. Za słabo znam się na css'ie żeby wywalić to tło...

okay, wszystkie obrazki przerzucam na swój serwer.

co do tego jak mi poradziłeś postbg, to zrobiłam tak, ale niestety rozmiar w tym momencie jest oryginalny. jak dopisać w ccs'sie obok

.postbg .lewy {
float: left;
margin-right: 5px;
}

konkretne rozmiary? żeby za każdym razem rozmiar był taki sam naturalnie, bez wpisywania danych w 'notkach'?

- nie używamy dużych liter w kodzie HTML
- nie używamy <br /> do oddzielania akapitów czy robienia odstępów
- nie używamy zdeprecjonowanych atrybutów typu height, align - od tego jest CSS
- nie określamy rozmiarów obrazka, do zmniejszania obrazków używamy PHP, a nie HTML
- nie opuszczamy wymaganego atrybutu alt dla img'ów

Kiedy wordspress sam mi to robi :> ja nie ustawiam tego...

[Pilsener]

są 2 ponieważ jedno to tło, a drugie obrazek, żeby był linkiem do strony głównej. Za słabo znam się na css'ie żeby wywalić to tło...

#header {width:960px; height:200px; background:url(images/stripe.png) repeat-x; text-align:center; }

- jak chcesz wywalić tło to zostaw tylko width i height

Można albo dać background dla <a> (ten sposób bym polecał), albo zwykłe <a><img /></a>.

Co do rozmiarów obrazków: NIE NALEŻY ich ustawiać poprzez width i height (nieważne czy w kodzie, czy w css), ponieważ w ten sposób obrazek 1 na 1 piksel może zajmować ze 300 kb. Należy użyć gotowego skryptu do automatycznego tworzenia miniaturek, lub samemu tworzyć miniaturki w programie graficznym - w tym przypadku nie powinno być problemu, ściągamy obrazek, tworzymy z niego miniaturkę, zapisujemy w odpowiednim folderze i już. Sam tak robię na swoim blogu

Kiedy wordspress sam mi to robi :> ja nie ustawiam tego...

No tak, gotowy CMS to pewne ograniczenia niestety...

Gdy serwis się rozwinie warto pomyśleć o dedykowanym, autorskim CMS'ie zamiast wordpresa. Co prawda wordpress jest super, ale w końcu to tylko darmowy system blogowy

Co do wyglądu: sugeruję upchać wszelkie linki i reklamy po prawej stronie, a treść przesunąć jak najwyżej. Treść jest najważniejsza i powinna być wyeksponowana najbardziej. I radzę nie przesadzać z reklamami google czy systemami wymiany linków. Te "najpopularniejsze posty" itp. upchać wg klikalności - menu najwyżej, najmniej istotne linki najniżej.

Nie zaszkodzi też dokształcić się troszkę w (x)HTML+CSS, w końcu prowadzimy serwis, a nie tylko dodajemy treść.

[bunitas]

dzięki bardzo

z tłem sobie poradziłam

co do obrazów, to racja, muszę zająć się zmianą wszystkich obrazków... ale to trochę pracy będzie syzyfowej.

co do specjalnego cmsa to oczywiście, pomyślę o tym ,ale to jak serwis będzie miał oglądalność powyżej 3k uu

co do wyglądu, to czy sugerujesz, że jest za dużo reklam?

[Pawka]

Bardzo ladny szablonik.

Z silnikiem wp trzeba uwazac. Nie dawac duzo wtyczek. Dac mu dobry serwer.

Piekna stronka.