Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: [wygląd strony] restauracja
Forum PHP.pl > Inne > Oceny
szafran86
Witam szanownych forumowiczów...

Pierwszy raz daję coś do oceny.

Dzisiaj zacząłem to robić więc jeszcze nie zająłem się czystością kodu jak poprawie to dam znać. Proszę o ocenę samego wyglądu.

Narazie jest tylko MainPage ale jak wygląd zostanie zatwierdzony to będę działał dalej.

Link do strony

Pozdro

Edit

Fotki są słabe bo nie ja je robiłem ale będą zmienione.

Ps. Nie bardzo podoba mi się ten napis na fotce z tym talerzem. Macie jakieś pomysły? Będę dźwięczny smile.gif
starach
Rozumiem że chciałeś zrobić podmieniane menu, ale przy obrazach graficznych nie robi się tego za pomocą hover'a.
Dlaczego ? Ponieważ jest spore opóźnienie zawsze i obrazek menu jest przez pewien czas niewidoczny.
Radze dać to menu przy użyciu JS albo statyczne.
Co do menu jeszcze to te obrazki mi w ogóle nie pasują są za ciemne i odstępy między nimi również nie pasują do reszty strony.
Lepiej je zmień na jednolite.
Top jest trochę za duży ale ujdzie natomiast w białe / srebrne tło w tytule topu dałbym trochę bardziej prześwitujące,
a cień w nim jest do bani i lepiej żeby go nie było na mój gust bo psuje wygląd całości.
Reszta jak dla mnie może być.
5,5 / 10
.radex
Wygląd jest ok, ale pozwolę sobie skomentować jedną rzecz odnośnie kodu:

Radzę semantyką i poprawnym kodem zajmować się od razu, bo później jest ciężko wprowadzić poprawki, albo nawet nie chce się ich wprowadzać.

Najbardziej nie podoba mi się ten fragment:

Kod
<ul id="navlist">
<li><a href="#" class="home">.</a></li>
<li><li><li><a href="#" class="history">.</a></li></li></li>
<li><li><a href="#" class="menu">.</a></li></li>
<li><li><li><a href="#" class="oysters">.</a></li></li></li>
</ul>


Brak opisu, podwójne/potrójne zamknięcia. No i też zamiast class moim zdaniem powinieneś dać id, bo element występuje jeden raz.

Kod
<ul id="navlist">
<li><a href="#" id="home">Home</a></li>
<li><a href="#" id="history">History</a></li>
<li><a href="#" id="menu">Menu</a></li>
<li><a href="#" id="oysters">About oysters</a></li>
</ul>


Aha, i jeszcze jedna rzecz:

Proponuję nie używać ditheringu w JPG-ach, bo zamiast zmniejszyć wagę, czasem ją zwiększa. Dithering ma sens tylko w takich formatach jak GIF, czy PNG.

Sorry, że wcale nie oceniałem wyglądu, tylko resztę, ale rady mogą się Tobie przydać.
szafran86
Dzięki za odpowiedź.

Wrócę z roboty to wszystko dokładnie przeanalizuje.
koderrr
Cytat(orglee @ 27.06.2008, 09:12:05 ) *
Rozumiem że chciałeś zrobić podmieniane menu, ale przy obrazach graficznych nie robi się tego za pomocą hover'a.
Dlaczego ? Ponieważ jest spore opóźnienie zawsze i obrazek menu jest przez pewien czas niewidoczny.
Radze dać to menu przy użyciu JS albo statyczne.
Co do menu jeszcze to te obrazki mi w ogóle nie pasują są za ciemne i odstępy między nimi również nie pasują do reszty strony.
Lepiej je zmień na jednolite.
Top jest trochę za duży ale ujdzie natomiast w białe / srebrne tło w tytule topu dałbym trochę bardziej prześwitujące,
a cień w nim jest do bani i lepiej żeby go nie było na mój gust bo psuje wygląd całości.
Reszta jak dla mnie może być.
5,5 / 10

gdzie sie naczytales takich pierdol ?
takie menu mozna zrobic czystym cssem i to bez uciazliwego doladowywania sie obrazkow w locie

polecam sprawdzic
http://www.ucho24.info/porady/rollover.php

co do strony
ogolny klimat jest ok
uklad tez wmiare

ale brakuje stopki
menu bym poprawil graficznie jak i technicznie (doladowywujace sie obrazki)

top zupelnie nie pasuje
netvalue
Zdjęcie w topie zajmuję 200 kilo, a jego jakość to porażka smile.gif Zrób coś z tym!
MWL
Logo i header są BEZNADZIEJNE, poza tym jest ok tongue.gif
Kildyt
Grafika do kosza. Wygląda jak strona we front page'u.

Ocena: 2/10.
cbagov
Lay w klimaty amerykanckih, czcionka za mala w P, fotki jak mowisz do wymiany, zwlaszcza ten talerz z flashem, czcionka "the Weir.." nie pasuje do "MORAN's Oyster...", gorne menu ma niewyrazne fonty i jest slabo wykonczone, w gornej czesci mozesz nieco wygasic border wokol strony bo tam nie pasuje, mozesz dodac lekkie cieniowanie wokol glownego div content. Brak stopki.
Tonacja calkiem ok.
starach
Cytat(koderrr @ 27.06.2008, 10:41:46 ) *
(...)

Nie pierdół tylko piszę jak jest. Tak się składa że czytałem CSS Według Erica Mayera
i wiem że można. Jednak prościej jest to zrobić JSem bo nie trzeba się bawić w dołączanie żadnych plików do IE.
A to co podałeś NIE MA SIĘ NIJAK DO TEGO CO NAPISAŁEM !
Więc jak już mnie pouczasz to zadaj sobie trud i przeczytaj to co napisałem dokładnie.
.radex
JS-a? CSS jest od warstwy prezentacji.

Cytat
Rozumiem że chciałeś zrobić podmieniane menu, ale przy obrazach graficznych nie robi się tego za pomocą hover'a.


Niby dlaczego? To jest najprostszy i najlepszy (pomijając problemy z IE) sposób.
mike
~orglee przykro mi ale jednak nie masz racji. Podmianę obrazków można zrobić za pomocą :hover dla linków i wcale nie ma żadnego narzutu czasu związanego z ładowaniem plików.
Nie pisz więc, że się nie da zrobić tylko co najwyżej, że Ty nie wiesz jak to zrobić.
starach
Zakładam że umiecie czytać więc może przeczytacie to co napisałem ?
Czy ja do diaska napisałem że się nie DA !? !? !?
A mój drugi post ? Link który podał koderrr to tylko ZWYKŁY HOVER tego też nie sprawdziliście a mnie się czepiacie.
edit>
A tak poza tym
To menu JUŻ JEST ZROBIONE TYM SPOSOBEM ( tylko bez dodatkowego pliku zachowawczego dla IE )
Nawet się nie pokwapiliście żeby sprawdzić tylko od razu do komentowania się zabieracie i to jeszcze na dodatek nieczytając postów tylko wyrwane z kontekstu cytaty. No kto jak kto ale myślałem że użytkownik z takim stażem na forum jak twój mike nie popełnia takich błędów.
koderrr
http://www.ucho24.info/porady/rollover.php

przeczytaj raz jeszcze
ja proponuje rozwiazanie menu przy pomocy 1(jednego) obrazka dla
dla hovera i a


strona ktora oceniamy
ma zrobione menu
na hoverze ale przy pomocy 2(dwoch) obrazkow
co powoduje w momencie najechania
nieprzyjemne ladowanie obrazka w locie
piotrooo89
ładnie sie to prezentuje... choć jak mam ocenić tylko wygląd to powiem że ten skorpion troszkę szpeci bandę... jak dla mnie nie jest to potrzebne.\

ocena 9/10
szafran86
@koderrr

Stopka będzie dziś.

Tylko zastanawiam się czy zrobić ją w głównym div-ie czy pod nim...

@netvalue

Co do fotek napisałem ze będą zmienione. Te znalazłem w necie, po zatwierdzeniu przez klienta zrobię nowe i wtedy będzie ok.

@Kildyt

Tylko ty tak myślisz....

@cbagov

Poeksperymentuje ze zmianą wielkości czcionki...

Jak wygasić border?? Chodzi mi o sposób w jaki to zrobić.

Cieniowanie jak najbardziej - zapisałem...

@koderrr

No tak jeden obrazek i tylko zmienić jego położenie....

@piotrooo89

To nie skorpion tylko Homar... naprawdę wygląda jak skorpion?questionmark.gif :/

-------------------

Poodpowiedałem to teraz czas zabrać się do roboty. Dzięki za oceny i rady. Jak coś konkretnego już będę miał to dam znać. Zależy mi na Waszym zdaniu.
cbagov
Sam border chyba takiej opcji nie ma bo to przeciez niewykonalne dla W3;) ale do rzeczy.
Na dole wyglada niezle a na gorze srednio, wiec albo mozna go wywalic albo pokombinowac.

Z kombinerek to przychodzi mi tak na szybko albo rozciagalny 1px pasek po lewej i po prawej, nawet rozciagniety na cala szerokosc BG obejmujacego calosc DIVa. To czy od dolu czy dla samego headera to juz ja ci wyjdzie po wykonaniu. A na gorze powinien zanikac, gorny sprobowalbym na ciemno, poproboj. Po sprawdzeniu moze okazac sie, ze nie gra, wiec jak dla mnie to powodowaloby skutek - usun border.
Zobaczymy co pokazesz pozniej.
Stopke rob pod divem w kolejnym.
szafran86
@cbagov

Nic mi z tego co podpowiadałeś nie wyszło :/ Za bardzo się wkurzam jak mi coś nie wychodzi. :/ A uwierz bez border wygląda jeszcze gorzej.

--------

To co zmieniłem to menu u góry i obrazek w topie.

Kod jest czysty przynajmniej według walidatora. smile.gif

Ps.
Mi się podoba biggrin.gif Mam nadzieję że klientowi również.



Sorki za post za postem ale chciałem odświeżyć smile.gif

Po kolejnej rozmowie klient stwierdził, że chce coś "bardziej nowoczesnego", więc.... zmieniłem wszystko.

Co o tym sądzicie??

link
koderrr
http://moran.szafran.xon.pl/si/specials.jpg

http://moran.szafran.xon.pl/si/specials.jpg


to mi sie nie podoba reszta ok
starach
No teraz to to wygląda jak trzeba smile.gif
Tylko wydaje mi się że efekt byłby lepszy jakby czcionka się zmniejszała po najechaniu na przycisk w menu.
9/10
.radex
Waliduje się - to plus. Ale nadal mało semantycznie - np. to:

Kod
<img src="si/specials.jpg" alt="" />


Skąd użyszkodnik przeglądarki tekstowej / boty mają się domyślać co tam jest napisane? Po to został stworzony atrybut alt="", żeby opisywać obrazki.

PS.

Kod
<p>&# 169; <b>Copyright 2008</b> - Moran's Oyster Cottage</p>


Od tego jest &copy; tongue.gif
koderrr
nie co tam pisze
tylko co jest napisane biggrin.gif



pozatym w3c tez popada troche w bezsens


np po co dodawac pusty alt=""
jesli nie chcemy go tam dawac ?
semantyczniej i lzej plikowo byloby go nie dodawac wcale
.radex
chodzi o to, że gdyby nie było trzeba dawać alt="", to wtedy wielu ludzi nie dawałoby go. A to semantycznym ciężko nazwać.
szafran86
Cytat(koderrr @ 30.06.2008, 08:42:02 ) *


Co dokładnie Ci się nie podoba.... cień, czcionka, kolory?

-----------------

@radex_p

Alty już poprawiłem a właściwie uzupełniłem.

Widzisz jeszcze jakieś błędy??
.radex
Błędów nie zauważyłem, ale wiele rzeczy można by było ulepszyć/poprawić. Ja np. zmieniłbym kodowanie na UTF-8 i używał tylko cudzysłowów (po co mieszać z apostrofami?) itd, ale w tej chwili jest już dobrze.
koderrr
poprostu mi sie nie podobaja te grafiki
i moim zdaniem nie pasuja
bim2
Akurat to jest OK. smile.gif Troche menu mi się z starym Mambo kojarzy. Nie wiem czemu, ale jakoś tak dziwnie. :] Jeśli się spodoba klientowi to wystarczy. Nie ma sensu męczyć się dla paru osób. ;P
cbagov
KLIENT to takie cos co wcale nie musi miec gustu, wiec twoj wybor to albo zrobic jak on chce, albo zrobic jak umiesz najlepiej i na odpowiednim poziomie i przekonac.
Czasem cienki gust naprawisz jesli pokazesz cos na prawde wypracowanego i jesli sluchasz klienta, jesli w czasie rozmowy zaczniesz go przekabacac, ale tu trzeba sie wykazac pewnymi zdolnosciami.
Jesli szanowny klient jednak chce miec kleksa i klopsa na stronie a ciebie to boli, to najwyzej nie podpiszesz sie w stopce winksmiley.jpg

A co do tej pokazanej nowej wersji, dla mnie jest staroswiecka i nieciekawa, takich byly tysiace. Poprzednia byla lepsza choc takze nie skonczona.
Pseudo 3D na buttonach, szare proste kreski paski, wszystko niedokonczone. W ogole pierwszy pcha sie w oczy ten szary BAR a po co. Obrazki maja kiepskie ramki a 1 ma na dodatek cien itd itd.
Nie ma co sie rozpisywac.
Zerknij tu po inspiracje: WEB DESIGN TRENDS
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.