Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

> [www] Strona szkoły
krzysiek_raven
post
Post #1





Grupa: Zarejestrowani
Postów: 86
Pomógł: 0
Dołączył: 28.09.2007

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


Witam,

Chcę się dowiedzieć jak oceniacie wykonaną przeze mnie stronę. Stronka jest już gotowa i zaakceptowana, i nie bedzie już zmieniana w najbliższym czasie, interesuje mnie tylko wasza opinia.
Adres: oxford.home.pl
Go to the top of the page
+Quote Post
 
Start new topic
Odpowiedzi (1 - 13)
scanner
post
Post #2





Grupa: Zarząd
Postów: 3 503
Pomógł: 28
Dołączył: 17.10.2002
Skąd: Wrocław




Ogólnie całość przypomina mi tablice ogłoszeń z przypiętymi pinezkami kawałkami kartek - ogromne menu a'la lata '90 wisi sobie po lewej, pośrodku słabo sie komponujące z resztą "body" a nad tym ogromny baner "Witaj szkoło". NIc szczególnego - powiedziałbym, że to dolne stany średnie.
Go to the top of the page
+Quote Post
JoShiMa
post
Post #3





Grupa: Zarejestrowani
Postów: 1 374
Pomógł: 149
Dołączył: 1.03.2006

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


Wyglądem nie powala na kolana, ale ja się na tym nie znam, więc nie doradzę.

Co do kodu:
Czemu to miało służyć:
  1. <div class="topBG">
  2. <div class="banner"></div>
  3. </div>

i dlaczego to nie jest zwykły img tylko dwa puste divy?

Menu się robi raczej na listach <ul> U ciebie to powinny być listy zagnieżdżone.

Coś jest nie tak z numerowaniem nagłówków. Najpierw w kodzie jest <h3/> a dopiero potem <h1/> i <h2/> Nie jestem pewna, ale chyba nie powinno tak być. Jest to chyba wina bezsensownej konstrukcji samego menu.


W jakim celu zastosowałeś kolejny pusty div?
  1. <div class="menuEnd"></div>



Takie konstrukcje też niemają większego sensu:
  1. <p class="artLinks">
  2. <a href="index.php?option=artDetails&amp;nID=3">-> Przeczytaj całóść</a>
  3. </p>
  4.  
  5. <p class="dateTime">2009-08-11 17:33:33</p>
  6.  
  7. <div class="artHeader">
  8. <h2>Nowa strona internetowa</h2>
  9. </div>



W jakim celu (IMO bez sensu) umieściłeś zarówno top jak i stopkę w podwójnym bloku div?

Sporo temu kodowi brakuje żeby stał się zgodny ze specyfikacją (IMG:style_emoticons/default/sad.gif) Dużo jeszcze nauki przed Tobą.
Go to the top of the page
+Quote Post
krzysiek_raven
post
Post #4





Grupa: Zarejestrowani
Postów: 86
Pomógł: 0
Dołączył: 28.09.2007

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


W topie i stopce są podwójne divy bo pierwszy jest rozciągnięty na całą szerokość ekranu i na nim znajduje się gradient ustawiony jako tło, drugi div jest wyśrodkowany o szerokości 990px i na nim jest banner. Podobnie jest ze stopką i środkową częścią strony. Dzieki temu można ustawić różne tła dla topu, środka i stopki. Wpadłem akurat na taki pomysł i dlatego to tak wygląda w kodzie;) W pustych divach są zwykle obrazki. Ten z klasą menuEnd ma ustawiony obrazekz dolnym zaokrągleniem do menu. Jakoś sie tak przyzwyczaiłem że obrazki ustawiam sobie w CSS jako tło i rzadko stosuje <img> (IMG:style_emoticons/default/tongue.gif)

Wiem ze grafika nie powala ale myśle ze jest w miare przejrzysta.

Dzięki za opinie.
Go to the top of the page
+Quote Post
JoShiMa
post
Post #5





Grupa: Zarejestrowani
Postów: 1 374
Pomógł: 149
Dołączył: 1.03.2006

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


Cytat(krzysiek_raven @ 19.08.2009, 18:48:51 ) *
W topie i stopce są podwójne divy bo pierwszy jest rozciągnięty na całą szerokość ekranu i na nim znajduje się gradient ustawiony jako tło, drugi div jest wyśrodkowany o szerokości 990px i na nim jest banner. Podobnie jest ze stopką i środkową częścią strony.

Jeśli tak to zamiast drugiego DIVa z banerem ustawionym jako tło powinieneś mieć poprostu IMG i to w dwóch kawałkach, żeby logo było "klilkalne".



Cytat(krzysiek_raven @ 19.08.2009, 18:48:51 ) *
W pustych divach są zwykle obrazki. Ten z klasą menuEnd ma ustawiony obrazekz dolnym zaokrągleniem do menu. Jakoś sie tak przyzwyczaiłem że obrazki ustawiam sobie w CSS jako tło i rzadko stosuje <img> (IMG:style_emoticons/default/tongue.gif)

To się odzwyczaj. Do wstawiania obrazków służy, według specyfikacji HTMLa znacznik IMG a nie DIV. Tak jak Ty robisz to tylko wtedy, gdy na obrazku mają być jakieś inne elementy nałożone.

Go to the top of the page
+Quote Post
tsharek
post
Post #6





Grupa: Zarejestrowani
Postów: 300
Pomógł: 1
Dołączył: 22.09.2003
Skąd: Czeladź

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


Cytat(JoShiMa @ 19.08.2009, 21:35:50 ) *
To się odzwyczaj. Do wstawiania obrazków służy, według specyfikacji HTMLa znacznik IMG a nie DIV. Tak jak Ty robisz to tylko wtedy, gdy na obrazku mają być jakieś inne elementy nałożone.


Nie zgodzę się z Tobą - zależy jaki obrazek. Jeżeli chodzi o różnego rodzaju tła bądź inne elementy dekoracyjne - to jak najbardziej IMO powinno być to zrobione jak to tutaj jest (pustego DIVu czasami nie da się uniknąć, albo problematyczne jest aby go wyrzucić). Z jednym masz racje - Logo powinno być w IMG i być klikalne, ale jak wspomniałem to nie jest TŁO lub jakiś tam element dekoracyjny. Tak samo obrazki w contencie (zdjęcia) powinny być na IMG. Footer zrobił jak najbardziej poprawnie
Go to the top of the page
+Quote Post
JoShiMa
post
Post #7





Grupa: Zarejestrowani
Postów: 1 374
Pomógł: 149
Dołączył: 1.03.2006

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


Cytat(tsharek @ 21.08.2009, 15:04:49 ) *
IMO powinno być

Tu nie ma IMO, tylko specyfikacja i standardy (IMG:style_emoticons/default/tongue.gif)
Go to the top of the page
+Quote Post
vokiel
post
Post #8





Grupa: Zarejestrowani
Postów: 2 592
Pomógł: 445
Dołączył: 12.03.2007

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


Obrazek jako obrazek - IMG
Obrazek jako tło - CSS, czyli background dla odpowiedniego elementu, jeśli to jest struktura strony, to dla elementu blokowego, np.: div
Go to the top of the page
+Quote Post
Pawel_W
post
Post #9





Grupa: Zarejestrowani
Postów: 1 675
Pomógł: 286
Dołączył: 15.06.2009
Skąd: Wieliczka

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


a posprawdzajcie jakieś darmowe szablony, typu styleshout.com, tam jest wszystko na CSS (IMG:style_emoticons/default/winksmiley.jpg)

Wyboraź sobie że masz stronę na obrazkach, ktoś ma wolne łącze, wchodzi na stronę i zaczyna czytać tekst, który w trakcie ładowania obrazków np. menu zjeżdża mu coraz bardziej w dół

a teraz druga opcja, wchodzi na stronę, gdzie wszystko jest już na swoim miejscu, divy powoli wypełniają się tłem i nic nie lata


jak myślisz, którą opcję woli internauta?
Go to the top of the page
+Quote Post
phpion
post
Post #10





Grupa: Moderatorzy
Postów: 6 072
Pomógł: 861
Dołączył: 10.12.2003
Skąd: Dąbrowa Górnicza




Cytat(Pawel_W @ 23.08.2009, 11:46:34 ) *
Wyboraź sobie że masz stronę na obrazkach, ktoś ma wolne łącze, wchodzi na stronę i zaczyna czytać tekst, który w trakcie ładowania obrazków np. menu zjeżdża mu coraz bardziej w dół

Wystarczy stosować atrybuty width i height dla znacznika img i po sprawie.
Go to the top of the page
+Quote Post
Quantum
post
Post #11





Grupa: Zarejestrowani
Postów: 450
Pomógł: 84
Dołączył: 27.11.2008
Skąd: Warszawa

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


jeśli chodzi o sam wygląd to ode mnie wielki plus za estetyczność, strona jest przejrzysta, łatwo można znaleźć to czego się szuka (IMG:style_emoticons/default/smile.gif) 8/10
Go to the top of the page
+Quote Post
drake88
post
Post #12





Grupa: Zarejestrowani
Postów: 283
Pomógł: 15
Dołączył: 22.03.2009
Skąd: Bytów

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


Cellpiki takie sobie, background słabiutki, stopka do niczego totalnie. 4,55/10,00
Go to the top of the page
+Quote Post
JoShiMa
post
Post #13





Grupa: Zarejestrowani
Postów: 1 374
Pomógł: 149
Dołączył: 1.03.2006

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


Cytat(Pawel_W @ 23.08.2009, 11:46:34 ) *
a posprawdzajcie jakieś darmowe szablony, typu styleshout.com, tam jest wszystko na CSS (IMG:style_emoticons/default/winksmiley.jpg)

I co to według Ciebie oznacza, że to są standardy?

Cytat(Pawel_W @ 23.08.2009, 11:46:34 ) *
Wyboraź sobie że masz stronę na obrazkach, ktoś ma wolne łącze, wchodzi na stronę i zaczyna czytać tekst, który w trakcie ładowania obrazków np. menu zjeżdża mu coraz bardziej w dół

Wyobraź sobie niewidomego, któremu odpowiednie urządzenie czyta stronę złożoną w dużej części z pustych divów. Potrafisz sobie wyobrazić jak go krew zalewa?
Go to the top of the page
+Quote Post
f1xer
post
Post #14





Grupa: Zarejestrowani
Postów: 403
Pomógł: 68
Dołączył: 20.03.2008

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


  1. <h1>
  2. <a title="Tutaj opis naszej strony!" href="#">
  3. <span>Tak to jest dla oprogramowaia czytającego tekst</span>
  4. </a>
  5. </h1>


do tego a - display:block i background obrazek z tłem, span display:none;. Moim zdaniem wilk syty i owca cała, aczkolwiek logo to logo i fajnie by było aby było w img, co do tła dla menuEnd to moim zdaniem poprawne rozwiązanie. Gdyby zastosował background dla powiedzmy obrazka w newsie no to to by było niedobre.
Go to the top of the page
+Quote Post

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

 



RSS Aktualny czas: 22.08.2025 - 16:22