Stworzyłem prosty skrypt który służy mi do menu. Gdy kliknę na plus cyli ikonę + otwiera mi się div i plus zamienia na - lecz gdy do tego chce otworzyc kolejnego diva to zamiast klikać raz żeby go otworzyć muszę kliknąć 2 razy, tak samo z zamknięciem. Gdy otwieram i zamykam tylko jednego diva to działa poprawnie po jednym razie klikam.

kod :

[XML] pobierz, plaintext 
<script type="text/javascript">
<!--
var counter=0;
var plusminus;
function hide(el)
{
    if(counter==0)
    {
        document.getElementById(el).style.display="block";
        counter=1;
        document.getElementById(el + "_icon").src = "images/minus.png";
    }
    else
    {
        document.getElementById(el).style.display="none";
        counter=0;
        document.getElementById(el + "_icon").src = "images/krzyz.png";
    }
}
//-->
</script>
[XML] pobierz, plaintext 

zapomniałeś o wklejeniu kodu HTML.

[HTML] pobierz, plaintext 
<div id="menulewe">
<div id="menu_lewe">
<dl id="menu0">
<dt id="bialy"><a href="index.php">Strona Główna</a></dt>
<dt id="all"><div onClick="hide('div1')"><div id="menu_click">Moje Zakłady<div id="plus"><img src="images/krzyz.png" id="div1_icon" border="0"></div></div></div></dt>
<div id="div1">
<dd><a href="w_grze.php">- W grze</a></dd>
<dd><a href="historia.php">- Historia</a></dd>
</div>
<dt id="all"><div onClick="hide('div2')"><div id="menu_click">Konto<div id="plus1"><img src="images/krzyz.png" id="div2_icon" border="0"></div></div></div></dt> 
<div id="div2">
<dd>- Panel administracyjny</dd>
<dd><a href='admin.php?third'>- Wiadomości</dd>
<dd><a href="dane.php">- Dane Konta</dd></a>
<dd><a href="ustawienia.php?pierwszy=1">- Ustawienia</a></dd>
<dd><a href="kupno.php">- Kupno</a></dd>
<dd><a href="sprzedaz.php">- Sprzedaż</a></dd>
</div>
<dt id="lewe_inne"><a href="wyloguj.php">Wyloguj</a></dt>
</dl>
</div>
[HTML] pobierz, plaintext 

Moim zdaniem jest wina w count bo gdy otworzysz jednego diva skrypt zrobi się raz i przypisze cyfrę 1 gdy wywoła się drugi raz musi zrobić 0 i po podwójnym kliku znowu 1 żeby się otworzyło

Dokładnie. Tylko jak to zmienić żeby skrypt obsługiwał dwa zdarzenia

Nie potrzebujesz żeby zmieniać jakieś liczniki.
Kiedy pracujesz z obiektem to pracuj z nim.

Odnosisz się do stylu to sprawdzaj jaki styl jest obecnie w tym obiekcie ustawiony i odpowiednio go zmieniaj

[HTML] pobierz, plaintext 
 
    function hide(el)
    {
		n = document.getElementById(el);
		if ( n.style.display=="none" )
		{
			n.style.display="block";
			document.getElementById(el + "_icon").src = "images/minus.png";
		}
		else
		{
			n.style.display="none";
			document.getElementById(el + "_icon").src = "images/krzyz.png";
		}
	}
[HTML] pobierz, plaintext 

teraz oba divy za 2 kliknięciem . musze kliknąć dwa razy na jeden i dwa razy na drugi, a po tym już działają na jedno kliknięcie poprawnie. nie wiem dlaczego tak jest nie ma countiera. Inne pomysły?

spróbuj to wytestować i zobaczysz ten błąd

?

Znaczy że nie jest dla nich ustalony styl początkowy.

Jeśli wciąż nie zda testu to nie zapisuj jako counter tylko w tablicy
var counter[el]=1 lub 0

ustawiłem styl początkowy lecz nie do tego diva co trzeba. Dzięki za pomoc już śmiga