![]() |
![]() |
![]()
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 ![]() Ten post edytował szafran86 27.06.2008, 01:59:30 -------------------- ![]() |
|
|
![]() |
![]()
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 |
|
|
![]()
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ć. -------------------- |
|
|
![]()
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. -------------------- ![]() |
|
|
![]()
Post
#5
|
|
Grupa: Zarejestrowani Postów: 255 Pomógł: 16 Dołączył: 4.07.2007 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 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 |
|
|
![]()
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
![]() |
|
|
![]()
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
![]() -------------------- Wpadaj na mój kanał o PHP. Dużo mięsa 🥩!
|
|
|
![]()
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. -------------------- |
|
|
![]()
Post
#9
|
|
Grupa: Zarejestrowani Postów: 181 Pomógł: 18 Dołączył: 19.04.2008 Ostrzeżenie: (10%) ![]() ![]() |
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 |
|
|
![]()
Post
#10
|
|
![]() Grupa: Zarejestrowani Postów: 999 Pomógł: 30 Dołączył: 14.01.2007 Skąd: wiesz ? Ostrzeżenie: (0%) ![]() ![]() |
(...) 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. |
|
|
![]()
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. -------------------- |
|
|
![]()
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ć. |
|
|
![]()
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 |
|
|
![]()
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 |
|
|
![]()
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 -------------------- |
|
|
![]()
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? ![]() ------------------- 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. -------------------- ![]() |
|
|
![]()
Post
#17
|
|
Grupa: Zarejestrowani Postów: 181 Pomógł: 18 Dołączył: 19.04.2008 Ostrzeżenie: (10%) ![]() ![]() |
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. |
|
|
![]()
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. ![]() Ps. Mi się podoba ![]() Sorki za post za postem ale chciałem odświeżyć ![]() 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 -------------------- ![]() |
|
|
![]()
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 |
|
|
![]()
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
![]() Tylko wydaje mi się że efekt byłby lepszy jakby czcionka się zmniejszała po najechaniu na przycisk w menu. 9/10 |
|
|
![]() ![]() |
![]() |
Aktualny czas: 20.08.2025 - 04:27 |