[js] Problem z getElementById().style.visibility

[_Adam_]

Witam,
Niby prosty kod, niby powinno działać, albo ślepnę już... Pomocy, czemu Mozilla sobie z tym nie radzi? :/ Znając życie błąd jest banalny, ale ja już nie mam siły Będę wdzięczny za każdą wskazówkę.

Kod: Zaznacz cały

<script type="text/javascript">
<!--
function changeMap(to)
{
 map.src="img/map"+to+".gif";
 if(document.getElementById)
 {       
  document.getElementById("div1").style.visibility = "hidden";
  document.getElementById("div2").style.visibility = "hidden";
  document.getElementById("div3").style.visibility = "hidden";
  document.getElementById("div"+to).style.visibility = "visible";
 }
 else if(document.all)
 {
  document.all["div1"].style.visibility = "hidden";
  document.all["div2"].style.visibility = "hidden";
  document.all["div3"].style.visibility = "hidden";
  document.all["div"+to].style.visibility = "visible";
 }
 else if(document.layers)
 {
  document.layers["div1"].visibility = "hide";
  document.layers["div2"].visibility = "hide";
  document.layers["div3"].visibility = "hide";
  document.layers["div"+to].style.visibility = "show";
 }       
}
//-->
</script>

<div id="div1" style="visibility: hidden"><a href="javascript:void();" onClick="changeMap(2);">Zoom Out</a></div>
<div id="div2" style="visibility: visible"><a href="javascript:void();" onClick="changeMap(1);">Zoom In</a><br><input type="button" class="button" onClick="changeMap(3);" value="Zoom Out"></div>
<div id="div3" style="visibility: hidden"><input type="button" class="button" onClick="changeMap(2);" value="Zoom In"></div>


[MH]

Na moje oko pownienes dac zamiast:

Kod: Zaznacz cały

if(document.getElementById)

ten kod:

Kod: Zaznacz cały

if(getElementById)

Jako ze sprawdzamy obsluge funkcji (tak ja to rozumiem).

Nie jestem jeszcze pewny co do poprawnosci tego skrawka (no ale to juz Mozilli nie dotyczy) 8)

Kod: Zaznacz cały

  document.layers["div1"].visibility = "hide";
  document.layers["div2"].visibility = "hide";
  document.layers["div3"].visibility = "hide";
  document.layers["div"+to].style.visibility = "show";

nie powinno byc tak?

Kod: Zaznacz cały

  document.layers["div1"].style.visibility = "hidden";
  document.layers["div2"].style.visibility = "hidden";
  document.layers["div3"].style.visibility = "hidden";
  document.layers["div"+to].style.visibility = "visible";

[marcoos]

Na moje oko pownienes dac zamiast:

Kod: Zaznacz cały

if(document.getElementById)

ten kod:

Kod: Zaznacz cały

if(getElementById)

Jako ze sprawdzamy obsluge funkcji (tak ja to rozumiem).

Bzdura.

Nie jestem jeszcze pewny co do poprawnosci tego skrawka (no ale to juz Mozilli nie dotyczy)

Jest ok, w NN4 tak się to nazywało. (Inna sprawa, czy jest sens jeszcze robić cokolwiek pod takie zabytki jak NN4...)

Problem leży gdzie indziej - obiekt map jest niezdefiniowany (tzn. map nie wskazuje na nic, powinieneś mu coś przypisać, choćby przez getElementById). Gdybyś użył Konsoli JS (w menu Narzędzia Firefoksa lub Narzędzia > programistyczne w Mozilli), to byś ten błąd od razu znalazł.

Drugi błąd to nieprawidłowe użycie void() - powinno być np. void(0);

[_Adam_]

"map" jest definiowane wcześniej za pomocą:

Kod: Zaznacz cały

<img alt="Map" name="map" src="img/map2.gif" border=0 width=400 height=375>


- tyle starczy?

Powodem problemu nie jest też void(), nie działa także gdy zamiast linków stosuje <input type="button" onClick=...>

Pozdr.

[kfassssss]

miałem podobny problem z widzialnością, niewidzialnością divów w mozilli. Dla pewności zastosuj raczej:

display=none
i
display=block

zamiast
style.visibility...

a jak zadziała to wróć i podziękuj

[Amato]

Wg porady MWD jak udzielil mi nie dawno ;]

dla:
<div id="xDIV">cosik tam</div>

Kod: Zaznacz cały

// dla Mozilli
xxDIV=document.getElement.ById('xDIV');
xxDIV.style.visibility='hidden';


choc ja do przypisania zmiennej obiektu DIV uzywam wlasnej konstrukcji, zapewnia mnie to ze pod wiekszoscia przegladarek nie bedzieproblemu :]

Kod: Zaznacz cały

function GetObject(jaki){
  if (!document.layers&&!document.all){
    if (document.getElementById) {
      rerun (getElementById(jaki));
    }
  }
  if (document.all) {
    return (document.all[jaki]);
  }
  if (document.layers) {
     return (document.layers[jaki]);
  }
  return (jaki);
}

// a uzywa sie wtedy tego tak:
// [do w.w przykladu]
xxDIV=GetObject('xDIV'); // koniecznie w cudzyslow!!
xxDIV.style.visibility='hidden';


[/code]

[Skorek-1990]

Ja też sie ostatnio bawie w skryptowanie w JS i też ostatno napodkałem problem z modyfikacją styli.

Nie ma innego sposobu edycji styli niż "document.getElementById('xxx').style.xxx" ??

Może jakimś innym sposobem dało by rade zmienić styl.

[Grzegorz]

Najprościej style edytować bez JS w Stylish.