Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

 
Reply to this topicStart new topic
> http://kongo-ogrody.pl
andrew1985
post 14.01.2008, 10:55:34
Post #1





Grupa: Zarejestrowani
Postów: 48
Pomógł: 4
Dołączył: 28.12.2007

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


Zapraszam do oceny strony
http://kongo-ogrody.pl
Go to the top of the page
+Quote Post
grzesiek_g
post 14.01.2008, 11:17:32
Post #2





Grupa: Zarejestrowani
Postów: 249
Pomógł: 30
Dołączył: 18.07.2007

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


1. Menu do niczego, taki sam efekt uzyskasz mniejszą ilością kodu, bez javascript w css.
2. Piszesz doctype XHTML 1.0 a dajesz onclick? W XHTML inaczej wywołuje się skrypty.
3. Tekst raczej powinno umieszczać się w p, span, ale nie bezpośrednio w div. Dotyczy to m.in. prawego bloku.
4. Nie wiem jak sprawa wygląda w IE, ale widzę, że masz na górze deklarację xml, mam nadzieję, że serwujesz to tylko dla Opery i FF.
5. Galeria na tabelce w dodatu źle zbudowanej - zdjęcie i link nie mają ze sobą żadnego związku w kodzie.
6. Jak wyłączę obrazki nie zobaczę nic w zamian nagłówka.


--------------------
Warsztat: Ubuntu 12.10, PHPStorm
http://vertoo.pl
Go to the top of the page
+Quote Post
andrew1985
post 14.01.2008, 13:55:40
Post #3





Grupa: Zarejestrowani
Postów: 48
Pomógł: 4
Dołączył: 28.12.2007

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


Cytat(grzesiek_g @ 14.01.2008, 11:17:32 ) *
1. Menu do niczego, taki sam efekt uzyskasz mniejszą ilością kodu, bez javascript w css.
2. Piszesz doctype XHTML 1.0 a dajesz onclick? W XHTML inaczej wywołuje się skrypty.
3. Tekst raczej powinno umieszczać się w p, span, ale nie bezpośrednio w div. Dotyczy to m.in. prawego bloku.
4. Nie wiem jak sprawa wygląda w IE, ale widzę, że masz na górze deklarację xml, mam nadzieję, że serwujesz to tylko dla Opery i FF.
5. Galeria na tabelce w dodatu źle zbudowanej - zdjęcie i link nie mają ze sobą żadnego związku w kodzie.
6. Jak wyłączę obrazki nie zobaczę nic w zamian nagłówka.


1. Tak zrobisz to w css przy pomocy nazwa_klasy:hover ale pod IE6 to nie działa wiec pozostaje DHTML. Co więcej DHTML działa pod wszystkimi przeglądarkami.
2. XHTML obsługuje onlick czego najlepszym dowodem jest poprawna validacja kodu. Proponuje douczyć się standardów W3C. Stosowanie zdarzeń inline jest dużo szybsze niż dopinanie ich przez JS
3. Może choć nie koniecznie. Powinno się rozdzielać akapity znacznikami <p>, jak jest ich wiecej niż 1, kontakt po prawej to tylko jeden akapit więc a po co zagniezdzać jeszcze jeden znacznik w tym kontakcie ?
4. Nie, serwuje to pod wszystkie znane przeglądarki (IE6, IE7, Safari, Opera, Firefox) - i strona jest wyświetlana we wszystkich POPRAWNIE.
5. A dlaczego ma nie być na tabelce ? Tabelki służą do tabelarycznego prezentowania danych, w tym przypadku są nimi obrazki. Nie popadaj w skrajność. W dobie CSS i XHTML też korzysta się z tabelek ale nie do budowy layoutu tylko do tego, czego zostały wymyślone. Uwagi z tym, ze linki i zdjęcia nie maja ze sobą związku jakoś nie rozumiem. Możesz bliżej to przybliżyć questionmark.gif
6. Jak wyłączysz monitor to nic nie zobaczysz. A jak już komputer wyłączysz to już w ogóle.

Ten post edytował andrew1985 14.01.2008, 14:06:52
Go to the top of the page
+Quote Post
mike
post 14.01.2008, 14:06:51
Post #4





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

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


Cytat(andrew1985 @ 14.01.2008, 13:55:40 ) *
1. Tak zrobisz to w css przy pomocy nazwa_klasy:hover ale pod IE6 to nie działa wiec pozostaje DHTML. Co więcej DHTML działa pod wszystkimi przeglądarkami.
Tak się składa, że wykonanie to menu jest przykładem jak prostą rzecz można maksymalnie skomplikować.
Akurat w tym ~grzesiek_g ma rację. Dużo mniej kodu, brak JavaScript i efekt ten sam.

Cała strona jest dość ładna. Schludna i przejrzysta.
Dodałbym tylko stronicownaie w galeriach gdzie jest dużo zdjęć.
Go to the top of the page
+Quote Post
andrew1985
post 14.01.2008, 14:12:09
Post #5





Grupa: Zarejestrowani
Postów: 48
Pomógł: 4
Dołączył: 28.12.2007

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


Cytat(mike @ 14.01.2008, 14:06:51 ) *
Tak się składa, że wykonanie to menu jest przykładem jak prostą rzecz można maksymalnie skomplikować.
Akurat w tym ~grzesiek_g ma rację. Dużo mniej kodu, brak JavaScript i efekt ten sam.


Oki, zgoda. Da się to zrobić, jest to dużo łatwiejsze i przyjemniejsze ale nie działa pod Internet Explorer 6 który bądź co bądź ale jest jeszcze jedną z najpopularniejszych przeglądarek. Zróbcie hover na divi-e bez JS tak aby to DZIAŁAŁO pod IE6 to zwrócę honor.
Go to the top of the page
+Quote Post
tiraeth
post 14.01.2008, 14:16:10
Post #6





Grupa: Przyjaciele php.pl
Postów: 1 789
Pomógł: 41
Dołączył: 30.10.2003
Skąd: Wrocław

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


Cytat(andrew1985 @ 14.01.2008, 14:12:09 ) *
Oki, zgoda. Da się to zrobić, jest to dużo łatwiejsze i przyjemniejsze ale nie działa pod Internet Explorer 6 który bądź co bądź ale jest jeszcze jedną z najpopularniejszych przeglądarek. Zróbcie hover na divi-e bez JS tak aby to DZIAŁAŁO pod IE6 to zwrócę honor.


Już któryś raz z kolei użyję słowa bzdura. Tagowi a także można nadawać styl. a:hover jest respektowany przez IE6. Po prostu napisz, że nie wiesz jak to zrobić i nie gadaj, że trzeba DIVA hoverować. Baaa, menu to się w ul/li albo dt powinno robić smile.gif A styluje się sam link, wtedy można uzyskać potrzebny efekt.
Go to the top of the page
+Quote Post
mike
post 14.01.2008, 14:17:10
Post #7





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

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


Cytat(andrew1985 @ 14.01.2008, 14:12:09 ) *
Oki, zgoda. Da się to zrobić, jest to dużo łatwiejsze i przyjemniejsze ale nie działa pod Internet Explorer 6 który bądź co bądź ale jest jeszcze jedną z najpopularniejszych przeglądarek. Zróbcie hover na divi-e bez JS tak aby to DZIAŁAŁO pod IE6 to zwrócę honor.
A co tak się czepiłeś tych divów. To menu powinno wyglądać tak:
  1. <ul>
  2. <li><a/></li>
  3. <li><a/></li>
  4. <li><a/></li>
  5. <li><a/></li>
  6. </ul>
i będzie wyglądąło i działało identycznie jak Twoje, również pod każdą przeglądarką.
Go to the top of the page
+Quote Post
Cezar708
post 14.01.2008, 14:26:46
Post #8





Grupa: Zarejestrowani
Postów: 1 116
Pomógł: 119
Dołączył: 10.05.2005
Skąd: Poznań

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


Stronka wg mnie bardzo ładna. I mnie osobiście zachęciłaby do współpracy z Wami (jak będę miał dom to zrobicie mi ogród)

Jeśli chodzi o menu, również muszę zgodzić się z przedmówcami. Bo Twoje rozwiązanie jest niepotrzebnie za ciężkie. Proponuję abyś zaadoptował techniki z tego menu. Tam jest dość schludnie pokazane jak używać css do wykonania menu.
Go to the top of the page
+Quote Post
mike
post 14.01.2008, 14:29:25
Post #9





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

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


Cytat(andrew1985 @ 14.01.2008, 14:25:24 ) *
P.S. Nie pisz ze bzdura bo nie podałeś rozwiązania jak zrobić działający hover na divie pod IE6 tylko obeszłeś problem omijając go. Klient chce mieć takie przyciski to trzeba mu je zrobić a nie proponować łatwiejsze, zupełnie inne rozwiązanie.
Czytasz to co się do Ciebie pisze?
Twoje menu jest błędnie skonstruowane ponieważ jest oparte o <div> a powinno byc listą <ul> z elementami <li> a nich osadzone <a>. Koniec! Trzy znaczniki, gdzie :hover jest dla <a>. Wszystko będzie działać identycznie jak To co masz w tej chwili.
Przykłady: cssplay.co.uk :: menus - setki różnych rozwiązań.
Go to the top of the page
+Quote Post
andrew1985
post 14.01.2008, 14:34:11
Post #10





Grupa: Zarejestrowani
Postów: 48
Pomógł: 4
Dołączył: 28.12.2007

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


Cytat(mike @ 14.01.2008, 14:17:10 ) *
A co tak się czepiłeś tych divów. To menu powinno wyglądać tak:
  1. <ul>
  2. <li><a/></li>
  3. <li><a/></li>
  4. <li><a/></li>
  5. <li><a/></li>
  6. </ul>
i będzie wyglądąło i działało identycznie jak Twoje, również pod każdą przeglądarką.


Jesteś pewien questionmark.gif Na moje twój kod w przeglądarce będzie wyglądał tak:
  • link
  • link
  • link
  • link
  • link
Czy to wygląda i działa tak samo ? Nie sądze. Po pierwsze jak nie klikne w linka nie otworzy mi sie strona, po drugie jak nie najade myszka na linka to nie zmieni mi sie jego styl, po trzecie jak najade myszka na linka i obok niego to nie zmieni mi sie tło na okreslonym obsarze wokół linka.

Za chwile ktoś napisze ze to tego podanego w przykładzie html trzeba dopisać css. No właśnie ale go autor przykładu nie napisał. Wiec wygląda to tak jak podałem w przykładzie wyżej, czyli wypunktowana lista.
Go to the top of the page
+Quote Post
Cezar708
post 14.01.2008, 14:37:15
Post #11





Grupa: Zarejestrowani
Postów: 1 116
Pomógł: 119
Dołączył: 10.05.2005
Skąd: Poznań

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


Cytat(andrew1985 @ 14.01.2008, 14:34:11 ) *
Jesteś pewien questionmark.gif Na moje twój kod w przeglądarce będzie wyglądał tak:
  • link
  • link
  • link
  • link
  • link
Czy to wygląda i działa tak samo ?


echh.. przeczytaj linka który Ci wcześniej podałeś. Za pomocą CSS możesz zmienić styl <lu><li>...

PS: Radzę nie podawać na stronie maila. Lepiej stwórz formatkę z możliwością wysłania do Ciebie maila. Roboty-spamery znajdą w końcu Twojego maila i będą Ci słali chłam na pocztę.
Go to the top of the page
+Quote Post
andrew1985
post 14.01.2008, 14:37:38
Post #12





Grupa: Zarejestrowani
Postów: 48
Pomógł: 4
Dołączył: 28.12.2007

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


Cytat(mike @ 14.01.2008, 14:29:25 ) *
Przykłady: cssplay.co.uk :: menus - setki różnych rozwiązań.


Dzięki za linka. Podpatrze te rozwiązania.
Co do <ul><li> to wykorzystam to przy okazju budowania kolejnej strony.

Ten post edytował andrew1985 14.01.2008, 14:45:26
Go to the top of the page
+Quote Post
grzesiek_g
post 14.01.2008, 14:46:21
Post #13





Grupa: Zarejestrowani
Postów: 249
Pomógł: 30
Dołączył: 18.07.2007

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


Cytat(andrew1985 @ 14.01.2008, 13:55:40 ) *
1. Tak zrobisz to w css przy pomocy nazwa_klasy:hover ale pod IE6 to nie działa wiec pozostaje DHTML. Co więcej DHTML działa pod wszystkimi przeglądarkami.

Działa, ja nigdy nie miałem trudności z tym, by menu było poprawne, bez js i do tego obszar klikalny znacznika a był większy niż sam tekst, masa przykładów:http://www.cssplay.co.uk/menus/
Cytat(andrew1985 @ 14.01.2008, 13:55:40 ) *
2. XHTML obsługuje onlick czego najlepszym dowodem jest poprawna validacja kodu. Proponuje douczyć się standardów W3C. Stosowanie zdarzeń inline jest dużo szybsze niż dopinanie ich przez JS

I walidator jest wyznacznikiem standardów? To nie jest jakaś tam wyrocznia tylko proste, mało inteligentne narzędzie które sprawdza poprawność kodu pod względem składniowym, powiedz mi, jeśli stronę zbudujesz jedynie na samych div to czy się zwaliduje? Proponuję nie cieszenie się tak z powodu, że strona przechodzi walidację.
Cytat(andrew1985 @ 14.01.2008, 13:55:40 ) *
5. A dlaczego ma nie być na tabelce ? Tabelki służą do tabelarycznego prezentowania danych, w tym przypadku są nimi obrazki. Nie popadaj w skrajność. W dobie CSS i XHTML też korzysta się z tabelek ale nie do budowy layoutu tylko do tego, czego zostały wymyślone. Uwagi z tym, ze linki i zdjęcia nie maja ze sobą związku jakoś nie rozumiem. Możesz bliżej to przybliżyć questionmark.gif

Chociażby dlatego, że cierpi na tym dostępność, weźmy na to osobę słabo widzącą która korzysta ze screenreadera.
Cytat(andrew1985 @ 14.01.2008, 13:55:40 ) *
6. Jak wyłączysz monitor to nic nie zobaczysz. A jak już komputer wyłączysz to już w ogóle.

Proponuję połączyć się z netem przez modem komórki, ja łącze się normalnie, ale jakbym tak robił to po pierwsze wyłączyłbym obrazki, i włączał je tylko w razie potrzeby.

Edit: nie zauważyłem że mike podałeś ten sam link do przykładów menu co ja:)

Ten post edytował grzesiek_g 14.01.2008, 14:48:19


--------------------
Warsztat: Ubuntu 12.10, PHPStorm
http://vertoo.pl
Go to the top of the page
+Quote Post
andrew1985
post 14.01.2008, 14:55:53
Post #14





Grupa: Zarejestrowani
Postów: 48
Pomógł: 4
Dołączył: 28.12.2007

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


Co do menu to mnie przekonaliście.
Ale co do onlick to nie zmienie zdania bo przy okazji robienia innego projektu, początkowo dołanczanie wszystkich akcji odbywało się za pomocą JS. W XHTML nie było żadnego onlick, onmouseout itp. Trochę tych podpięć było. Po zmianie na wszystkich akcji na inline czyli DHTML-wym sposobem strona ładowała się średnio o 0,5-1 sekundy szybciej exclamation.gif
Dopinanie akcji przez JS może i jest eleganckie ale nie jest za szybkie. A osoby o słabym procku szczególnie to zauważą.
Go to the top of the page
+Quote Post
grzesiek_g
post 14.01.2008, 15:05:24
Post #15





Grupa: Zarejestrowani
Postów: 249
Pomógł: 30
Dołączył: 18.07.2007

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


Cytat(andrew1985 @ 14.01.2008, 14:55:53 ) *
A osoby o słabym procku szczególnie to zauważą.

Więc proponuję dodać do 100 elementów onclick. Jak dużo czasu stracisz, jak dużo kodu będziesz musiał dodatkowo wysłać?


--------------------
Warsztat: Ubuntu 12.10, PHPStorm
http://vertoo.pl
Go to the top of the page
+Quote Post
andrew1985
post 14.01.2008, 15:19:19
Post #16





Grupa: Zarejestrowani
Postów: 48
Pomógł: 4
Dołączył: 28.12.2007

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


Cytat(grzesiek_g @ 14.01.2008, 15:05:24 ) *
Więc proponuję dodać do 100 elementów onclick. Jak dużo czasu stracisz, jak dużo kodu będziesz musiał dodatkowo wysłać?

Właśnie przy 100 elegantach zaczyna się zauważać wyrazna różnicę. Ba wystarczy juz 20 a strona po załadowaniu naprawdę muli. Szczególnie pod FF i IE gdzie szybkość JS pozostawia wiele do życzenia. Dodasz 100 razy onlick do kodu i na pewno te kilka klio szybciej sie załaduję niż JS zdąży przelecieć cały kod i podpinać akcje. Pisze to z doświadczenia bo jak pisałem wcześniej, przy okazji innego większego projektu, na początku wszystko ładnie było dopinane przez JS ale po zmianie na DHTML strona naprawdę zyskała na wydajności.

Co nie ukrywa faktu ze chciało by sie pisać to elegancko za pomocą JS.

P.S. Jak się robi strony generowane dynamicznie to onlick pisze tylko raz. 100 razy powtorzy mi to parser danego jezyka za pomocą pętli.

Ten post edytował andrew1985 14.01.2008, 15:22:01
Go to the top of the page
+Quote Post
grzesiek_g
post 14.01.2008, 15:27:01
Post #17





Grupa: Zarejestrowani
Postów: 249
Pomógł: 30
Dołączył: 18.07.2007

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


OK. Kończę tą dyskusję bo widzę, że nie ma ona żadnego skutku. DHTML, kiedy to już było?


--------------------
Warsztat: Ubuntu 12.10, PHPStorm
http://vertoo.pl
Go to the top of the page
+Quote Post
tiraeth
post 14.01.2008, 17:40:52
Post #18





Grupa: Przyjaciele php.pl
Postów: 1 789
Pomógł: 41
Dołączył: 30.10.2003
Skąd: Wrocław

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


Cytat
Co do menu to mnie przekonaliście.

Więc? Kto pisze bzdury?

Te obrazki w tabelkach to już draka na maksa. Gdybyś ładnie wyfloatował divy, mógłbyś otrzymać taki sam efekt i mniej kodu do wysłania, A jak ktoś wyłączy styl CSS to zdecydowanie łatwiej będzie mu przeglądać to na komórce, palmtopie czy gdziekolwiek indziej.

Ostatnia sprawa. Używasz prototype (tak wynika z tego, że ładujesz do go strony). Czemu ograniczasz go tylko do Lightboxa? Po co takie wygibasy jak getElementsByTagName?

Wyłaczyłem CSS przy oglądaniu tej strony i muszę powiedzieć, że gdybyś odpowiednio używał znaczników i odpowiednio rozmieszczał elementy, to w rezultacie powinienem dostać czysty XHTML, z samym tekstem - gdzieniegdzie z jakimiś ikonkami. Pomijam już fakt sztywnego ustawiania kolorów dla tego menu.

Cytat
(...) ale po zmianie na DHTML strona naprawdę zyskała na wydajności.

Masz wyniki benchmark?
Go to the top of the page
+Quote Post
.radex
post 15.01.2008, 20:40:59
Post #19





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

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


ehh...

Kod
#header li{
 display: inline;    
}
#header li a{
 display: block;
 float: left;

/* .... */
}


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

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 Wersja Lo-Fi Aktualny czas: 14.08.2025 - 04:40