Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

 
Reply to this topicStart new topic
> [portal] Monstrum - portal o grach MMORPG, www.monstrum.com.pl
Kildyt
post 31.08.2008, 17:49:20
Post #1





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

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


Cześć!

Wszystkie opinie mile widziane. winksmiley.jpg
Adres strony: www.monstrum.com.pl

Strona już oficjalnie ruszyła. smile.gif


--------------------
Go to the top of the page
+Quote Post
webber
post 1.09.2008, 09:49:10
Post #2





Grupa: Zarejestrowani
Postów: 107
Pomógł: 19
Dołączył: 30.10.2007

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


O prawdopodobnie będę pierwszy smile.gif

Zacznę od design`u. Od góry...

Header - Prosty, background taki troszkę abstract, ładnie to wygląda.

Menu również ładne. Jadę dalej...
Napis - "O czym jest ta strona?" Nie wiadomo dlaczego robiony jako img, zrobiłbym to normalnym textem... ale to moja opinia.
Widzę boxa gdzie jest "Recenzja gry Wow", polecałbym zrobić tak, że gdy najadę na obrazek background również się zmieni, a nie tylko gdy najadę na text "Recenzja gry WoW". To samo z box`ami pod spodem. Aktualności troszkę mało wyeksponowane, przydałaby się jakaś miniaturka przy news`ie etc.
Next... Czemu "Ostatnie na forum" nie jest w box`ie?

Stopka mi się nie podoba, polecam:
http://www.smashingmagazine.com/2007/01/09...dern-solutions/

Wg mnie zbyt rażący gradient przy tekście "Jesteś nowy[...]"

Lista gier, oraz statystyki - w sumie zadnych zastrzeżeń.

Masz troszkę dziwny układ menu...
Strona główna, Aktualności, LOGOWANIE? Nie pomyślałeś że powinno być w miejscu bardziej hmm... widocznym? Albo od razu gdzieś formularz zamieszczony? Możesz użyć rozwijanego logowania coś a`la wykop.pl

Ogólnie rzecz biorąc - typografia leży.

Ale macie plusa za oryginalną kolorystykę aaevil.gif

PS: Jesteście krytykami w dziedzinie gier MMORPG? Najlepsza ocena 3/10 WoW biggrin.gif

Uhh... przejdźmy do menu... masakra tongue.gif

  1. <div class="b1a_hr"></div>
  2. <ul class="b2a"><p><a href="page.php?page=4">Strona główna</a></p></ul>
  3.  
  4. <div class="b1a_hr"></div>
  5. <ul class="b2"><p><a href="page.php?page=6">Aktualności</a></p></ul>
  6. <div class="b1a_hr"></div>
  7. <ul class="b2"><p><a href="page.php?page=7&amp;menu=1">Logowanie</a></p></ul>
  8. <div class="b1a_hr"></div>
  9. <ul class="b2"><p><a href="page.php?page=9">Forum</a></p></ul>
  10. <div class="b1a_hr"></div>
  11.  
  12. <ul class="b2"><p><a href="page.php?page=11">Artykuły</a></p></ul>
  13. <div class="b1a_hr"></div>
  14. <ul class="b2"><p><a href="page.php?page=12">Recenzje</a></p></ul>
  15. <div class="b1a_hr"></div>
  16. <ul class="b2"><p><a href="page.php?page=8">Lista gier</a></p></ul>
  17. <div class="b1a_hr"></div>
  18. <ul class="b2"><p><a href="page.php?page=13">Screenshots</a></p></ul>
  19.  
  20. <div class="b1a_hr"></div>
  21. <ul class="b2"><p><a href="page.php?page=14">Pliki</a></p></ul>
  22. <div class="b1a_hr"></div>
  23. <ul class="b2"><p><a href="page.php?page=15">Szukaj</a></p></ul>
  24. <div class="b1a_hr"></div>


Chyba najlepszy przykład "divitis", daj to w jedną listę...


--------------------
"CSS is like a girlfriend. No matter how hard you try, she will always interpret it a different way."
Go to the top of the page
+Quote Post
Kildyt
post 1.09.2008, 11:15:05
Post #3





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

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


Cytat
Napis - "O czym jest ta strona?" Nie wiadomo dlaczego robiony jako img, zrobiłbym to normalnym textem... ale to moja opinia.
racja, nie pomyślałem nad tym

Cytat
Możesz użyć rozwijanego logowania coś a`la wykop.pl
raczej coś takiego odpada. Myślę, że stali użytkownicy zaznaczą funkcję, zapamiętaj mnie" i zaglądanie w logowanie będzie zbędne.

Cytat
miała być prosta i bez zbędnych bajerów smile.gif

Cytat
Wg mnie zbyt rażący gradient przy tekście "Jesteś nowy[...]"
miało się rzucać w oczy, ale chyba rzeczywiście trochę przesadziłem smile.gif

Cytat
Ale macie plusa za oryginalną kolorystykę aaevil.gif
dzięki smile.gif, miało być zauważalnie tongue.gif

Cytat
PS: Jesteście krytykami w dziedzinie gier MMORPG? Najlepsza ocena 3/10 WoW biggrin.gif
no wczoraj dopiero wystartowaliśmy tongue.gif Widocznie ktoś WoW-a nie lubi winksmiley.jpg

Cytat
Uhh... przejdźmy do menu... masakra tongue.gif
próbowałem <p>, zmienić na <li>, ale kółka czarne wyświetla (jak to w menu), a <div class="b1a_hr"></div> miałem najpierw jako <hr>, ale w kochanym IE nie rozumiało, że jak margin ma 0px to ma mieć 0px.

Dzięki za ocenę winksmiley.jpg


EDIT
Validacja css poprawiona.
Box z ostatnimi tematami na forum został zmieniony.

Ten post edytował Kildyt 1.09.2008, 13:04:39


--------------------
Go to the top of the page
+Quote Post
webber
post 1.09.2008, 11:35:07
Post #4





Grupa: Zarejestrowani
Postów: 107
Pomógł: 19
Dołączył: 30.10.2007

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


Cytat
próbowałem <p>, zmienić na <li>, ale kółka czarne wyświetla (jak to w menu), a <div class="b1a_hr"></div> miałem najpierw jako <hr>, ale w kochanym IE nie rozumiało, że jak margin ma 0px to ma mieć 0px.


<li> heh, no i kropki miałeś? dajesz coś takiego:

.menu li { list-type: none; } i już kółek nie ma tongue.gif

Ehh... szukanie nie boli!

http://css.maxdesign.com.au/listamatic/

Cytat
miała być prosta i bez zbędnych bajerów smile.gif


A wyszła brzydka... sad.gif

I polecam poczytać o typografii, to na prawdę ważne by strona była czytelna, Wasza nie jest, dział aktualności wygląda źle.

Ten post edytował webber 1.09.2008, 11:36:14


--------------------
"CSS is like a girlfriend. No matter how hard you try, she will always interpret it a different way."
Go to the top of the page
+Quote Post
tiraeth
post 1.09.2008, 11:43:28
Post #5





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

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


Kompletnie nie znasz podstawowych zasad tworzenia stron pod względem kodu. Nie wiesz, że każdy znacznik można stylować przy pomocy CSS. Poza tym, szablon za ciemny, brzydkie rozmycia w ramkach. Zbyt mały font po prawej stronie. Nagłówek jest beznadziejny, a stopka wtapia się w całą stronę, a powinna być takim zakończeniem strony.

Cała nawigacja na górze powinna być zbudowana w ten sposób:
  1. <ul class="nawigacja">
  2. <li><a href="#" class="aktywny">Strona główna</a></li>
  3. <li><a href="#">Aktualności</a></li>
  4. <li><a href="#">Logowanie</a></li>
  5. <li><a href="#">Forum</a></li>
  6. </ul>


Z rzeczy technicznych, strona w HTML 4.01 a i tak się nie waliduje. CSS także nie przechodzi walidacji. Treść artykułów w samym <div>, z <br /> rozdzielającymi akapity. Wiesz, do czego służy znacznik <p>? Elementy formularza logowania umieszczasz w divach. Brakuje <label>. Mieszasz style zewnętrzne z wewnętrznymi. Tabela z listą gier też nie zachwyca - mamy znaczniki <tbody>, <thead> i <th>...
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: 25.04.2024 - 17:13