Форум

ksa · #1 (**permalink**) 18.12.2013, 14:23

Вот пример...

<!DOCTYPE html>
<html>
<head>
<script src="http://code.jquery.com/jquery-latest.js"></script>
<!--
<link rel="stylesheet" type="text/css" href="tmp.css" />
-->
<style type="text/css">
.on {
	background-color: green;
}
</style>
<script type="text/javascript">
$(function (){
	$('li').click(function (){
		if (this.className=='on') {
			action();
		} else {
			select(this);
		};
	});
	$('li').dblclick(function (){
		select(this);
		action();
	});
});
function select(Obj) {
	$(Obj.parentNode).find('li').removeClass('on');
	Obj.className='on';
};
function action() {
	alert('Action!')
};
</script>
</head>
<body>
<ul>
	<li>Item 1</li>
	<li>Item 2</li>
	<li>Item 3</li>
</ul>
</body>
</html>

При клике элемент либо выделяется... Либо срабатывает "экшн", если элемент уже выделен...
На двойной клик навешено выделение и запуск экшна...

Но при таком раскладе при двойном клике выполняется клик+клик+двойной_клик и экшн запускается два раза.
Как обойти это?
Пока решил проблему через переменную...

Но это не нравится... Есть еще какие варианты?

Rise · #2 (**permalink**) 18.12.2013, 14:46

Ну убери вообще dblclick и будет то что надо...

рони · #3 (**permalink**) 18.12.2013, 14:51

ksa,

<!DOCTYPE html>
<html>
<head>
<script src="http://code.jquery.com/jquery-latest.js"></script>
<!--
<link rel="stylesheet" type="text/css" href="tmp.css" />
-->
<style type="text/css">
.on {
	background-color: green;
}
</style>
<script type="text/javascript">
$(function (){
	$('ul').on('click','li:not(".on")',function (){
		select(this)
	});
    $('ul').on('click','.on',function (){
		action();
	});
});
function select(Obj) {
	$(Obj.parentNode).find('li').removeClass('on');
	$(Obj).addClass('on');
};
function action() {
	alert('Action!')
};
</script>
</head>
<body>
<ul>
	<li>Item 1</li>
	<li>Item 2</li>
	<li>Item 3</li>
</ul>
</body>
</html>

ksa · #4 (**permalink**) 18.12.2013, 14:51

Сообщение от Rise

Ну убери вообще dblclick и будет то что надо...

Праильное решение! (с)

ksa · #5 (**permalink**) 18.12.2013, 14:54

рони, принято как рабочий вариант...

А как-то "остановить" события не получится?

рони · #6 (**permalink**) 18.12.2013, 15:00

Сообщение от ksa

А как-то "остановить" события не получится?

обьясни не понимаю ?
пока укороченный вариант ...

<!DOCTYPE html>
<html>
<head>
<script src="http://code.jquery.com/jquery-latest.js"></script>
<!--
<link rel="stylesheet" type="text/css" href="tmp.css" />
-->
<style type="text/css">
.on {
	background-color: green;
}
</style>
<script type="text/javascript">
$(function (){
	$('ul').on('click','li',function (){
	   $(this).hasClass('on') ? action():select(this)
	});

});
function select(Obj) {
	$(Obj).addClass('on').siblings().removeClass('on');
};
function action() {
	alert('Action!')
};
</script>
</head>
<body>
<ul>
	<li>Item 1</li>
	<li>Item 2</li>
	<li>Item 3</li>
</ul>
</body>
</html>

ksa · #7 (**permalink**) 18.12.2013, 15:32

Сообщение от рони

обьясни не понимаю ?

Да я что-то зациклился на этом двойном клике...

А он действительно не нужен тут.

Rise · #8 (**permalink**) 18.12.2013, 15:58

ksa, еще можно так:

<script src="//code.jquery.com/jquery-1.10.2.min.js"></script>

<style>.on{background-color:green;}</style>

<ul>
	<li>Item 1</li>
	<li>Item 2</li>
	<li>Item 3</li>
</ul>

<script>
$(function (){
	$('li').click( { p: 0 }, function(e) {
		if (e.data.p == this) {
			alert('Action!');
			return;
		}
		if (e.data.p) e.data.p.className = '';
		this.className = 'on';
		e.data.p = this;
	});
});
</script>

ksa · #9 (**permalink**) 18.12.2013, 16:08

Rise, спасибо.

Переделал все по первому варианту рони...

ksa · #10 (**permalink**) 18.12.2013, 16:28

Сообщение от ksa

Да я что-то зациклился на этом двойном клике...

Вот почему.
В ИЕ двойной клик это "отдельное" событие... И там по двойному клику все эти примеры не запустят экшн.

Форум

Справочник

Discord чат

Курсы javascript