Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: Boome.pl
Forum PHP.pl > Inne > Oceny
Boome
Witam,

Proszę o ocenę kodu HTML i grafiki serwisu Boome.pl. Proszę nie dodawać fikcyjnych aukcji gdyż serwis został już uruchomiony, mimo że wiele rzeczy wymaga jeszcze pracy guitar.gif Jest to wynik mojej pracy w wolnych chwilach. Uprzedzam, że grafik ze mnie słaby więc może coś poradzicie smile.gif

Ideą serwisu jest umożliwienie bezgotówkowej wymiany przedmiotów i usług.

Link: Boome.pl - wymiana przedmiotów i usług
tiraeth
+ Valid XHTML 1.0 Strict
+ Valid CSS2
- Na pierwszy rzut oka wygląda to jak kopia last.fm
- Wyszukiwarka wyjechała poza sidebar (FF3, Unix)
- Nie działa opcja Pokaż inne przedmioty i Wyszukiwanie zaawansowane
- Lista kategorii zerżnięta z Allegro


Mimo powyższej oceny stwierdzam, że spamujesz kolego...
AjaxSrajax
Hej,

Zamiast:
Kod
<div id="logo"><a href="http://boome.pl/"></a></div>


dajemy:

Kod
<h1><a href="http://boome.pl/"><img src="..." alt="boome.pl" width="..." height="..." /></a></h1>
<h2><span>czas na zmiany</span></h2>

no i dla spana w h2 display:none;

Po co kolejny div o id="menu" ? id dla <ul> and div out.

Niesemantyczne brzydactwo:
Kod
<div id="header-shade"></div>


Taką robotę zwalamy na CSS, a jak nie, to zmieniamy doca na HTML Transitional biggrin.gif

Co za dużo to nie zdrowo:

Kod
<div class="container">    
<div id="left-column-container">
<div id="left-column-header"></div>    
<div id="left-column-body">


Następnie odpalamy naszą ulubioną stronę internetową i wpisujemy : "label form" w polu wyszukiwania winksmiley.jpg

Potem <hr />-y zamieniamy na border-bottomy w CSS

Patrz wyżej:
Kod
<div id="left-column-shade"></div>    
<div class="right-column-shade"></div>    
<div id="footer-shade"></div>


Mam zły dzień, więc do <br/>-a też się doczepię biggrin.gif

Takie obserwacje ze strony głównej.

Papa
Boome
Cytat(tiraeth @ 1.08.2008, 13:35:50 ) *
- Na pierwszy rzut oka wygląda to jak kopia last.fm


Nie widzę żadnego podobieństwa. Może konkretniej?

Cytat(tiraeth @ 1.08.2008, 13:35:50 ) *
- Nie działa opcja Pokaż inne przedmioty i Wyszukiwanie zaawansowane


Zgadza się. Do zrobienia.

Cytat(tiraeth @ 1.08.2008, 13:35:50 ) *
- Lista kategorii zerżnięta z Allegro


A może z np. subasta.pl ? smile.gif
.radex
Cytat(AjaxSrajax @ 1.08.2008, 13:35:53 ) *
Kod
<h1><a href="http://boome.pl/"><img src="..." alt="boome.pl" width="..." height="..." /></a></h1>
<h2><span>czas na zmiany</span></h2>

no i dla spana w h2 display:none;


bez width i height tongue.gif Takie rzeczy to w CSS tongue.gif No i można to też zrobić troszeczkę inaczej
tommy4
Lepiej design by wyglądał z wygładzaniem czcionek. smile.gif

Poza tym zakładki (Wyszukiwarka, etc.) można było zrobić bardziej bajto-oszczędniej. winksmiley.jpg Tzn. tekst żeby nie był na obrazku zakładki a po prostu wyświetlany, wtedy obrazek zakładki ładował by się tylko raz.
Kildyt
Design jak najszybciej do poprawy. Nie da sie przez ten czerwony wytrzymać długo na stronie.
Na różnego rodzaju listach stosuj wyróżnienie co drugiej pozycji, aby można było je szybko sobie w głowie zauważyć, ten obrazek do tego itd.

Popraw design jak najszybciej. smile.gif

Pozdrawiam!
AjaxSrajax
Cytat
@radex_p bez width i height tongue.gif Takie rzeczy to w CSS tongue.gif


Hehe, nie do końca. Nie zapominaj, że obrazki mogą być wyświetlane, gdy CSS może być wyłączony.
Niektórzy twierdzą, że warto, niektórzy, że nie. Ja się zaliczam do tych pierwszych.

Pierwsza linijka na www.w3.org:

Kod
<h1 id="logo"><img alt="The World Wide Web Consortium (W3C)" height="48" width="315" src="/Icons/w3c_main" /></h1>
Niektórzy nawet twierdzą, że strona może się bez tych atrybutów sypnąć, ale to już ortodoxy biggrin.gif

pzdr i sry za mały offTOP
Boome
Dzięki wszystkim za uwagi. Na pewno większość rzeczy, o których pisaliście poprawię. Dam znać o zmianach.
.radex
Cytat(AjaxSrajax @ 1.08.2008, 21:02:26 ) *
Hehe, nie do końca. Nie zapominaj, że obrazki mogą być wyświetlane, gdy CSS może być wyłączony.
Niektórzy twierdzą, że warto, niektórzy, że nie. Ja się zaliczam do tych pierwszych.

Pierwsza linijka na www.w3.org:

Kod
<h1 id="logo"><img alt="The World Wide Web Consortium (W3C)" height="48" width="315" src="/Icons/w3c_main" /></h1>
Niektórzy nawet twierdzą, że strona może się bez tych atrybutów sypnąć, ale to już ortodoxy biggrin.gif

pzdr i sry za mały offTOP


width i height są AFAIK tylko w transitionalu tongue.gif
AjaxSrajax
Cytat
@radex width i height są AFAIK tylko w transitionalu tongue.gif


Niby tak, dlaczego jednak walidator się nie pluje? (Tylko mi nie mów, że walidator jest cienki biggrin.gif)
Popatrz na Doctype www.w3.org.
pientka winksmiley.jpg
nrm
Cytat(tiraeth @ 1.08.2008, 11:35:50 ) *
- Na pierwszy rzut oka wygląda to jak kopia last.fm

co paliłeś? guitar.gif
tiraeth
normanos, myślisz, że Ci powiem guitar.gif biggrin.gif Nie wiem czemu, ale tak mi się skojarzyło biggrin.gif
Boome
Witam ponownie

Ze względu na dość liczne uwagi na temat czerwonego layoutu troszkę go zmieniłem. Głównie odcień - nie jest już tak jaskrawy i nieprzyjemny.

Nowa wersja: klik
Stara wersja:
To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.
Invision Power Board © 2001-2025 Invision Power Services, Inc.