Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: [ www ] design oraz cięcie
Forum PHP.pl > Inne > Oceny
kacper_
Witam, proszę o ocenę projektu graficznego oraz kodu serwisu z anonsami erotycznymi:

anonse erotyczne

Wszelkie uwagi mile widziane winksmiley.jpg
Moli
Kod
                        <h3>monia83</h3>
                        <h4>poznań</h4>

                        <h5>skomentowane przez</h5>
                        <h3>dlugi21</h3>
                        <h4>wrocław</h4>

Kod
<h1><img src="img/headers/reklama.jpg" alt="" /></h1>


Chyba nie rozumiesz o co chodzi w naglówkach smile.gif

Dużo masz obrazków typ:
Kod
<img src="img/leftColumn/header/left.jpg" alt="" />

Nie dało tego w css wsadzić ?
bigZbig
Na pierwszy rzut ok mi się podoba. Jest estetyczny, nie wulgarny, czytelny. Jedną rzecz bym zmienił. Zrobił bym przejście gradientem z fioletu do beżu.
kacper_
Cytat(Moli @ 25.02.2009, 19:27:25 ) *
Kod
<h1><img src="img/headers/reklama.jpg" alt="" /></h1>


Chyba nie rozumiesz o co chodzi w naglówkach smile.gif

Dużo masz obrazków typ:
Kod
<img src="img/leftColumn/header/left.jpg" alt="" />

Nie dało tego w css wsadzić ?


Co do tego nagłówka to w tym divie znajdują się dwa obrazki, zamiast stylować po klasach lepszy pomysłem wydawało mi się stylować po znacznikach w sposób 


Kod
.leftColumHeader h1 img { ... }


zamiast

Kod
.lefColumnHeader .headerImg { ... }


nie lubię używac klas jeśli można to obejść, a tu wydawało mi się to dobrym pomysłem, oczywiście zamiast h1 mogłem dać div, nie byłoby wtedy szumu pewnie winksmiley.jpg

Co do obrazków, to po co ładować w css w puste divy? skoro znacznik <img> służy do obraków, to go tak stosuje, źle? Wiem, że mogłem ten obrazek dać jako tło dla tego diva a drugi wtedy z drugiego zniknąłby problem z <h1> winksmiley.jpg cenna uwaga. Problemem było wtedy nadanie kilku klas bo te gwiazdki przy nagłówkach mają różne kolory, ale chyba to lepsze nawet się wydaje
Marr
menu główne ma tło korzystające z pseudoklasy hover, a w niej podmieniasz tlo na inne - bez sensu, poniewaz na wolniejszych laczach, zamulonym internecie lub nawet na normalnym przez chwile widac kolor tla bez tego obrazka - użyj lepiej obrazka składającego się z dwóch i przesuwaj położenie tła w nim - unikniesz w ten sposób przeładowania grafiki - zamiast tego przesunie się smile.gif
kacper_
Fajny trick, nie wpadłbym na to. Dzięki winksmiley.jpg , już poprawione. Z racji tego, że tło pod menu się powtarza musiałem przesuwać je pionowo. Nie wiem jak w starszych przeglądarkach jest interpretowana ujemna pozycja background. Mam tak:

Kod
background: url( ... ) 0 -69px repeat-x;


Bo z tego co pamiętam starsze IE albo opery nie interpretują ujemnych marginesów, nie wiem jak z pozycjami jest ?
Marr
to nie zaczynaj od ujemnej smile.gif zacznij od 0 a później przesuń sobie o wysokość przycisku
kacper_
Zaczynam od 0. Ten kod powyżej jest dla hover.

Kod
#top a {
 display: block;
 float: left;
 background: url( ../img/menu/menuBg.jpg ) repeat-x;
}
#top a:hover {
 background-position: 0 -69px;
}


Nie bardzo wiem jak można obejść ujemny margines
Marr
hmm w sumie tak.... w prawdzie ja tam żadnym specem nie jestem, może ktoś inny wpadnie na coś mądrego ale możesz jeszcze popróbować z np: top lub bottom - ale tak jak mówie, to tylko takie strzały - wg. mnie ujemne marginesy są ok (przynajmniej dla w3c) a jak jest z przeglądarkami to nie powiem - trzeba by było sprawdzić... empirycznie i paradoksalnie biggrin.gif
Moli
Tło do obrazków po to aby kod html odchudzić, po to jest css. A co do nagłówków, beznadziejnym pomysłem jest robienie ciągle hX czy nawet divów bo nie chce używać Ci się klas.
nexis
Wygląda dobrze na większości przeglądarek
To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.
Invision Power Board © 2001-2025 Invision Power Services, Inc.