Witam, mam taki skrypt do poczty swojej
[XML] pobierz, plaintext 
  1. function OpenLink(Ourl, element)
  2. {
  3. var someDiv = document.getElementById(element);
  4. myAajax.get({
  5.     url: Ourl,
  6.             onLoading : function() {
  7.                   /*Ładowanie*/
  8.                   someDiv.innerHTML = '<img src="images/preloader.gif" src="preloader">';
  9.             },
  10.     onSuccess : function(obj) {
  11.         /* Ustawienie nowej, pobranej z serwera wartości */
  12.         someDiv.innerHTML = obj.responseText;
  13.     },
  14.     onError : function(obj) {
  15.         /* W przypadku wystąpienia błędu wyświetlenie komunikatu */
  16.         alert("Błąd ładowania strony: " + obj.status);
  17.     }
  18. });
  19. }
  20.  
  21. //Otwieranie wiadomości
  22. function OpenMessage(id)
  23. {
  24.       var mainElement = document.getElementById('main_div');
  25.       var box_div = document.getElementById('box_div');
  26.       if (mainElement.style.visibility == 'hidden')
  27.       {
  28.             mainElement.style.visibility = 'visible';
  29.             box_div.style.visiblity = 'hidden';
  30.             box_div.style.heigth = '0';            
  31.       }
  32.       else
  33.       {
  34.             mainElement.style.visibility = 'hidden';
  35.             box_div.style.visiblity = 'visible';      
  36.       }
  37.       OpenLink('vMail.php?id='+id, 'box_div');
  38. }
[XML] pobierz, plaintext


HTML
<div id="box_div"></div>
<div id="main_box">Tutaj sąwyświetlane wiadomości(petla czyta je z bazy)</div>
Czy jest on dobry, da się go lepiej napisać Dziękuje