Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

> [www] LinkBazar - strona do oceny, Handel linkami
Nowaker
post
Post #1





Grupa: Zarejestrowani
Postów: 4
Pomógł: 0
Dołączył: 13.08.2007
Skąd: Gniew / Gdańsk

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


Kilka dni temu skończyłem projekt LinkBazar, proszę o ocenę.
Przyjmę jakiekolwiek sugestie ;) Zarówno do layoutu, kodu HTML, jak i funkcjonalności.
Z sugestii, co by nowego można dorzucić na stronę będę się jeszcze bardziej cieszył.


Oto adres: http://www.linkbazar.pl/

Mogę dodać, iż stronę napisałem na frameworku KohanaPHP.
Wszystkie podstrony są valid HTML 4.01 Strict.

Ten post edytował Nowaker 5.06.2008, 22:06:59
Go to the top of the page
+Quote Post
 
Start new topic
Odpowiedzi
.radex
post
Post #2





Grupa: Zarejestrowani
Postów: 1 657
Pomógł: 125
Dołączył: 29.04.2006

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


A co mam ocenić? Wygląd czy kod?

Wróć! Ocenię jedno i drugie (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)

Całkiem nowoczesny wygląd, ale trochę dziadowska kompresja obrazków. W zasadzie nie ma wiele co nie oceniać, więc przejdę do kodu.

Ładne kodowanie, ładny doctype (i się waliduje).

Jednak widzę tu trochę classitis i divitis (czy jakoś tak)...

Kod
<div class="buttons">

<div id="first" class="header">


a nie lepiej

.buttons #first ?

W zasadzie to powinno to być tak:

#buttons .first

bo buttons występuje tylko raz, a "first" *może* być bardziej uniwersalne.

Kod
<div>
LinkBazar zaczyna działalność w sieci. Możecie kupować i sprzedawać linki oraz zapisać się do programu partnerskiego. 15 zł to Wasza premia za każdą osobę, która z Waszego polecenia kupi linki za 120 zł. 15 zł zarobicie też za każdego, kto udostępni swoją stronę internetową do wyświetlania linków. Namówcie swoich znajomych. Razem zarobimy więcej.</div>


<p> ?

Kod
<p class="big"><a href="linki/rejestracja.html">Zarejestruj się</a></p>


Ja bym to zrobił tak:

<a href="linki/rejestracja.html" class="big">Zarejestruj się</a>

Lub jeszcze inaczej:

<big><a href="linki/rejestracja.html">Zarejestruj się</a></big>

Po to są <big> i <small> (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)

Kod
<p>
<a href="http://www.linkbazar.pl/">LinkBazar - sprzedaż i kupno linków tekstowych</a>.
</p>


Po jaką jest tu ten link? No i jeszcze ta niepotrzebna konstrukcja z <p>

Pozdro (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)
Go to the top of the page
+Quote Post

Posty w temacie


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: 10.10.2025 - 22:42