Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

 
Reply to this topicStart new topic
> [www] strona z pasjansami
kris2
post 28.05.2008, 21:55:28
Post #1





Grupa: Zarejestrowani
Postów: 150
Pomógł: 3
Dołączył: 15.08.2007

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


Proszę o ocene stronki z pasjansami
pasjans - gry karciane
czy jest prawidlowo wykonana pod wzgledem xhtml i optymalizacji contentu.
Go to the top of the page
+Quote Post
LonelyKnight
post 28.05.2008, 22:07:03
Post #2





Grupa: Zarejestrowani
Postów: 240
Pomógł: 13
Dołączył: 1.06.2007
Skąd: Wrocław

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


Kod może być.

Grafika niedopracowana - np. strzałki w menu albo zaokrąglenia narożników - wszędzie tła nie pasują.

Dużo ortografów, literówek itp.


--------------------
Good programming is 99% sweat and 1% coffee.
Make it idiot proof and someone will make a better idiot...
Go to the top of the page
+Quote Post
empathon
post 28.05.2008, 22:21:20
Post #3





Grupa: Zarejestrowani
Postów: 246
Pomógł: 31
Dołączył: 13.11.2006
Skąd: się znamy?

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


Ocena kodu.
Nagłówek h1 powinieneś mieć tu jeden. Kolejne poziomy jedziesz w dół a nie wracasz do h1. Barkuje title w linkach w img alt'ów.
http://www.pasjans.org/pasjanse/ < tu spokojnie mogłeś też dać liste

Z grafiki jedyne co mi się podoba to favicon winksmiley.jpg


--------------------
Goldenline: Łukasz Rodziewicz
Go to the top of the page
+Quote Post
koderrr
post 29.05.2008, 07:51:29
Post #4





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

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


malo starannie wycieta grafika

przyjrzyj sie naglowkowi do contentu
Go to the top of the page
+Quote Post
hateman
post 29.05.2008, 08:47:34
Post #5





Grupa: Zarejestrowani
Postów: 86
Pomógł: 18
Dołączył: 28.04.2008
Skąd: Poznań

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


Niedokładnie wycięta grfika, nieprzeźroczyste narożniki, logo też średni.
Za to świetny pomysł (pogram trochę w pracy smile.gif), na plus opis do każdego pasjansa.
Go to the top of the page
+Quote Post
kris2
post 30.05.2008, 08:01:27
Post #6





Grupa: Zarejestrowani
Postów: 150
Pomógł: 3
Dołączył: 15.08.2007

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


wielkie dzięki za uwagi

Poprawilem teksty i grafike. I zastosowalem sie do waszych porad.
Prosze o sprawdzenie czy jeszcze cos moglbym ulepszyc.
(z logo to i tak szczyt moich mozliwosci bo jestem programista a nie grafikiem winksmiley.jpg

Pozdrawiam

Kris
Go to the top of the page
+Quote Post
AjaxSrajax
post 30.05.2008, 15:36:46
Post #7





Grupa: Zarejestrowani
Postów: 172
Pomógł: 9
Dołączył: 22.03.2008
Skąd: Wrocław

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


<div style="clear: both;">&nbsp;</div> - ten wers gryzie w oczy. Co z tym h1? Jest h1
h2 potem znowu h1. Popraw to, bo żena. W sumie to na jakiejś stronce ( nie wiem, czy nie na jakiejś w3c) widziałem dwa nagłówki
h1, jeden master, drugi jako głowny nagłówek strony.
I nie to, żebym się czepiał, bo są stronki nawet guru kolesi z takim doctypem , ale
ja i tak bym go zmienil na XHTML, chociażby transitional.
Go to the top of the page
+Quote Post
kris2
post 30.05.2008, 15:58:31
Post #8





Grupa: Zarejestrowani
Postów: 150
Pomógł: 3
Dołączył: 15.08.2007

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


Dzięki jakoś przeoczylem to h1 na index.php
doctype tez poprawilem
a co Ciebie w tym <div style="clear: both;">&nbsp;</div> gryzie w oczy?
sorry ze pytam ale nie uwazam sie za eksperta od html'a wstydnis.gif
Go to the top of the page
+Quote Post
Shili
post 30.05.2008, 16:05:13
Post #9





Grupa: Zarejestrowani
Postów: 1 085
Pomógł: 231
Dołączył: 12.05.2008

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


Ten div niesamowicie łatwo można ominąć. Teraz jest jedynie nic nie wnoszącym w strukturę strony elementem, nie niesie ze sobą żadnej treści, jest krótko mówiąc mocno zbędny.
Go to the top of the page
+Quote Post
AjaxSrajax
post 30.05.2008, 16:36:15
Post #10





Grupa: Zarejestrowani
Postów: 172
Pomógł: 9
Dołączył: 22.03.2008
Skąd: Wrocław

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


Jak napisała koleżanka wyżej. XHTML nie służy przedstawianiu wyglądu strony, a jedynie ułożeniu struktury treści. Ten div wprost przeciwnie. Zlikwiduj float dodając do pierwszego blokowego potomka elementu pływającego właściwość clear. Widzę tam jeszcze spację, czyżby bez spacji się nie clearowało? Nie nabieraj złych nawyków, lepiej poczytaj sobie o layoutach css. glish, bluerobot, jeden z moich ulubionych, trzeba znać, warto znać. Pozdro.
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:11