Javascript-форум (https://javascript.ru/forum/)
-   Events/DOM/Window (https://javascript.ru/forum/events/)
-   -   Событие запускается со второго клика (https://javascript.ru/forum/events/20587-sobytie-zapuskaetsya-so-vtorogo-klika.html)

sympler 10.08.2011 15:18

Событие запускается со второго клика
 
не могу понять почему фукнция работает только со второго клика

<a href="javascript:void" onclick="showAdd();">Показать дополнительные поля</a><br>

<div id="add_tables" >

это блок

</div>


function showAdd() {if (document.getElementById("add_tables").style.display == "none") {document.getElementById("add_tables").style.display = "block";} else {document.getElementById("add_tables").style.display = "none";} }

devote 10.08.2011 15:21

<a href="javascript:void" onclick="showAdd(); return false;">Показать дополнительные поля</a><br>

ksa 10.08.2011 15:27

Цитата:

Сообщение от sympler
почему фукнция работает только со второго клика

Условие поменяй...

<!DOCTYPE html>
<html>
<head>
<style type="text/css">
#add_tables {
	display: none;
}
</style>
<script type="text/javascript">
function showAdd() {
	var o=document.getElementById("add_tables")
	if (o.style.display == "block") {
		o.style.display = "none";
	} else {
		o.style.display = "block";
	} 
}
</script>
</head>
<body>
<a href="javascript:void" onclick="showAdd();">Показать дополнительные поля</a>
<br />
<div id="add_tables">
	это блок
</div>
</body>
</html>


Первый раз нужное тебе значение = пусто...

devote 10.08.2011 15:32

Цитата:

Сообщение от ksa
Первый раз нужное тебе значение = пусто...

все у него с условием норм, то что оно пустое это говорит о том что оно отображается, но если оно скрыто, то 100% имеет значение none, а проверять на присутствие block не рекомендуется, потому как оно может иметь значение отличное от block, а значение none всегда стоит в случае если элемент скрыт даже свойствами css

ksa 10.08.2011 15:35

Цитата:

Сообщение от devote
если оно скрыто, то 100% имеет значение none

Бивас тест...

<!DOCTYPE html>
<html>
<head>
<style type="text/css">
#add_tables {
	display: none;
}
</style>
<script type="text/javascript">
function showAdd() {
	var o=document.getElementById("add_tables")
	alert(o.style.display)
	if (o.style.display == "none") {
		o.style.display = "block";
	} else {
		o.style.display = "none";
	} 
}
</script>
</head>
<body>
<a href="javascript:void" onclick="showAdd();">Показать дополнительные поля</a>
<br />
<div id="add_tables">
	это блок
</div>
</body>
</html>


Никакого none... :)

Цитата:

Сообщение от devote
а проверять на присутствие block не рекомендуется, потому как оно может иметь значение отличное от block

Это кто такое сказал? :blink:
У автора только два значения
- block
- none

devote 10.08.2011 15:36

Цитата:

Сообщение от ksa
Это кто такое сказал?

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

ksa 10.08.2011 15:38

Цитата:

Сообщение от devote
что бы в будущем прог знал что стоит придерживаться того а не того

Несогласный я... :no:

devote 10.08.2011 15:41

Цитата:

Сообщение от ksa
Никакого none...

Да видимо я попутал с тем когда вводишь display: none; в в атрибуте style="" хотя конечно лучше юзать currentStyle или computedStyle (FF) там четко стоит то что нужно

devote 10.08.2011 15:44

Цитата:

Сообщение от ksa
Несогласный я...

Гогда начнешь писать какой нить фреймфорк или плаг где тебе не будет извесно значение элемента по умолчанию, тогда согласишься и будешь проверять на значение none.

ksa 10.08.2011 15:54

devote, я буду проверять на значение которое сам буду прописывать. :) А не то какое стоит в стилях...

sympler 10.08.2011 15:57

Цитата:

Сообщение от ksa (Сообщение 119510)
Условие поменяй...

<!DOCTYPE html>
<html>
<head>
<style type="text/css">
#add_tables {
	display: none;
}
</style>
<script type="text/javascript">
function showAdd() {
	var o=document.getElementById("add_tables")
	if (o.style.display == "block") {
		o.style.display = "none";
	} else {
		o.style.display = "block";
	} 
}
</script>
</head>
<body>
<a href="javascript:void" onclick="showAdd();">Показать дополнительные поля</a>
<br />
<div id="add_tables">
	это блок
</div>
</body>
</html>


Первый раз нужное тебе значение = пусто...

странно.. все один в один...
только не в одном файле а в разных css, js, html
ну и переменную "o" не использую. разве есть разница? (за исключением более быстрой обработки и меньшей длины кода)

блок по умолчанию скрыт... display:none прописан в отдельном файле css
но скрипт работает с первого раза только при использовании
<div id="add_tables" style="display:none;">

ksa 10.08.2011 15:59

Цитата:

Сообщение от sympler
странно.. все один в один...

У меня условие не такое как у тебя в примере...

Цитата:

Сообщение от sympler
блок по умолчанию скрыт... display:none прописан в отдельном файле css

Потому-то его "первое" значение для тебя "" (пусто), а не none, как ты ожидаешь... ;)

ksa 10.08.2011 16:00

Цитата:

Сообщение от sympler
но скрипт работает с первого раза только при использовании
<div id="add_tables" style="display:none;">

Вот тогда "первое" значение действительно none. :yes:

ksa 10.08.2011 16:01

Цитата:

Сообщение от sympler
и переменную "o" не использую. разве есть разница?

Текста меньше - уже хорошо. :write:


Часовой пояс GMT +3, время: 16:56.