Не знаю как правильно написать условие

[Nird]

Добрый вечер. Редко имею дело сос криптами. но вот пришлось. Из-за отсутвия знаний обращаюсь за советом.

Есть рабочий код

function changestyle(type){
			param=document.getElementById(type);
			if(param.style.display == "block") param.style.display = "none"
			else param.style.display = "block"
			}

По ссылке передаем идентификатор, и блок скрывается. Если блок уже скрыт, то ссылка показывает его. Как добавить, что если блок с переданным индентификатором скрылся, то у другого блока изменился левый отступ, а если показался, то левый отступ вернулся?

В чем ошибка, синтаксис правильный:

function changestyle(type){
			param=document.getElementById(type);
			if(param.style.display == "block") 
			{
			param.style.display = "none";
			inright.style.margin = "0px";
			}
			else 
			{
			param.style.display = "block";
			inright.style.margin-left = "10px 20px";
			}	
			}

Edited December 16, 2009 by Nird

[hf3]

Ну В общем так. Только

inright в контексте неопреден… надо так наверное добавить в начале inright = document.getElementById("inright")

[Nird]

Ну В общем так. Только

inright в контексте неопреден… надо так наверное добавить в начале inright = document.getElementById("inright")

Не совсем, в фнкцию передается один идентификатор, а в отсуп меняется у блока с идентификатором "inright"

его нужно как то определять?

[hf3]

нет. Его просто нужно явно указать.

Джаваскрипт не знает что такое Inreight - он знает document.getElementById("inright")

поэтому либо

var inright = document.getElementById("inright")

либо

document.getElementById("inright").style.margin = "0px";

[Nird]

нет. Его просто нужно явно указать.

Джаваскрипт не знает что такое Inreight - он знает document.getElementById("inright")

поэтому либо

var inright = document.getElementById("inright")

либо

document.getElementById("inright").style.margin = "0px";

благодарю, сейчас попробуем

[psywalker]

inright.style.margin-left = "10px 20px"; - По правилам, если в словах встречается дефис, то пишется так marginLeft . И вообще интересно, как же ты ставишь левому Марджину аж 2 значения, так какое ему выбрать по-твоему?

[hf3]

Кстати да…

Даже и не заметил…

[Nird]

inright.style.margin-left = "10px 20px"; - По правилам, если в словах встречается дефис, то пишется так marginLeft . И вообще интересно, как же ты ставишь левому Марджину аж 2 значения, так какое ему выбрать по-твоему?

да нет. я тут запутался. я хотел либо marginLeft="20px" либо margin="10px 20px"

[psywalker]

да нет. я тут запутался. я хотел либо marginLeft="20px" либо margin="10px 20px"

Но уже распутался?

[Nird]

Но уже распутался?

в целом да. но почему такой глюк: скрипт начинает работать только после второго клика? первый раз кликаешь - не работает, втоорой,третий... все работает. страницу перезагружаешь и снова первый раз кликаешь - не работает, втоорой,третий... все работает

[hf3]

А у вас изначально блок открыт или скрыт? В ксс прописано свойство дисплэй?

[Nird]

А у вас изначально блок открыт или скрыт? В ксс прописано свойство дисплэй?

ой, девушка,милая. вы мне подарили два часа сна. спасибо огромное

[psywalker]

ой, девушка,милая. вы мне подарили два часа сна. спасибо огромное

А это не девушка - это оно

[Nird]

А это не девушка - это оно

после пятидней безвылазной верстки мог и ошибиться. сори

[andyp]

в целом да. но почему такой глюк: скрипт начинает работать только после второго клика? первый раз кликаешь - не работает, втоорой,третий... все работает. страницу перезагружаешь и снова первый раз кликаешь - не работает, втоорой,третий... все работает

Дело в том, что у Вас видимо вначале блок показан и у него не выставлено свойство display в атрибуте style.

elem.style возвращает только то, что прописано в атрибуте style, и не более того, оно не возвращает стили, которые

прописаны в других местах для этого элемента. Т.е. вначале у Вас значение свойства равняется пустой строке "".

[hf3]

А мне все равно приятно)

[Nird]

Дело в том, что у Вас видимо вначале блок показан и у него не выставлено свойство display в атрибуте style.

elem.style возвращает только то, что прописано в атрибуте style, и не более того, оно не возвращает стили, которые

прописаны в других местах для этого элемента. Т.е. вначале у Вас значение свойства равняется пустой строке "".

вы абсолютно правы