[skrypt]ShoutBoxChmura v3, Werjsa oparta o Ajax & PHP na plikach Opcje

[b_chmura]

Witam
Chciałbym abyście wypróbowali, ocenili i poradzili co zmienić/dodać/usunąć/polepszyć.

Linki:
- Przykład działania
- JavaScript/Ajax
- PHPs
- Index

Z góry śliczne dzięki

Pozdrawiam
Chmura

P.S: Kolejna wersja już w oparciu o bazę MySQL

EDIT
Może dodam że istnieje rarytas który na pierwszy rzut oka jest niezauważalny.
Wybór "Ukryj/Pokaż" jest zapisywany w ciasteczku dzięki czemu zostaje zapamiętany a opcje którą wybraliśmy nie trafia szlag przy przeładowaniu strony :)

EDIT
W następnej wersji oczywiście Panel administracyjny.
W kolejnych rejestrowani użytkownicy.

Ten post edytował b_chmura 15.08.2007, 16:29:44

[SHiP]

Nie analizowałem dokładnie kodu źródłowego ale mam kilka uwag:
- wyglad dema, popracuj nad tym bo jest marnie. I najlepiej poszerz całego shouta bo jest imho lekko za wąski.
- target="_blank" w generowanych linkach. Z tego co wiem to jest to poprawne(xhtml strict). Uzytkownik powinien decydowac czy to ma byc nowe okno czy nie...
- <font color="red"> - chyba nie musze tłumaczyc (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)
- ta mini funkcja dotycząca BBcode. Niby jest ok ale nie ma żadnej ochrony przed np przeplataniem znacznikow. Polecam uzycie czegos zewnetrznego(np. moj handyCode ^^)
- nie wiem po co włączyłes advAjaxa to tego samego pliku js co plik shouta, ja wole miec porządek (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) (wiem wiem czepiam sie)
- widze ze nie jestes przyzwyczajony do petli for (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

[PHP] pobierz, plaintext 
<?php
for($i=0; $i < $max; $i++) 
  { 
	$dana = explode('||', $dane[$i]); 
	$color = ($dana[0]%2)? '': '#E1E1E1'; 
	echo '<div style="background-color: '.$color.';"><b>['.$dana[1].']</b> '.strip_tags($dana[2]).' <br />'.chmuraCode($dana[3]).'</div><hr />'; 
  }
?>
[PHP] pobierz, plaintext