Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

> [wygląd strony] restauracja, proszę o ocenę i rady...
szafran86
post
Post #1





Grupa: Zarejestrowani
Postów: 58
Pomógł: 0
Dołączył: 15.01.2007

Ostrzeżenie: (0%)
-----


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

Ten post edytował szafran86 27.06.2008, 01:59:30


--------------------
Go to the top of the page
+Quote Post
2 Stron V   1 2 >  
Start new topic
Odpowiedzi (1 - 19)
starach
post
Post #2





Grupa: Zarejestrowani
Postów: 999
Pomógł: 30
Dołączył: 14.01.2007
Skąd: wiesz ?

Ostrzeżenie: (0%)
-----


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
Go to the top of the page
+Quote Post
.radex
post
Post #3





Grupa: Zarejestrowani
Postów: 1 657
Pomógł: 125
Dołączył: 29.04.2006

Ostrzeżenie: (0%)
-----


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ć.


--------------------
blog | Tadam — minutnik do Pomodoro na Maka :)
Go to the top of the page
+Quote Post
szafran86
post
Post #4





Grupa: Zarejestrowani
Postów: 58
Pomógł: 0
Dołączył: 15.01.2007

Ostrzeżenie: (0%)
-----


Dzięki za odpowiedź.

Wrócę z roboty to wszystko dokładnie przeanalizuje.


--------------------
Go to the top of the page
+Quote Post
koderrr
post
Post #5





Grupa: Zarejestrowani
Postów: 255
Pomógł: 16
Dołączył: 4.07.2007

Ostrzeżenie: (0%)
-----


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
Go to the top of the page
+Quote Post
netvalue
post
Post #6





Grupa: Zarejestrowani
Postów: 199
Pomógł: 2
Dołączył: 9.06.2008

Ostrzeżenie: (0%)
-----


Zdjęcie w topie zajmuję 200 kilo, a jego jakość to porażka smile.gif Zrób coś z tym!
Go to the top of the page
+Quote Post
MWL
post
Post #7





Grupa: Zarejestrowani
Postów: 493
Pomógł: 32
Dołączył: 14.04.2008
Skąd: Lenkowski.net

Ostrzeżenie: (0%)
-----


Logo i header są BEZNADZIEJNE, poza tym jest ok tongue.gif


--------------------
Wpadaj na mój kanał o PHP. Dużo mięsa 🥩!
Go to the top of the page
+Quote Post
Kildyt
post
Post #8





Grupa: Zarejestrowani
Postów: 869
Pomógł: 53
Dołączył: 20.10.2003
Skąd: Przeworsk

Ostrzeżenie: (0%)
-----


Grafika do kosza. Wygląda jak strona we front page'u.

Ocena: 2/10.


--------------------
Go to the top of the page
+Quote Post
cbagov
post
Post #9





Grupa: Zarejestrowani
Postów: 181
Pomógł: 18
Dołączył: 19.04.2008

Ostrzeżenie: (10%)
X----


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.

Ten post edytował cbagov 27.06.2008, 10:54:41
Go to the top of the page
+Quote Post
starach
post
Post #10





Grupa: Zarejestrowani
Postów: 999
Pomógł: 30
Dołączył: 14.01.2007
Skąd: wiesz ?

Ostrzeżenie: (0%)
-----


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.
Go to the top of the page
+Quote Post
.radex
post
Post #11





Grupa: Zarejestrowani
Postów: 1 657
Pomógł: 125
Dołączył: 29.04.2006

Ostrzeżenie: (0%)
-----


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.


--------------------
blog | Tadam — minutnik do Pomodoro na Maka :)
Go to the top of the page
+Quote Post
mike
post
Post #12





Grupa: Przyjaciele php.pl
Postów: 7 494
Pomógł: 302
Dołączył: 31.03.2004

Ostrzeżenie: (0%)
-----


~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ć.
Go to the top of the page
+Quote Post
starach
post
Post #13





Grupa: Zarejestrowani
Postów: 999
Pomógł: 30
Dołączył: 14.01.2007
Skąd: wiesz ?

Ostrzeżenie: (0%)
-----


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.

Ten post edytował orglee 27.06.2008, 18:03:05
Go to the top of the page
+Quote Post
koderrr
post
Post #14





Grupa: Zarejestrowani
Postów: 255
Pomógł: 16
Dołączył: 4.07.2007

Ostrzeżenie: (0%)
-----


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
Go to the top of the page
+Quote Post
piotrooo89
post
Post #15


Newsman


Grupa: Moderatorzy
Postów: 4 005
Pomógł: 548
Dołączył: 7.04.2008
Skąd: Trzebinia/Kraków




ł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


--------------------
Go to the top of the page
+Quote Post
szafran86
post
Post #16





Grupa: Zarejestrowani
Postów: 58
Pomógł: 0
Dołączył: 15.01.2007

Ostrzeżenie: (0%)
-----


@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.


--------------------
Go to the top of the page
+Quote Post
cbagov
post
Post #17





Grupa: Zarejestrowani
Postów: 181
Pomógł: 18
Dołączył: 19.04.2008

Ostrzeżenie: (10%)
X----


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.
Go to the top of the page
+Quote Post
szafran86
post
Post #18





Grupa: Zarejestrowani
Postów: 58
Pomógł: 0
Dołączył: 15.01.2007

Ostrzeżenie: (0%)
-----


@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

Ten post edytował szafran86 30.06.2008, 04:02:00


--------------------
Go to the top of the page
+Quote Post
koderrr
post
Post #19





Grupa: Zarejestrowani
Postów: 255
Pomógł: 16
Dołączył: 4.07.2007

Ostrzeżenie: (0%)
-----


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

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


to mi sie nie podoba reszta ok
Go to the top of the page
+Quote Post
starach
post
Post #20





Grupa: Zarejestrowani
Postów: 999
Pomógł: 30
Dołączył: 14.01.2007
Skąd: wiesz ?

Ostrzeżenie: (0%)
-----


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
Go to the top of the page
+Quote Post

2 Stron V   1 2 >
Reply to this topicStart new topic
1 Użytkowników czyta ten temat (1 Gości i 0 Anonimowych użytkowników)
0 Zarejestrowanych:

 



RSS Aktualny czas: 20.08.2025 - 04:27