Forum PHP.pl > www.krotex.net

gron

6.06.2008, 21:46:38

Ostatnio chodził mi długo pomysł po głowie i go zrealizowałem w końcu, co myślicie o tej stronie?
Krotex.net Jest to strona mająca za zadanie reklamować aukcje z Allegro, ale co ja tu będę

marcio

6.06.2008, 22:06:01

No fajnie czytelnie mi sie podoba kolorki tez ladnie dobrane ale:
http://www.krotex.net/index.php?action=rej.../)%3C/script%3E
I jeszcze jedno zalozylem sobie konto jako login dalem ' a jako email opisdsdsd waliduj takie rzeczy

EDIT:
http://www.krotex.net/index.php?action=rej...amp;task=%22%3E
http://www.krotex.net/index.php?action=rej...zdro..%3C/h1%3E
http://www.krotex.net/index.php?action=rej...-?%3C!-----

koderrr

6.06.2008, 22:15:05

nawet schludnie i pomysl tez dobry

ale wyglad nic nie mowi o dzialalnosci
i nijak kojarzy sie z allegro

uwidocznilbym to jakos

gron

6.06.2008, 22:20:16

Dzięki koderrr na takim poziomie oczekiwałem odpowiedzi i pomocy Plusik leci do Ciebie.
Nie chcę aby się kojarzyło z Allegro, bo niedługo dojdzie eBay, może kiedyś Świstak.
Ale z tego co kupiłem trochę ruchu tu i tam to zauważyłem że komunikacja i przesłanie tej strony trochę szwankuje. Myślę nad tym co zrobić, aby w przeciągu 20 sek wytłumaczyć ludziom co to jest i dlaczego jest dla nich korzystne.

Konradek

6.06.2008, 22:41:58

popraw "Chcesz coś sprzedać? Wypomuj swoje towary ",

no i title w meta tagach zaniemógł

//EDIT
pomysł fajny

nithajasz

6.06.2008, 23:26:07

...a mi się layout nie podoba, jest krzywo pocięty minimalnie ale jednak zauważalnie.

Wystające bloki powodują efekt, że czegoś mi brakuje zamiast białego tła.

Walidacji rejestrowania dalej jak widzę brakuje.

[HTML] pobierz, plaintext 
<div id="mmagic"> </div>
    <div id="header"> </div>
    <div id="spacer"> </div>
[HTML] pobierz, plaintext

Po co Ci puste divy?

[HTML] pobierz, plaintext 
<table><tr><td>
	<form method="POST" action="">
	Użytkownik:</td><td> <input type="text" name="name" value=""></td></tr><tr><td>
	Hasło: </td><td><input type="password" name="pass" value=""></td></tr><tr><td>
	Powtórz hasło:</td><td><input type="password" name="pass2" value=""></td></tr><tr><td>
	Email:</td><td><input type="text" name="email" value=""></td></tr><tr><td colspan="2">
	<input type="hidden" name="spr" value="1"><input type="submit" value="Rejestracja">
	</form>
</td></tr></table>
[HTML] pobierz, plaintext

Nie wygląda mi to na dane tabelaryczne.

Walidacja:
<h2 id="results" class="invalid">This page is not Valid XHTML 1.0 Strict!</h2> Result: Failed validation, 45 Errors
Jedyne co mi się podoba to chyba dobór kolorów

pyro

7.06.2008, 08:11:41

Cytat(gron @ 6.06.2008, 22:46:38 )

Ostatnio chodził mi długo pomysł po głowie i go zrealizowałem w końcu, co myślicie o tej stronie?
Krotex.net Jest to strona mająca za zadanie reklamować aukcje z Allegro, ale co ja tu będę

podatne na XSS

http://www.krotex.net/index.php?action=rej...3)%3C/script%3E

poza tym po prawej stronie jest taki napis:

Cytat

Krzycz, nie Cię usłyszą. Jak najwięcej osób musisz

W jakim to języku

?

.radex

7.06.2008, 10:20:14

Masz gdzieś niepodomykane/krzyżujące się tagi, bo jest mnóstwo błędów typu "Opening and ending tag mismatch"

AjaxSrajax

8.06.2008, 13:28:01

No to jadziem

<div id="top-nav"> - niepotrzebne, zamiast tego id="top-nav" dla <ul> ***.
znaczniki <b> nie są fajne, zamiast tego CSS.
<div id="nav"> - niepotrzebne, zamiast tego id="nav" dla <ul> ***
<div class="clear"> </div> - takie clearowanie nie ma sensu; zamiast tego clear dla pierwszego blokowego w kolejności.

Kod

<div id="visit">
<h2>Jak zacząć?</h2>
<p>1. Zakładasz konto na Krotex.net (Potem zatwierdzasz i logujesz.)</p>
<p>2. Dodajesz użytkownika lub kilku z Allegro. (Z tym systemem w tej chwili współpracujemy.)</p>
<p>3. Na forach internetowych lub blogach w "podpis" lub "sygnaturkę" dodajesz skrypt który generuje Krotex.net (Tą część wykonujesz tylko raz. Możesz też dodawać swój podpis w dowolnych miejscach.)</p>
<p>4. I to wszystko. Od tej chwili zawsze gdy sprzedajesz coś przez Allegro, twoje aukcje widzi wiele osób.</p>
</div>

Tu aż się prosi pozbyć diva i użyć listy uporządkowanej <ol>

To mi też wygląda na listę, tym razem <ul>:

Kod

<p>
                                       zestaw_sam,
                                                    PlayForFan,
                                                    Hazard,
                                                    Chardy1975,
                                                    <a target="_blank" href="?id=37">demo</a>,
                                                    Mazak,
                                                    Akira,
                                                    info,
                                                    stopspamu,
                                                    nowy,
                                                    Centkowany,
                                                    fuhflesz,
                                                    modna-kasia,
                                                    blach0r,
                                                    Patosław,
                                                    BLUE,
                                                    Ma_niek,
                                                    Eltona,
                                                    Rafał,
                                                    gronsoft,
                                                    mała,
                                                    szybki,
                                                    ka$a,
                                                    maxim,
                                                    <a target="_blank" href="?id=14">konsole</a>,
                                </p>

brak labeli dla forma - nieładnie...
To tak z pierwszej strony. W porównaniu z kodem innych nienajgorzej, ale daleko do ideału. Za dużo divów ciut ciut

*** może jestem maniakiem, ale dałbym troszkę bardziej semantyczne nazwy klas oraz id.

pozdro