В глаза долблюсь, с первого раза эту жесть в тегах не заметил. Если кратко, то открытие и закрытие формы происходит в разных уровнях, что в свою очередь обычно ведет к самозакрытию. Т.е. кнопка, которая вызывает событие находится уже вне самой формы при рендеринге. Я не перелопачивал код и не буду, но инфа сотка, так оно и есть.
ЗЫ: Оно вообще работать не должно, сжечь я бы рекомендовал. И верстать научится.
ЗЫЫ: Без обид, держи
валидатор.