[JavaScript]Problem z otwieraniem się elementu po kliknięciu Opcje

[Mody23]

Witam!

Mam gotowy kod JS, który powoduje, że po kliknięciu w odnośnik, otworzy się menu. Jednak problem mam taki, że po kliknięciu, nic się nie dzieje. Próbowałem umieszczać kod JS w sekcji head, jak i odtwarzać go z oddzielnego pliku, ale nijak mi nie chce działać.

Kod js:

Kod

<!--
<script type="text/javascript">
var show = 1;
function hide_menu(group_panel) {
document.getElementById("group_panel").style.visibility = "hidden";
show = 1;
}

function menu(group_panel) {

if (show == 1) {
document.getElementById("group_panel").style.visibility = "visible";
show = 0;
}
else {
document.getElementById("group_panel").style.visibility = "hidden";
show = 1;
}
}
//
-->
</script>

A później tak się odwołuję do tego kodu:

[HTML] pobierz, plaintext 
<span onclick="menu(1)" align="left" style="cursor: pointer; color: red; font-weight: bold; text-decoration: underline;">Menu Moderatora</span>
		<span id="group_panel" style="position:absolute; left:?px; top:?px; z-index:1; visibility:hidden">
			<table style="border: 1px solid black" cellspacing="0" cellpadding="1" class="forumline">
				<tr>
					<td align="center" class="row2" height="25" colspan="2"><b><a href="java script:hide_menu(1)"><span style="color: red; font-weight: bold; text-decoration: underline;">[ MENU - UKRYJ ]</span></a></b></td>
				</tr>
				<tr>
					<td align="center" class="row1" width="36" height="36">{S_TOPIC_MOD_DELETE_IMG}</td>
					<td class="row1" width="220"><b>{S_TOPIC_MOD_DELETE}</b></td>
				</tr>
			</table></span>
[HTML] pobierz, plaintext 

Gdzie jest problem?

Pozdr.

[Prph]

Używaj narzędzi do debugowania, np. Firebug dla Firefoksa i Chrome. Jak go włączysz i przełączysz się na zakładkę konsola, to zobaczysz błędy javascript, o ile takie są.

A tak na oko, to coś z position jest nie tak: position:absolute; left:?px; top:?px; z-index:1; visibility:hidden. Skąd te ?px.
Druga sprawa: zamiast visibility używaj display: none i block.

[Mody23]

Tyle, że ja wziąłem ten kod ze strony (gotowiec). Podmieniłem tylko nazwy "group_panel". Wcześniej były inne nazwy. Co ciekawe, tam działa to bez zarzutu.

[nospor]

To pokaż oryginał, bo po kodzie co pokazałeś to na 90% jestem pewien, że te twoje "tylko" jest zrobione źle.

[Mody23]

JS:

Kod

var show = 1;
function hide_menu(moderator_panel) {
document.getElementById("moderator_panel").style.visibility = "hidden";
show = 1;
}

function menu(moderator_panel) {

if (show == 1) {
document.getElementById("moderator_panel").style.visibility = "visible";
show = 0;
}
else {
document.getElementById("moderator_panel").style.visibility = "hidden";
show = 1;
}

}
//-->
</script>

HTML:

[HTML] pobierz, plaintext 
<!-- BEGIN modpanel_link -->
<tr>
	<td align="left" valign="top" nowrap="nowrap"><span onclick="menu(1)" align="left" style="cursor: pointer; color: red; font-weight: bold; text-decoration: underline;">Menu Moderatora</span>
		<span id="group_panel" style="position:absolute; left:?px; top:?px; z-index:1; visibility:hidden">
			<table style="border: 1px solid black" cellspacing="0" cellpadding="1" class="forumline">
				<tr>
					<td align="center" class="row2" height="25" colspan="2"><b><a href="java script:hide_menu(1)"><span style="color: red; font-weight: bold; text-decoration: underline;">[ MENU MODERATORA - UKRYJ ]</span></a></b></td>
				</tr>
		<!-- BEGIN delete -->
				<tr>
					<td align="center" class="row1" width="36" height="36">{S_TOPIC_MOD_DELETE_IMG}</td>
					<td class="row1" width="220"><b>{S_TOPIC_MOD_DELETE}</b></td>
				</tr>
		<!-- END delete -->
				<tr>
					<td align="center" class="row2" width="36" height="36">{S_TOPIC_MOD_MOVE_IMG}</td>
					<td class="row2"><b>{S_TOPIC_MOD_MOVE}</b></td>
				</tr>
				<tr>
					<td align="center" class="row1" width="36" height="36">{S_TOPIC_MOD_MERGE_IMG}</td>
					<td class="row1"><b>{S_TOPIC_MOD_MERGE}</b></td>
				</tr>
				<tr>
					<td align="center" class="row2" width="36" height="36">{S_TOPIC_MOD_ULOCK_IMG}</td>
					<td class="row2"><b>{S_TOPIC_MOD_LOCK}</b></td>
				</tr>
				<tr>
					<td align="center" class="row1" width="36" height="36">{S_TOPIC_MOD_SPLIT_IMG}</td>
					<td class="row1"><b>{S_TOPIC_MOD_SPLIT}</b></td>
				</tr>
		<!-- BEGIN sticky -->
				<tr>
					<td align="center" class="row2" width="36" height="36">{S_TOPIC_MOD_NSA_IMG}</td>
					<td class="row2"><b>{S_TOPIC_MOD_NSA}</b></td>
				</tr>
		<!-- END sticky -->
		<!-- BEGIN announce -->
				<tr>
					<td align="center" class="row1" width="36" height="36">{S_TOPIC_MOD_NSA2_IMG}</td>
					<td class="row1"><b>{S_TOPIC_MOD_NSA2}</b></td>
				</tr>
		<!-- END announce -->
				<tr>
					<td align="center" class="row2" width="36" height="36">{S_TOPIC_MOD_EXPIRE_IMG}</td>
					<td class="row2"><b>{S_TOPIC_MOD_EXPIRE}</b></td>
				</tr>
			</table></span></td>
</tr>
<!-- END modpanel_link -->
[HTML] pobierz, plaintext