не корректно работает onClick

[Darhan]

Даже не совсем знаю куда лучше отнести это в php или в js

Ну а по сути, имеем следующую ситуацию

<script language="Javascript">
  function show_div(div_id) {
	  if (document.getElementById(div_id).style.display=="none")
		  document.getElementById(div_id).style.display="block";   
	  else
		  document.getElementById(div_id).style.display="none";	 
} 
</script>


<div>
	<p align="center" style="font-size:14px; color:#5c584e;">Новости</p>
	<div class="page">
<?php	
		for ($i=0, $n=count($rows);$i<$n;$i++)
			{
				$row = $rows[$i];
				mosMakeHtmlSafe($row);
				$div="div".$i;
?>
				<div style="border-bottom-style:dashed; border-bottom-color:#5c584e; border-bottom-width:1px; ">
					<p>
						<span style="color:#0000FF">
							<?php echo $row->start_date; ?>:
						</span> <?php echo $row->title; ?> 
						<a href="java script:void(0);" onClick="show_div('<?php echo $div; ?>'); return false;" style="color:#0000CC"> >>>>></a> 
					</p>
					<div id="<?php echo $div; ?>" class="message">
							<?php echo $row->message; ?>
					</div>
				</div>

<?php 
			}
?>
	 </div>
</div>
<?php


?>

где класс

.message {
	display:none; 
	color:#5c584e; 
	border-style:dashed; 
	border-color:#5c584e; 
	border-width:1px; 
	margin: 10px; 
	padding: 5px;
}

и всебы и отлично да приходиться по нажимать по разу по всем ссылкам, чтоб они корректно по событию онклик открывались закрывались

если вместо <?php echo $div; ?> вставить допустим "div1", то событие нормально отрабатывает, тобиш один раз нажал и показался див, нажал и он скрылся

а так приходиться нажать, потом только можно нажимать и он будет показываться и скрываться...

что делать?

Edited April 26, 2010 by Darhan

[Great Rash]

Зачем вы вот так пишете: href="java script:void(0);" если у вас return false; стоит?

Так: onClick= писать неправильно - правильно писать все с маленьких букв onclick=.

Так: <script language="Javascript"> тоже писать нельзя, так только ИЕ понимает - правильно будет <script type="text/javascript">.

Попробуйте все исправить, может в этом дело?

UPD:

Опять же у вас идет проверка if (document.getElementById(div_id).style.display=="none"), но дело в том что если у вас в теге явно не указано style="display: none;", то document.getElementById(div_id).style.display вернет пустую строку (или undefined, точно не помню). Так что постройте условие по другому:

// или так
function show_div(div_id) {
	if (document.getElementById(div_id).style.display == "block")
		document.getElementById(div_id).style.display = "none";   
	else
		document.getElementById(div_id).style.display = "block";	 
}

// или так
function show_div(div_id) {
	if (document.getElementById(div_id).style.display == "none" || !document.getElementById(div_id).style.display)
		document.getElementById(div_id).style.display = "block";   
	else
		document.getElementById(div_id).style.display = "none";	 
}

[Darhan]

да действительно, чегот я подумал что если в классе укажу display: none, то этого достаточно будет

спасибо, помогло