Forum PHP.pl

Witam, mam taki skrypt do poczty swojej

[XML] pobierz, plaintext 
function OpenLink(Ourl, element)
{
var someDiv = document.getElementById(element);
myAajax.get({
    url: Ourl,
            onLoading : function() {
                  /*Ładowanie*/
                  someDiv.innerHTML = '<img src="images/preloader.gif" src="preloader">';
            },
    onSuccess : function(obj) {
        /* Ustawienie nowej, pobranej z serwera wartości */
        someDiv.innerHTML = obj.responseText;
    },
    onError : function(obj) {
        /* W przypadku wystąpienia błędu wyświetlenie komunikatu */
        alert("Błąd ładowania strony: " + obj.status);
    }
});
}
 
//Otwieranie wiadomości
function OpenMessage(id)
{
      var mainElement = document.getElementById('main_div');
      var box_div = document.getElementById('box_div');
      if (mainElement.style.visibility == 'hidden')
      {
            mainElement.style.visibility = 'visible';
            box_div.style.visiblity = 'hidden';
            box_div.style.heigth = '0';            
      }
      else
      {
            mainElement.style.visibility = 'hidden';
            box_div.style.visiblity = 'visible';      
      }
      OpenLink('vMail.php?id='+id, 'box_div');
}
[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