![]() |
![]() |
![]()
Post
#1
|
|
Grupa: Zarejestrowani Postów: 230 Pomógł: 36 Dołączył: 31.03.2006 Skąd: Zielona Góra Ostrzeżenie: (0%) ![]() ![]() |
Witam.
Jako, że projekt został już zatwierdzony i nie ma możliwości większych zmian, proszę o ocenę przede wszystkim kodu html oraz ogólnego wrażenia. Adres: marynizacja.izgo.pl edit: dodanie taga Ten post edytował woj_tas 29.05.2008, 10:22:26 |
|
|
![]() |
![]()
Post
#2
|
|
Grupa: Zarejestrowani Postów: 172 Pomógł: 9 Dołączył: 22.03.2008 Skąd: Wrocław Ostrzeżenie: (0%) ![]() ![]() |
Te beery w kodzie wyglądają jak domyśl się co na środku ładnie wysprzątanego salonu. Myślę, że przy tak prostym lejku i kodzie nie ma sensu tworzyć identyfikatorów dla h1, h2, h3. Przeciez należą one do logicznych fragmentów strony, więc lepiej je stylizować div#nazwabloku h1 zamiast h1#nag-x. Raz stosujesz semantyczne klasy i id typu: artykuł, content, a raz malo znaczace jak ipb-top-right-link. Nie podobają mi się też akcje typu <div id="artykul"> po co tak, zamiast #content p{....}. Ten div juz jest w divie... Więc bym go wywalił... W dziale ogloszenia lista ogloszeń. Zamiast <p class="dodano"> dalbym spana o tej samej klasie. Następnie treść ogłoszenia. Po co <p>? Nie lepiej ująć to po prostu jako tekst elementu <li>? Oczywiście po wcześniejszej zmianie <ul id="list"> ( jak by ktoś nie wiedział, że to lista;) na <ul id="ogloszenia">. Wtedy tekst w li bedzie rozumiany jako ogłoszenie. Gdybym był maniakiem to bym tam walnął <cite> (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) .Aha, style w kodzie xhtml też nie są wporzo if u ask me (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) . Ale nie jest najgorzej. Za chęci i kod 7/10. Piona
|
|
|
![]() ![]() |
![]() |
Aktualny czas: 10.10.2025 - 09:32 |